Merge pull request #347 from pocke/fix-lambada-after-commit

[Fix #345] Fix error of `Rails/AfterCommitOverride` on `after_commit`with a lambda

CHANGELOG.md

@@ -2,6 +2,10 @@
 
 ## master (unreleased)
 
+### Bug fixes
+
+* [#345](https://github.com/rubocop-hq/rubocop-rails/issues/345): Fix error of `Rails/AfterCommitOverride` on `after_commit` with a lambda. ([@pocke][])
+
 ## 2.8.0 (2020-09-04)
 
 ### New features

lib/rubocop/cop/rails/after_commit_override.rb

@@ -59,7 +59,7 @@ def on_class(class_node)
 
         def each_after_commit_callback(class_node)
           class_send_nodes(class_node).each do |node|
-            yield node if after_commit_callback?(node)
+            yield node if after_commit_callback?(node) && named_callback?(node)
           end
         end
 
@@ -78,6 +78,13 @@ def class_send_nodes(class_node)
         def after_commit_callback?(node)
           AFTER_COMMIT_CALLBACKS.include?(node.method_name)
         end
+
+        def named_callback?(node)
+          name = node.first_argument
+          return false unless name
+
+          name.sym_type?
+        end
       end
     end
   end

spec/rubocop/cop/rails/after_commit_override_spec.rb

@@ -32,4 +32,22 @@ class User < ApplicationRecord
       end
     RUBY
   end
+
+  it 'does not register an offense when the callbacks are defined with lambdas' do
+    expect_no_offenses(<<~RUBY)
+      class User < ApplicationRecord
+        after_commit -> { foo }
+        after_destroy_commit -> { foo }
+      end
+    RUBY
+  end
+
+  it 'ignores after_commit method call without arguments' do
+    expect_no_offenses(<<~RUBY)
+      class User < ApplicationRecord
+        after_commit
+        after_destroy_commit
+      end
+    RUBY
+  end
 end