lib/rubocop/cop/performance/ancestors_include.rb
# frozen_string_literal: true module RuboCop module Cop module Performance # Identifies usages of `ancestors.include?` and change them to use `<=` instead. # # @safety # This cop is unsafe because it can't tell whether the receiver is a class or an object. # e.g. the false positive was for `Nokogiri::XML::Node#ancestors`. # # @example # # bad # A.ancestors.include?(B) # # # good # A <= B # class AncestorsInclude < Base include RangeHelp extend AutoCorrector MSG = 'Use `<=` instead of `ancestors.include?`.' RESTRICT_ON_SEND = %i[include?].freeze def_node_matcher :ancestors_include_candidate?, <<~PATTERN (send (send $_subclass :ancestors) :include? $_superclass) PATTERN def on_send(node) return unless (subclass, superclass = ancestors_include_candidate?(node)) return if subclass && !subclass.const_type? add_offense(range(node)) do |corrector| subclass_source = subclass ? subclass.source : 'self' corrector.replace(node, "#{subclass_source} <= #{superclass.source}") end end private def range(node) location_of_ancestors = node.children[0].loc.selector.begin_pos end_location = node.loc.selector.end_pos range_between(location_of_ancestors, end_location) end end end end end