[Fix: rubocop#1110] Fix false positive for `Rails/RedundantActiveReco…

…rdAllMethod` when `all` has arguments

changelog/fix_false_for_rails_redundant_active_record_all_method_positive_when_all_has_arguments.md

@@ -0,0 +1 @@
+* [#1110](https://github.com/rubocop/rubocop-rails/issues/1110): Fix false positive for `Rails/RedundantActiveRecordAllMethod` when `all` has arguments. ([@masato-bkn][])

lib/rubocop/cop/rails/redundant_active_record_all_method.rb

@@ -127,7 +127,7 @@ class RedundantActiveRecordAllMethod < Base
         ].to_set.freeze
 
         def_node_matcher :followed_by_query_method?, <<~PATTERN
-          (send (send _ :all ...) QUERYING_METHODS ...)
+          (send (send _ :all) QUERYING_METHODS ...)
         PATTERN
 
         def on_send(node)

spec/rubocop/cop/rails/redundant_active_record_all_method_spec.rb

@@ -260,6 +260,26 @@
         RUBY
       end
     end
+
+    context 'when `all` has parameters (=not `all` is not Active Record method)' do
+      it 'does not register an offense' do
+        expect_no_offenses(<<~RUBY)
+          page.all(some_argument)
+        RUBY
+      end
+
+      it 'does not register an offense when method follows `all`' do
+        expect_no_offenses(<<~RUBY)
+          page.all(some_argument).do_something
+        RUBY
+      end
+
+      it 'does not register an offense when method from `ActiveRecord::Querying::QUERYING_METHODS` follows `all`' do
+        expect_no_offenses(<<~RUBY)
+          page.all(some_argument).select(some_filter)
+        RUBY
+      end
+    end
   end
 
   context 'with no receiver' do