class RuboCop::Cop::Bundler::DuplicatedGroup
gem ‘rspec’, groups: [:development, :test]
gem ‘rubocop’, groups: [:development, :test]
# good
end
gem ‘rspec’
group :development, :test do
end
gem ‘rubocop’
group :development do
# good
end
gem ‘rspec’
group :test, :development do
end
gem ‘rubocop’
group :development, :test do
# bad (same set of groups declared twice)
end
gem ‘rubocop-rails’
group :development do
end
gem ‘rubocop’
group :development do
# bad
@example
—–
end
end
gem ‘jruby-openssl’
group :default do
platforms :jruby do
end
end
gem ‘openssl’
group :default do
platforms :ruby do
—–
[source,ruby]
surrounding ‘group` are different, no offense will be registered:
For example, if the values of `source`, `git`, `platforms`, or `path`
A Gem group, or a set of groups, should be listed only once in a Gemfile.
def duplicated_group_nodes
def duplicated_group_nodes group_declarations = group_declarations(processed_source.ast) group_keys = group_declarations.group_by do |node| source_key = find_source_key(node) group_attributes = group_attributes(node).sort.join "#{source_key}#{group_attributes}" end group_keys.values.select { |nodes| nodes.size > 1 } end
def find_source_key(node)
def find_source_key(node) source_block = node.each_ancestor(:block).find do |block_node| SOURCE_BLOCK_NAMES.include?(block_node.method_name) end return unless source_block "#{source_block.method_name}#{source_block.send_node.first_argument&.source}" end
def group_attributes(node)
def group_attributes(node) node.arguments.map do |argument| if argument.hash_type? argument.pairs.map(&:source).sort.join(', ') else argument.respond_to?(:value) ? argument.value.to_s : argument.source end end end
def on_new_investigation
def on_new_investigation return if processed_source.blank? duplicated_group_nodes.each do |nodes| nodes[1..].each do |node| group_name = node.arguments.map(&:source).join(', ') register_offense(node, group_name, nodes.first.first_line) end end end
def register_offense(node, group_name, line_of_first_occurrence)
def register_offense(node, group_name, line_of_first_occurrence) line_range = node.loc.column...node.loc.last_column offense_location = source_range(processed_source.buffer, node.first_line, line_range) message = format( MSG, group_name: group_name, line_of_first_occurrence: line_of_first_occurrence ) add_offense(offense_location, message: message) end