cookstyle 7.30.1 → 7.31.1

data/lib/rubocop/cop/chef/deprecation/resource_overrides_provides_method.rb

@@ -37,9 +37,9 @@ module RuboCop
           def_node_search :calls_provides?, '(send nil? :provides ...)'
 
           def on_def(node)
-            return unless node.method_name == :provides?
+            return unless node.method?(:provides?)
 
-            add_offense(node, message: MSG, severity: :warning) unless calls_provides?(processed_source.ast)
+            add_offense(node, severity: :warning) unless calls_provides?(processed_source.ast)
           end
         end
       end

data/lib/rubocop/cop/chef/deprecation/resource_uses_dsl_name_method.rb

@@ -34,7 +34,7 @@ module RuboCop
           RESTRICT_ON_SEND = [:dsl_name].freeze
 
           def on_send(node)
-            add_offense(node, message: MSG, severity: :warning)
+            add_offense(node, severity: :warning)
           end
 
           # potential autocorrect is new_resource.updated_by_last_action true, but we need to actually see what class we were called from

data/lib/rubocop/cop/chef/deprecation/resource_uses_only_resource_name.rb

@@ -68,7 +68,7 @@ module RuboCop
           def on_send(node)
             resource_name?(node) do |name|
               return if valid_provides?(name)
-              add_offense(node, message: MSG, severity: :warning) do |corrector|
+              add_offense(node, severity: :warning) do |corrector|
                 if name.to_s == "#{cookbook_name}_#{File.basename(processed_source.path, '.rb')}"
                   corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
                 else

data/lib/rubocop/cop/chef/deprecation/resource_uses_provider_base_method.rb

@@ -31,7 +31,7 @@ module RuboCop
           RESTRICT_ON_SEND = [:provider_base].freeze
 
           def on_send(node)
-            add_offense(node, message: MSG, severity: :warning)
+            add_offense(node, severity: :warning)
           end
         end
       end

data/lib/rubocop/cop/chef/deprecation/resource_uses_updated_method.rb

@@ -38,7 +38,7 @@ module RuboCop
           MSG = "Don't use updated = true/false to update resource state. This will cause failures in Chef Infra Client 13 and later."
 
           def on_lvasgn(node)
-            add_offense(node, message: MSG, severity: :warning) if node.node_parts.first == :updated
+            add_offense(node, severity: :warning) if node.node_parts.first == :updated
           end
 
           # potential autocorrect is new_resource.updated_by_last_action true, but we need to actually see what class we were called from

data/lib/rubocop/cop/chef/deprecation/resource_without_unified_mode_true.rb

@@ -60,7 +60,7 @@ module RuboCop
             # Using range similar to RuboCop::Cop::Naming::Filename (file_name.rb)
             return if unified_mode?(processed_source.ast)
             range = source_range(processed_source.buffer, 1, 0)
-            add_offense(range, message: MSG, severity: :refactor) do |corrector|
+            add_offense(range, severity: :refactor) do |corrector|
               insert_below_provides(corrector) || insert_below_resource_name(corrector)
             end
           end

data/lib/rubocop/cop/chef/deprecation/ruby_27_keyword_argument_warnings.rb

@@ -44,7 +44,7 @@ module RuboCop
           def on_send(node)
             positional_shellout?(node) do |h|
               next unless h.braces?
-              add_offense(h, message: MSG, severity: :refactor) do |corrector|
+              add_offense(h, severity: :refactor) do |corrector|
                 corrector.replace(h, h.source[1..-2])
               end
             end

data/lib/rubocop/cop/chef/deprecation/ruby_block_create_action.rb

@@ -49,7 +49,7 @@ module RuboCop
             match_property_in_resource?(:ruby_block, 'action', node) do |ruby_action|
               ruby_action.arguments.each do |action|
                 next unless action.source == ':create'
-                add_offense(action, message: MSG, severity: :warning) do |corrector|
+                add_offense(action, severity: :warning) do |corrector|
                   corrector.replace(action, ':run')
                 end
               end

data/lib/rubocop/cop/chef/deprecation/run_command_helper.rb

@@ -45,15 +45,15 @@ module RuboCop
 
           def on_send(node)
             calls_run_command?(node) do
-              add_offense(node, message: MSG, severity: :warning) unless defines_run_command?(processed_source.ast)
+              add_offense(node, severity: :warning) unless defines_run_command?(processed_source.ast)
             end
 
             require_mixin_command?(node) do
-              add_offense(node, message: MSG, severity: :warning)
+              add_offense(node, severity: :warning)
             end
 
             include_mixin_command?(node) do
