Avoid duplicating definition of ActiveRecord::Relation.destroy_all

[reu]

Motivation

destroy_all is defined both as a hardcoded common method (line 612) and dynamically via the RELATION_METHODS loop (line 862), since ActiveRecord::Relation includes destroy_all as an instance method. This results in a duplicated def destroy_all; end signature in the generated RBI files.

Implementation

Added a guard (unless RELATION_METHODS.include? :destroy_all) to the hardcoded destroy_all definition so it is only generated when destroy_all is not already part of RELATION_METHODS. When ActiveRecord includes it in Relation.instance_methods(false), the dynamic loop handles it and the hardcoded one is skipped, avoiding the duplicate.

Updated the expected output in the spec to remove the duplicated destroy_all entries.

Tests

Updated existing specs to reflect the corrected (non-duplicated) output.

[reu]

I have signed the CLA!