lib/rubocop/cop/lint/unneeded_splat_expansion.rb in rubocop-0.50.0 vs lib/rubocop/cop/lint/unneeded_splat_expansion.rb in rubocop-0.51.0
- old
+ new
@@ -54,11 +54,11 @@
ARRAY_PARAM_MSG = 'Pass array contents as separate arguments.'.freeze
PERCENT_W = '%w'.freeze
PERCENT_CAPITAL_W = '%W'.freeze
PERCENT_I = '%i'.freeze
PERCENT_CAPITAL_I = '%I'.freeze
- ARRAY_NEW_PATTERN = '$(send (const nil :Array) :new ...)'.freeze
+ ARRAY_NEW_PATTERN = '$(send (const nil? :Array) :new ...)'.freeze
ASSIGNMENT_TYPES = %i[lvasgn ivasgn cvasgn gvasgn].freeze
def_node_matcher :literal_expansion?, <<-PATTERN
(splat {$({str dstr int float array} ...) (block #{ARRAY_NEW_PATTERN} ...) #{ARRAY_NEW_PATTERN}} ...)
PATTERN
@@ -69,10 +69,10 @@
return unless ASSIGNMENT_TYPES.include?(node.parent.parent.type)
end
if array_splat?(node) &&
(method_argument?(node) || part_of_an_array?(node))
- add_offense(node, :expression, ARRAY_PARAM_MSG)
+ add_offense(node, message: ARRAY_PARAM_MSG)
else
add_offense(node)
end
end
end