-              add_offense(node, message: MSG, severity: :warning)
+              add_offense(node, severity: :warning)
             end
           end
         end

data/lib/rubocop/cop/chef/deprecation/search_uses_positional_parameters.rb

@@ -49,7 +49,7 @@ module RuboCop
 
           def on_send(node)
             search_method?(node) do
-              add_offense(node, message: MSG, severity: :warning) do |corrector|
+              add_offense(node, severity: :warning) do |corrector|
                 corrector.replace(node, corrected_string(node))
               end if positional_arguments?(node)
             end

data/lib/rubocop/cop/chef/deprecation/use_automatic_resource_name.rb

@@ -40,7 +40,7 @@ module RuboCop
           RESTRICT_ON_SEND = [:use_automatic_resource_name].freeze
 
           def on_send(node)
-            add_offense(node.loc.selector, message: MSG, severity: :warning) do |corrector|
+            add_offense(node.loc.selector, severity: :warning) do |corrector|
               corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
             end
           end

data/lib/rubocop/cop/chef/deprecation/use_inline_resources.rb

@@ -46,7 +46,7 @@ module RuboCop
               node = node.parent
             end
 
-            add_offense(node, message: MSG, severity: :warning) do |corrector|
+            add_offense(node, severity: :warning) do |corrector|
               corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
             end
           end

data/lib/rubocop/cop/chef/deprecation/use_yaml_dump.rb

@@ -36,7 +36,7 @@ module RuboCop
           RESTRICT_ON_SEND = [:to_yaml].freeze
 
           def on_send(node)
-            add_offense(node, message: MSG, severity: :warning) do |corrector|
+            add_offense(node, severity: :warning) do |corrector|
               corrector.replace(node, "YAML.dump(#{node.receiver.source})")
             end
           end

data/lib/rubocop/cop/chef/deprecation/user_supports_property.rb

@@ -49,13 +49,13 @@ module RuboCop
 
           def on_block(node)
             match_property_in_resource?(:user, 'supports', node) do |property|
-              add_offense(property, message: MSG, severity: :warning) do |corrector|
+              add_offense(property, severity: :warning) do |corrector|
                 new_text = []
 
                 property.arguments.first.each_pair do |k, v|
                   # account for a strange edge case where the person incorrectly makes "manage_home a method
                   # the code would be broken, but without this handling cookstyle would explode
-                  key_value = (k.send_type? && k.method_name == :manage_home) ? 'manage_home' : k.value
+                  key_value = (k.send_type? && k.method?(:manage_home)) ? 'manage_home' : k.value
 
                   new_text << "#{key_value} #{v.source}"
                 end

data/lib/rubocop/cop/chef/deprecation/verify_property_file_expansion.rb

@@ -45,7 +45,7 @@ module RuboCop
           def on_block(node)
             match_property_in_resource?(nil, 'verify', node) do |verify|
               return unless verify.source.match?(/%{file}/)
-              add_offense(verify, message: MSG, severity: :warning) do |corrector|
+              add_offense(verify, severity: :warning) do |corrector|
                 corrector.replace(verify.loc.expression, verify.loc.expression.source.gsub('%{file}', '%{path}'))
               end
             end

data/lib/rubocop/cop/chef/deprecation/windows_feature_servermanagercmd.rb

@@ -48,7 +48,7 @@ module RuboCop
 
           def on_block(node)
             match_property_in_resource?(:windows_feature, :install_method, node) do |prop_node|
-              add_offense(prop_node, message: MSG, severity: :warning) if prop_node.source.include?(':servermanagercmd')
+              add_offense(prop_node, severity: :warning) if prop_node.source.include?(':servermanagercmd')
             end
           end
         end

data/lib/rubocop/cop/chef/deprecation/windows_package_installer_type_string.rb

@@ -48,7 +48,7 @@ module RuboCop
               return unless offense.arguments.count == 1 # we can only analyze simple string args
               return unless offense.arguments.first.str_type? # anything else is fine
 
-              add_offense(offense, message: MSG, severity: :warning) do |corrector|
+              add_offense(offense, severity: :warning) do |corrector|
                 corrector.replace(offense.arguments.first.source_range, ":#{offense.arguments.first.value}")
               end
             end

data/lib/rubocop/cop/chef/deprecation/windows_task_change_action.rb

@@ -65,7 +65,7 @@ module RuboCop
 
           def check_action(ast_obj)
             if ast_obj.respond_to?(:value) && ast_obj.value == :change
