debian-mirror-gitlab/rubocop/cop/lint/last_keyword_argument.rb

Ignoring revisions in .git-blame-ignore-revs. Click here to bypass and see the normal blame view.

90 lines
2.8 KiB
Ruby
Raw Permalink Normal View History

2021-02-22 17:27:13 +05:30
# frozen_string_literal: true
module RuboCop
module Cop
module Lint
# This cop only works if there are files from deprecation_toolkit. You can
# generate these files by:
#
# 1. Running specs with RECORD_DEPRECATIONS=1
# 1. Downloading the complete set of deprecations/ files from a CI
# pipeline (see https://gitlab.com/gitlab-org/gitlab/-/merge_requests/47720)
2022-10-11 01:57:18 +05:30
class LastKeywordArgument < RuboCop::Cop::Base
extend RuboCop::Cop::AutoCorrector
2021-04-29 21:17:54 +05:30
MSG = 'Using the last argument as keyword parameters is deprecated'
2021-02-22 17:27:13 +05:30
DEPRECATIONS_GLOB = File.expand_path('../../../deprecations/**/*.yml', __dir__)
KEYWORD_DEPRECATION_STR = 'maybe ** should be added to the call'
def on_send(node)
2023-03-17 16:20:25 +05:30
return if target_ruby_version >= 3.0
2021-03-08 18:12:59 +05:30
arg = get_last_argument(node)
2021-02-22 17:27:13 +05:30
return unless arg
return unless known_match?(processed_source.file_path, node.first_line, node.method_name.to_s)
return if arg.children.first.respond_to?(:kwsplat_type?) && arg.children.first&.kwsplat_type?
# parser thinks `a: :b, c: :d` is hash type, it's actually kwargs
return if arg.hash_type? && !arg.source.match(/\A{/)
2022-10-11 01:57:18 +05:30
add_offense(arg) do |corrector|
2021-02-22 17:27:13 +05:30
if arg.hash_type?
kwarg = arg.source.sub(/\A{\s*/, '').sub(/\s*}\z/, '')
corrector.replace(arg, kwarg)
elsif arg.splat_type?
corrector.insert_before(arg, '*')
else
corrector.insert_before(arg, '**')
end
end
end
private
2021-03-08 18:12:59 +05:30
def get_last_argument(node)
return node.arguments[-2] if node.block_argument?
node.arguments.last
end
2021-02-22 17:27:13 +05:30
def known_match?(file_path, line_number, method_name)
method_name = 'initialize' if method_name == 'new'
2021-03-08 18:12:59 +05:30
return unless self.class.keyword_warnings[method_name]
2023-03-17 16:20:25 +05:30
file_path_from_root = file_path.sub(File.expand_path('../../..', __dir__), '')
file_and_line = "#{file_path_from_root}:#{line_number}"
2021-03-08 18:12:59 +05:30
self.class.keyword_warnings[method_name].any? do |warning|
warning.include?(file_and_line)
2021-02-22 17:27:13 +05:30
end
end
def self.keyword_warnings
@keyword_warnings ||= keywords_list
end
def self.keywords_list
hash = Dir.glob(DEPRECATIONS_GLOB).each_with_object({}) do |file, hash|
hash.merge!(YAML.safe_load(File.read(file)))
end
2021-03-08 18:12:59 +05:30
hash.values.flatten.each_with_object({}) do |str, results|
next unless str.include?(KEYWORD_DEPRECATION_STR)
match_data = str.match(/called method `([^\s]+)'/)
next unless match_data
key = match_data[1]
results[key] ||= []
results[key] << str
end
2021-02-22 17:27:13 +05:30
end
end
end
end
end