Rails/FindEach not safe-auto-correct

[altherlex]

My class is not ActiveRecord but it has a method called all

Rubocop-rails changes from .all.each to all.find_each

[andyw8]

Which version of the gem are you using? The spec indicates it should ignore non-AR classes.

rubocop-rails/spec/rubocop/cop/rails/find_each_spec.rb

Lines 94 to 100 in 944394b

	it 'does not register an offense when not inheriting `ApplicationRecord`' do
	expect_no_offenses(<<~RUBY)
	class C < Foo
	all.each { |u| u.x }
	end
	RUBY
	end

[altherlex]

rubocop (1.26.1)
rubocop-rails (2.14.2)

[altherlex]

it is a module thou

module Symptoms
  def self.all
    data
  end
end

Symptoms.all.each

[palkan]

Surprisingly, it's not safe even for Active Record:

users = User.where(admin: true).each(&:itself) #=> [<User...>, ...]

users = User.where(admin: true).find_each(&:itself) #=> nil

[TastyPi]

I've just hit this using 2.17.4, it's trying to convert each to find_each when the object is an Array. The class the call is in is not an ActiveRecord, though it is defined under app/models.

It is using ActiveModel::Model, might that be the issue? I can try to make a minimal reproduction if that helps.