-              add_offense(ast_obj, message: MSG, severity: :warning) do |corrector|
+              add_offense(ast_obj, severity: :warning) do |corrector|
                 if ast_obj.parent.send_type? # :change was the only action
                   corrector.replace(ast_obj, ':create')
                 # chances are it's [:create, :change] since that's all that makes sense, but double check that theory

data/lib/rubocop/cop/chef/deprecation/windows_version_helpers.rb

@@ -50,7 +50,7 @@ module RuboCop
 
           def on_send(node)
             windows_helper?(node) do |method|
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 case method
                 when :nt_version
                   corrector.replace(node, 'node[\'platform_version\'].to_f')

data/lib/rubocop/cop/chef/deprecation/xml_ruby_recipe.rb

@@ -39,7 +39,7 @@ module RuboCop
           def on_send(node)
             xml_ruby_recipe?(node) do
               node = node.parent if node.parent&.conditional? && node.parent&.single_line_condition? # make sure we catch any inline conditionals
-              add_offense(node, message: MSG, severity: :warning) do |corrector|
+              add_offense(node, severity: :warning) do |corrector|
                 corrector.remove(node)
               end
             end

data/lib/rubocop/cop/chef/deprecation/yum_dnf_compat_recipe.rb

@@ -41,7 +41,7 @@ module RuboCop
           def on_send(node)
             yum_dnf_compat_recipe_usage?(node) do
               node = node.parent if node.parent&.conditional? && node.parent&.single_line?
-              add_offense(node, message: MSG, severity: :warning) do |corrector|
+              add_offense(node, severity: :warning) do |corrector|
                 corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
               end
             end

data/lib/rubocop/cop/chef/effortless/berksfile.rb

@@ -30,7 +30,7 @@ module RuboCop
             # Using range similar to RuboCop::Cop::Naming::Filename (file_name.rb)
             range = source_range(processed_source.buffer, 1, 0)
 
-            add_offense(range, message: MSG, severity: :refactor)
+            add_offense(range, severity: :refactor)
           end
         end
       end

data/lib/rubocop/cop/chef/effortless/chef_vault_used.rb

@@ -72,12 +72,12 @@ module RuboCop
             return unless require?(node) ||
                           chef_vault_item_for_environment?(node) ||
                           chef_vault_item?(node)
-            add_offense(node.loc.expression, message: MSG, severity: :refactor)
+            add_offense(node.loc.expression, severity: :refactor)
           end
 
           def on_const(node)
             vault_const?(node) do
-              add_offense(node.loc.expression, message: MSG, severity: :refactor)
+              add_offense(node.loc.expression, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/effortless/data_bags.rb

@@ -31,7 +31,7 @@ module RuboCop
           RESTRICT_ON_SEND = [:data_bag, :data_bag_item].freeze
 
           def on_send(node)
-            add_offense(node, message: MSG, severity: :refactor)
+            add_offense(node, severity: :refactor)
           end
         end
       end

data/lib/rubocop/cop/chef/effortless/depends_chef_vault.rb

@@ -37,7 +37,7 @@ module RuboCop
 
           def on_send(node)
             depends?(node) do
-              add_offense(node.loc.expression, message: MSG, severity: :refactor)
+              add_offense(node.loc.expression, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/effortless/node_environment.rb

@@ -34,8 +34,8 @@ module RuboCop
           def on_send(node)
             if node.receiver &&
                node.receiver.send_type? &&
-               node.receiver.method_name == :node
-              add_offense(node, message: MSG, severity: :refactor)
+               node.receiver.method?(:node)
+              add_offense(node, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/effortless/node_policygroup.rb

@@ -33,8 +33,8 @@ module RuboCop
           def on_send(node)
             if node.receiver &&
                node.receiver.send_type? &&
-               node.receiver.method_name == :node
-              add_offense(node, message: MSG, severity: :refactor)
+               node.receiver.method?(:node)
+              add_offense(node, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/effortless/node_roles.rb

@@ -34,8 +34,8 @@ module RuboCop
           def on_send(node)
             if node.receiver &&
                node.receiver.send_type? &&
-               node.receiver.method_name == :node
-              add_offense(node, message: MSG, severity: :refactor)
+               node.receiver.method?(:node)
+              add_offense(node, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/effortless/search_for_environments_or_roles.rb

@@ -33,7 +33,7 @@ module RuboCop
 
           def on_send(node)
             if node.arguments[1]&.value&.match?(/chef_environment|role/)
-              add_offense(node, message: MSG, severity: :refactor)
+              add_offense(node, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/effortless/search_used.rb

@@ -31,7 +31,7 @@ module RuboCop
           RESTRICT_ON_SEND = [:search].freeze
 
           def on_send(node)
-            add_offense(node, message: MSG, severity: :refactor)
+            add_offense(node, severity: :refactor)
           end
         end
       end

data/lib/rubocop/cop/chef/modernize/action_method_in_resource.rb

@@ -45,7 +45,7 @@ module RuboCop
             return if node.arguments? # if they passed in arguments they may actually need this
             return if node.parent && includes_poise?(node.parent)
 
-            add_offense(node, message: MSG, severity: :refactor) do |corrector|
+            add_offense(node, severity: :refactor) do |corrector|
               # @todo when we drop ruby 2.4 support we can convert this to use delete_suffix
               corrector.replace(node, node.source.gsub("def #{node.method_name}", "action :#{node.method_name.to_s.gsub(/^action_/, '')} do"))
             end

data/lib/rubocop/cop/chef/modernize/allowed_actions_initializer.rb

@@ -45,7 +45,7 @@ module RuboCop
           MSG = 'The allowed actions of a resource can be set with the "allowed_actions" helper instead of using the initialize method.'
 
           def on_def(node)
-            return unless node.method_name == :initialize
+            return unless node.method?(:initialize)
             return if node.body.nil? # nil body is an empty initialize method
 
             node.body.each_node do |x|
@@ -54,7 +54,7 @@ module RuboCop
                           !x.node_parts.empty? &&
                           %i(@actions @allowed_actions).include?(x.node_parts.first)
 
-              add_offense(x, message: MSG, severity: :refactor) do |corrector|
+              add_offense(x, severity: :refactor) do |corrector|
                 # insert the new allowed_actions call above the initialize method, but not if one already exists (this is sadly common)
                 unless action_methods?(processed_source.ast)
                   initialize_node = initialize_method(processed_source.ast).first

data/lib/rubocop/cop/chef/modernize/apt_default_recipe.rb

@@ -44,7 +44,7 @@ module RuboCop
 
           def on_send(node)
             apt_recipe_usage?(node) do
-              add_offense(node, message: MSG, severity: :refactor)
+              add_offense(node, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/modernize/berksfile_source.rb

@@ -52,13 +52,13 @@ module RuboCop
 
           def on_send(node)
             berksfile_source?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.replace(node, "source 'https://supermarket.chef.io'")
               end
             end
 
             berksfile_site?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.replace(node, "source 'https://supermarket.chef.io'")
               end
             end

data/lib/rubocop/cop/chef/modernize/build_essential.rb

@@ -43,7 +43,7 @@ module RuboCop
 
           def on_send(node)
             build_essential_recipe_usage?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.replace(node, "build_essential 'install compilation tools'")
               end
             end

data/lib/rubocop/cop/chef/modernize/chef_14_resources.rb

@@ -48,7 +48,7 @@ module RuboCop
 
           def on_send(node)
             legacy_depends?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
               end
             end

data/lib/rubocop/cop/chef/modernize/chef_15_resources.rb

@@ -45,7 +45,7 @@ module RuboCop
 
           def on_send(node)
             legacy_depends?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
               end
             end

data/lib/rubocop/cop/chef/modernize/chef_gem_nokogiri.rb

@@ -41,7 +41,7 @@ module RuboCop
           def on_block(node)
             match_property_in_resource?(:chef_gem, 'package_name', node) do |pkg_name|
               return unless pkg_name.arguments&.first&.str_content == 'nokogiri'
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 node = node.parent if node.parent&.block_type? # make sure we get the whole block not just the method in the block
                 corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
               end
@@ -50,7 +50,7 @@ module RuboCop
 
           def on_send(node)
             nokogiri_install?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 node = node.parent if node.parent&.block_type? # make sure we get the whole block not just the method in the block
                 corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
               end

data/lib/rubocop/cop/chef/modernize/class_eval_action_class.rb

@@ -51,7 +51,7 @@ module RuboCop
 
           def on_block(node)
             class_eval_action_class?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.replace(node, node.source.gsub('.class_eval', ''))
               end
             end

data/lib/rubocop/cop/chef/modernize/compile_time_resources.rb

@@ -43,7 +43,7 @@ module RuboCop
 
           def on_send(node)
             compile_time_resource?(node) do
-              add_offense(node, message: MSG, severity: :refactor)
+              add_offense(node, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/modernize/conditional_using_test.rb

@@ -42,7 +42,7 @@ module RuboCop
           def on_send(node)
             resource_conditional?(node) do |conditional|
               return unless conditional.value.match?(/^test -[ef] \S*$/)
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 new_string = "{ ::File.exist?('#{conditional.value.match(/^test -[ef] (\S*)$/)[1]}') }"
                 corrector.replace(conditional, new_string)
               end

data/lib/rubocop/cop/chef/modernize/cron_d_file_or_template.rb

@@ -78,7 +78,7 @@ module RuboCop
           def on_block(node)
             file_or_template?(node) do |file_name|
               return unless file_name.start_with?('/etc/cron.d/')
-              add_offense(node, message: MSG, severity: :refactor)
+              add_offense(node, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/modernize/cron_manage_resource.rb

@@ -39,7 +39,7 @@ module RuboCop
           RESTRICT_ON_SEND = [:cron_manage].freeze
 
           def on_send(node)
-            add_offense(node, message: MSG, severity: :refactor) do |corrector|
+            add_offense(node, severity: :refactor) do |corrector|
               corrector.replace(node, node.source.gsub(/^cron_manage/, 'cron_access'))
             end
           end

data/lib/rubocop/cop/chef/modernize/databag_helpers.rb

@@ -46,7 +46,7 @@ module RuboCop
 
           def on_send(node)
             data_bag_class_load?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.replace(node,
                    node.source.gsub(/Chef::(EncryptedDataBagItem|DataBagItem).load/, 'data_bag_item'))
               end

data/lib/rubocop/cop/chef/modernize/declare_action_class.rb

@@ -43,7 +43,7 @@ module RuboCop
           RESTRICT_ON_SEND = [:declare_action_class].freeze
 
           def on_send(node)
-            add_offense(node, message: MSG, severity: :refactor) do |corrector|
+            add_offense(node, severity: :refactor) do |corrector|
               corrector.replace(node, node.source.gsub('declare_action_class', 'action_class'))
             end
           end

data/lib/rubocop/cop/chef/modernize/default_action_initializer.rb

@@ -55,7 +55,7 @@ module RuboCop
           def on_ivasgn(node)
             action_variable_assignment?(node) do
               return unless initialize_method(node.parent.parent)
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 # insert the new default_action call above the initialize method, but not if one already exists (this is sadly common)
                 unless default_action_method?(processed_source.ast)
                   initialize_node = initialize_method(processed_source.ast).first

data/lib/rubocop/cop/chef/modernize/defines_chefspec_matchers.rb

@@ -42,7 +42,7 @@ module RuboCop
 
           def on_if(node)
             chefspec_matcher?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
               end
             end

data/lib/rubocop/cop/chef/modernize/definitions.rb

@@ -27,7 +27,7 @@ module RuboCop
           MSG = 'Legacy Chef Infra definitions should be rewritten as custom resources to take full advantage of the Chef Infra feature set.'
 
           def on_block(node)
-            add_offense(node, message: MSG, severity: :refactor) if node.respond_to?(:method_name) && node.method_name == :define
+            add_offense(node, severity: :refactor) if node.respond_to?(:method_name) && node.method?(:define)
           end
         end
       end

data/lib/rubocop/cop/chef/modernize/depends_chef_vault_cookbook.rb

@@ -42,7 +42,7 @@ module RuboCop
 
           def on_send(node)
             legacy_depends?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
               end
             end

data/lib/rubocop/cop/chef/modernize/depends_chocolatey_cookbooks.rb

@@ -43,7 +43,7 @@ module RuboCop
 
           def on_send(node)
             legacy_depends?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
               end
             end

data/lib/rubocop/cop/chef/modernize/depends_kernel_module_cookbook.rb

@@ -42,7 +42,7 @@ module RuboCop
 
           def on_send(node)
             legacy_depends?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
               end
             end

data/lib/rubocop/cop/chef/modernize/depends_locale_cookbook.rb

@@ -42,7 +42,7 @@ module RuboCop
 
           def on_send(node)
             legacy_depends?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
               end
             end

data/lib/rubocop/cop/chef/modernize/depends_openssl_cookbook.rb

@@ -42,7 +42,7 @@ module RuboCop
 
           def on_send(node)
             legacy_depends?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
               end
             end

data/lib/rubocop/cop/chef/modernize/depends_timezone_lwrp_cookbook.rb

@@ -42,7 +42,7 @@ module RuboCop
 
           def on_send(node)
             legacy_depends?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
               end
             end

data/lib/rubocop/cop/chef/modernize/depends_windows_firewall_cookbook.rb

@@ -42,7 +42,7 @@ module RuboCop
 
           def on_send(node)
             legacy_depends?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
               end
             end