cookstyle 7.30.1 → 7.30.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: af05586e951fed9a9a11777583d2f8b0ec7634cca38b3b5b9641e2afb11be50b
-  data.tar.gz: 8d727e0f4898bf20149fa74ee6d15782840577c98eeb9fe25048ae71ab13c4b1
+  metadata.gz: 94be7df501ab0a3a055ee36aab695764830d816e3db7005a7b18231d3a703505
+  data.tar.gz: 3cf466a2d3fec655c473df01a769bb9e577359ed04ea3569f6cc6fb0f5c59ea0
 SHA512:
-  metadata.gz: dbd9a0129be207bd120c0ba860260726c4547b8e0356eda5c221b70d29aaf9ec92fdce5af7450b919f8a74774b79820c128220749e883c226de4c82f45c21915
-  data.tar.gz: 74e57634c9aef93e952858a542cac3966c849fa3e1deb2dd5b0ef4b7cd9f8167eba767e55635b72c2cbc7f3d08228a3fed26ed4f2e072e4eb6d8c78a09c3b151
+  metadata.gz: 339ccca52365ef809d5976ceadcea5ead13bcef57b294725a26d0f6e5ddfb306f1e003c72d1fd99448640018f0da00d4188ea48adfda39593b2cc02cc541d95e
+  data.tar.gz: d02e13fed47f62938a3e3f3dd07c86c8873294bd9fbb96395534f33419b62084a2814581717c8540db139668a67657e009bb3060432a553b111e99c1d0830bc5

data/config/upstream.yml

@@ -2496,6 +2496,7 @@ Naming/BlockForwarding:
   SupportedStyles:
     - anonymous
     - explicit
+  BlockForwardingName: block
 
 Naming/BlockParameterName:
   Description: >-
@@ -3698,6 +3699,8 @@ Style/HashSyntax:
     - always
     # forces use of explicit hash literal value.
     - never
+    # accepts both shorthand and explicit use of hash literal value.
+    - either
   # Force hashes that have a symbol value to use hash rockets
   UseHashRocketsWithSymbolValues: false
   # Do not suggest { a?: 1 } over { :a? => 1 } in ruby19 style

data/lib/cookstyle/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 module Cookstyle
-  VERSION = "7.30.1" # rubocop: disable Style/StringLiterals
-  RUBOCOP_VERSION = '1.24.1'
+  VERSION = "7.30.3" # rubocop: disable Style/StringLiterals
+  RUBOCOP_VERSION = '1.25.0'
 end

data/lib/rubocop/cop/chef/correctness/block_guard_clause_string_only.rb

@@ -48,7 +48,7 @@ module RuboCop
 
           def on_block(node)
             block_guard_with_only_string?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 new_val = "#{node.method_name} #{node.body.source}"
                 corrector.replace(node, new_val)
               end

data/lib/rubocop/cop/chef/correctness/chef_application_fatal.rb

@@ -44,7 +44,7 @@ module RuboCop
 
           def on_send(node)
             application_fatal?(node) do |val|
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.replace(node, "raise(#{val.source})")
               end
             end

data/lib/rubocop/cop/chef/correctness/conditional_ruby_shellout.rb

@@ -57,7 +57,7 @@ module RuboCop
 
           def on_block(node)
             conditional_shellout?(node) do |type, val|
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.replace(node, "#{type} #{val.source}")
               end
             end

data/lib/rubocop/cop/chef/correctness/dnf_package_allow_downgrades.rb

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

data/lib/rubocop/cop/chef/correctness/incorrect_library_injection.rb

@@ -60,7 +60,7 @@ module RuboCop
 
           def on_send(node)
             legacy_injection?(node) do
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 if node.parent && correct_injection?(node.parent)
                   corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
                 else

data/lib/rubocop/cop/chef/correctness/invalid_cookbook_name.rb

@@ -37,7 +37,7 @@ module RuboCop
 
           def on_send(node)
             has_name?(node) do |val|
-              add_offense(node, message: MSG, severity: :refactor) if val.value.include?('.')
+              add_offense(node, severity: :refactor) if val.value.include?('.')
             end
           end
         end

data/lib/rubocop/cop/chef/correctness/invalid_default_action.rb

@@ -38,7 +38,7 @@ module RuboCop
           def on_send(node)
             default_action?(node) do |match|
               return if %i(send sym array).include?(match.type)
-              add_offense(node, message: MSG, severity: :refactor)
+              add_offense(node, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/correctness/invalid_notification_resource.rb

@@ -49,7 +49,7 @@ module RuboCop
 
           def on_send(node)
             invalid_notification?(node) do |resource|
-              add_offense(resource, message: MSG, severity: :refactor)
+              add_offense(resource, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/correctness/invalid_notification_timing.rb

@@ -45,7 +45,7 @@ module RuboCop
 
           def on_send(node)
             notification_with_timing?(node) do |timing|
-              add_offense(timing, message: MSG, severity: :refactor) unless %i(immediate immediately delayed before).include?(timing.value)
+              add_offense(timing, severity: :refactor) unless %i(immediate immediately delayed before).include?(timing.value)
             end
           end
         end

data/lib/rubocop/cop/chef/correctness/invalid_platform_family_helper.rb

@@ -47,7 +47,7 @@ module RuboCop
             platform_family_helper?(node) do |plats|
               plats.to_a.each do |p|
                 next unless INVALID_PLATFORM_FAMILIES.key?(p.value)
-                add_offense(p, message: MSG, severity: :refactor) do |corrector|
+                add_offense(p, severity: :refactor) do |corrector|
                   replacement_platform = INVALID_PLATFORM_FAMILIES[p.value]
                   all_passed_platforms = p.parent.arguments.map(&:value)
 

data/lib/rubocop/cop/chef/correctness/invalid_platform_family_values_in_case.rb

@@ -50,7 +50,7 @@ module RuboCop
                   new_value = INVALID_PLATFORM_FAMILIES[con.str_content]
                   next unless new_value
 
-                  add_offense(con, message: MSG, severity: :refactor) do |corrector|
+                  add_offense(con, severity: :refactor) do |corrector|
                     # if the correct value already exists in the when statement then we just want to delete this node
                     if con.parent.conditions.any? { |x| x.str_content == new_value }
                       range = range_with_surrounding_comma(range_with_surrounding_space(range: con.loc.expression, side: :left), :both)

data/lib/rubocop/cop/chef/correctness/invalid_platform_helper.rb

@@ -46,7 +46,7 @@ module RuboCop
           def on_send(node)
             platform_helper?(node) do |plat|
               plat.to_a.each do |p|
-                add_offense(p, message: MSG, severity: :refactor) if INVALID_PLATFORMS.key?(p.value)
+                add_offense(p, severity: :refactor) if INVALID_PLATFORMS.key?(p.value)
               end
             end
           end

data/lib/rubocop/cop/chef/correctness/invalid_platform_metadata.rb

@@ -52,7 +52,7 @@ module RuboCop
           def on_send(node)
             supports?(node) do |plat|
               next unless INVALID_PLATFORMS[plat.str_content]
-              add_offense(plat, message: MSG, severity: :refactor) do |corrector|
+              add_offense(plat, severity: :refactor) do |corrector|
                 correct_string = corrected_platform_source(plat)
                 next unless correct_string
                 corrector.replace(plat, correct_string)
@@ -64,7 +64,7 @@ module RuboCop
             supports_array?(node) do |plats|
               plats.values.each do |plat|
                 next unless INVALID_PLATFORMS[plat.str_content]
-                add_offense(plat, message: MSG, severity: :refactor) do |corrector|
+                add_offense(plat, severity: :refactor) do |corrector|
                   correct_string = corrected_platform_source(plat)
                   next unless correct_string
                   corrector.replace(plat, correct_string)

data/lib/rubocop/cop/chef/correctness/invalid_platform_values_in_case.rb

@@ -50,7 +50,7 @@ module RuboCop
                   # some invalid platform have no direct correction value and return nil instead
                   next unless new_value
 
-                  add_offense(con, message: MSG, severity: :refactor) do |corrector|
+                  add_offense(con, severity: :refactor) do |corrector|
                     # if the correct value already exists in the when statement then we just want to delete this node
                     if con.parent.conditions.any? { |x| x.str_content == new_value }
                       range = range_with_surrounding_comma(range_with_surrounding_space(range: con.loc.expression, side: :left), :both)

data/lib/rubocop/cop/chef/correctness/invalid_value_for_platform_family_helper.rb

@@ -55,10 +55,10 @@ module RuboCop
                 if p_hash.key.array_type?
                   p_hash.key.values.each do |plat|
                     next unless INVALID_PLATFORM_FAMILIES.key?(plat.value)
-                    add_offense(plat, message: MSG, severity: :refactor)
+                    add_offense(plat, severity: :refactor)
                   end
                 elsif INVALID_PLATFORM_FAMILIES.key?(p_hash.key.value)
-                  add_offense(p_hash.key, message: MSG, severity: :refactor)
+                  add_offense(p_hash.key, severity: :refactor)
                 end
               end
             end

data/lib/rubocop/cop/chef/correctness/invalid_value_for_platform_helper.rb

@@ -54,10 +54,10 @@ module RuboCop
                 if p_hash.key.array_type?
                   p_hash.key.values.each do |plat|
                     next unless INVALID_PLATFORMS.key?(plat.value)
-                    add_offense(plat, message: MSG, severity: :refactor)
+                    add_offense(plat, severity: :refactor)
                   end
                 elsif INVALID_PLATFORMS.key?(p_hash.key.value)
-                  add_offense(p_hash.key, message: MSG, severity: :refactor)
+                  add_offense(p_hash.key, severity: :refactor)
                 end
               end
             end

data/lib/rubocop/cop/chef/correctness/invalid_version_metadata.rb

@@ -38,7 +38,7 @@ module RuboCop
           def on_send(node)
             version?(node) do |ver|
               next if /\A\d+\.\d+(\.\d+)?\z/.match?(ver.value) # entirely borrowed from Foodcritic.
-              add_offense(ver, message: MSG, severity: :refactor)
+              add_offense(ver, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/correctness/lazy_eval_node_attribute_defaults.rb

@@ -41,7 +41,7 @@ module RuboCop
 
           def on_send(node)
             non_lazy_node_attribute_default?(node) do |default|
-              add_offense(default, message: MSG, severity: :refactor) do |corrector|
+              add_offense(default, severity: :refactor) do |corrector|
                 corrector.replace(default, "lazy { #{default.source} }")
               end
             end

data/lib/rubocop/cop/chef/correctness/lazy_in_resource_guard.rb

@@ -55,7 +55,7 @@ module RuboCop
 
           def on_block(node)
             lazy_in_guard?(node) do |type, code|
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.replace(node, "#{type} { #{code.source} }")
               end
             end

data/lib/rubocop/cop/chef/correctness/macos_userdefaults_invalid_type.rb

@@ -54,7 +54,7 @@ module RuboCop
             match_property_in_resource?(:macos_userdefaults, 'type', node) do |type|
               type_val = method_arg_ast_to_string(type)
               return if VALID_VALUES.include?(type_val)
-              add_offense(type, message: MSG, severity: :refactor) do |corrector|
+              add_offense(type, severity: :refactor) do |corrector|
                 next unless INVALID_VALUE_MAP[type_val]
                 corrector.replace(type, "type '#{INVALID_VALUE_MAP[type_val]}'")
               end

data/lib/rubocop/cop/chef/correctness/metadata_malformed_version.rb

@@ -46,7 +46,7 @@ module RuboCop
 
           def on_send(node)
             depends_without_comma?(node) do |cb, ver|
-              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              add_offense(node, severity: :refactor) do |corrector|
                 corrector.replace(node, "depends '#{cb.value}', '#{ver.value}'")
               end
             end

data/lib/rubocop/cop/chef/correctness/metadata_missing_name.rb

@@ -38,7 +38,7 @@ module RuboCop
             # Using range similar to RuboCop::Cop::Naming::Filename (file_name.rb)
             return if cb_name?(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|
               path = processed_source.path
               cb_name = File.basename(File.dirname(path))
               corrector.insert_before(processed_source.ast, "name '#{cb_name}'\n")

data/lib/rubocop/cop/chef/correctness/metadata_missing_version.rb

@@ -39,7 +39,7 @@ module RuboCop
             # Using range similar to RuboCop::Cop::Naming::Filename (file_name.rb)
             return if cb_version?(processed_source.ast)
             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/correctness/node_normal.rb

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

data/lib/rubocop/cop/chef/correctness/node_normal_unless.rb

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

data/lib/rubocop/cop/chef/correctness/node_save.rb

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

data/lib/rubocop/cop/chef/correctness/notifies_action_not_symbol.rb

@@ -51,7 +51,7 @@ module RuboCop
             match_property_in_resource?(nil, %w(notifies subscribes), node) do |notifies_property|
               return unless notifies_property.node_parts[2].str_type?
 
-              add_offense(notifies_property, message: MSG, severity: :refactor) do |corrector|
+              add_offense(notifies_property, severity: :refactor) do |corrector|
                 corrector.replace(notifies_property.first_argument,
                   ":#{notifies_property.node_parts[2].value}")
               end

data/lib/rubocop/cop/chef/correctness/octal_mode_as_string.rb

@@ -39,7 +39,7 @@ module RuboCop
 
           def on_send(node)
             return unless node.arguments.first&.str_type? && node.arguments.first.value.match?(/^0o/)
-            add_offense(node, message: MSG, severity: :refactor)
+            add_offense(node, severity: :refactor)
           end
         end
       end

data/lib/rubocop/cop/chef/correctness/openssl_password_helpers.rb

@@ -36,7 +36,7 @@ module RuboCop
 
           def on_const(node)
             openssl_helper?(node) do
-              add_offense(node, message: MSG, severity: :warning)
+              add_offense(node, severity: :warning)
             end
           end
         end

data/lib/rubocop/cop/chef/correctness/powershell_delete_file.rb

@@ -44,7 +44,7 @@ module RuboCop
               property_data = method_arg_ast_to_string(code_property)
               return unless property_data && property_data.match?(/^remove-item/i) &&
                             !property_data.include?('*')
-              add_offense(node, message: MSG, severity: :refactor)
+              add_offense(node, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/correctness/powershell_file_exists.rb

@@ -40,7 +40,7 @@ module RuboCop
           def on_send(node)
             powershell_out_exists?(node) do |exists_string|
               return unless exists_string.match?(/^Test-Path/)
-              add_offense(node, message: MSG, severity: :refactor)
+              add_offense(node, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/correctness/property_without_type.rb

@@ -49,7 +49,7 @@ module RuboCop
             property_without_type?(node) do |hash_vals|
               return if hash_vals&.first&.keys&.include?(s(:sym, :kind_of))
 
-              add_offense(node, message: MSG, severity: :refactor)
+              add_offense(node, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/correctness/resource_sets_internal_properties.rb

@@ -41,7 +41,7 @@ module RuboCop
 
           def on_block(node)
             match_property_in_resource?(:service, 'running', node) do |prop|
-              add_offense(prop, message: MSG, severity: :refactor)
+              add_offense(prop, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/correctness/resource_sets_name_property.rb

@@ -40,7 +40,7 @@ module RuboCop
 
           def on_block(node)
             match_property_in_resource?(nil, 'name', node) do |name_node|
-              add_offense(name_node, message: MSG, severity: :refactor)
+              add_offense(name_node, severity: :refactor)
             end
           end
         end

data/lib/rubocop/cop/chef/correctness/resource_with_none_action.rb

@@ -43,7 +43,7 @@ module RuboCop
             match_property_in_resource?(nil, 'action', node) do |action_node|
               action_node.arguments.each do |action|
                 next unless action.source == ':none'
-                add_offense(action, message: MSG, severity: :refactor) do |corrector|
+                add_offense(action, severity: :refactor) do |corrector|
                   corrector.replace(action, ':nothing')
                 end
               end

data/lib/rubocop/cop/chef/correctness/scoped_file_exist.rb

@@ -40,7 +40,7 @@ module RuboCop
 
           def on_block(node)
             unscoped_file_exist?(node) do |m|
-              add_offense(m, message: MSG, severity: :refactor) do |corrector|
+              add_offense(m, severity: :refactor) do |corrector|
                 corrector.replace(m, '::File')
               end
             end

data/lib/rubocop/cop/chef/correctness/service_resource.rb

@@ -37,7 +37,7 @@ module RuboCop
           def on_send(node)
             execute_command?(node) do |command|
               if starts_service?(command)
-                add_offense(command, message: MSG, severity: :refactor)
+                add_offense(command, severity: :refactor)
               end
             end
           end

data/lib/rubocop/cop/chef/correctness/supports_must_be_float.rb

@@ -40,7 +40,7 @@ module RuboCop
           def on_send(node)
             supports_with_constraint?(node) do |ver|
               return if ver.source.include?('.')
-              add_offense(ver, message: MSG, severity: :refactor) do |corrector|
+              add_offense(ver, severity: :refactor) do |corrector|
                 corrector.replace(ver, ver.source.gsub(ver.value, (ver.value + '.0')))
               end
             end

data/lib/rubocop/cop/chef/correctness/tmp_path.rb

@@ -39,7 +39,7 @@ module RuboCop
           def on_send(node)
             remote_file?(node) do |command|
               return unless hardcoded_tmp?(command) && !file_cache_path?(command)
-              add_offense(command, message: MSG, severity: :refactor)
+              add_offense(command, severity: :refactor)
             end
           end
 

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

@@ -44,7 +44,7 @@ module RuboCop
             cb_name?(node) do
               dependencies(processed_source.ast).each do |dep|
                 next unless dep.arguments == node.arguments
-                add_offense(dep, message: MSG, severity: :refactor) do |corrector|
+                add_offense(dep, severity: :refactor) do |corrector|
                   corrector.remove(range_with_surrounding_space(range: dep.loc.expression, side: :left))
                 end
               end

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

@@ -40,7 +40,7 @@ module RuboCop
 
           def on_send(node)
             chef_handler_recipe?(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/deprecation/chef_handler_supports.rb

@@ -41,7 +41,7 @@ module RuboCop
 
           def on_block(node)
             match_property_in_resource?(:chef_handler, 'supports', node) do |prop_node|
-              add_offense(prop_node, message: MSG, severity: :warning) do |corrector|
+              add_offense(prop_node, severity: :warning) do |corrector|
                 # make sure to delete leading and trailing {}s that would create invalid ruby syntax
                 extracted_val = prop_node.arguments.first.source.gsub(/{|}/, '')
                 corrector.replace(prop_node, "type #{extracted_val}")

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

@@ -41,13 +41,13 @@ module RuboCop
 
           def on_send(node)
             require_rest?(node) do
-              add_offense(node, message: MSG, severity: :warning)
+              add_offense(node, severity: :warning)
             end
           end
 
           def on_const(node)
             rest_const?(node) do
-              add_offense(node, message: MSG, severity: :warning)
+              add_offense(node, severity: :warning)
             end
           end
         end

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

@@ -64,20 +64,20 @@ module RuboCop
 
           def on_send(node)
             rewind_gem_install?(node) do
-              add_offense(node, message: MSG, severity: :warning) do |corrector|
+              add_offense(node, severity: :warning) 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
             end
 
             require_rewind?(node) do
-              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
 
             rewind_resources?(node) do |string|
-              add_offense(node, message: MSG, severity: :warning) do |corrector|
+              add_offense(node, severity: :warning) do |corrector|
                 corrector.replace(node, node.source.gsub(string.to_s, MAPPING[string]))
               end
             end
@@ -86,7 +86,7 @@ module RuboCop
           def on_block(node)
             match_property_in_resource?(:chef_gem, 'package_name', node) do |pkg_name|
               next unless pkg_name.arguments&.first&.str_content == 'chef-rewind'
-              add_offense(node, message: MSG, severity: :warning) do |corrector|
+              add_offense(node, severity: :warning) do |corrector|
                 corrector.remove(node) if pkg_name.arguments&.first&.str_content == 'chef-rewind'
               end
             end

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

@@ -59,19 +59,19 @@ module RuboCop
 
           def on_send(node)
             include_shellout?(node) do
-              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
 
             require_shellout?(node) do
-              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
 
             shellout_new?(node) do
-              add_offense(node, message: MSG, severity: :warning) do |corrector|
+              add_offense(node, severity: :warning) do |corrector|
                 corrector.replace(node, node.source.gsub('Chef::ShellOut', 'Mixlib::ShellOut'))
               end
             end

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

@@ -46,7 +46,7 @@ module RuboCop
           RESTRICT_ON_SEND = [:vagrant_key?, :vagrant_domain?, :vagrant_user?, :require_chef_gem, :best_ip_for, :nexus?, :ios_xr?, :ruby_20?, :ruby_19?, :includes_recipe?, :wrlinux?, :dev_null, :nexentacore_platform?, :opensolaris_platform?, :nexentacore?, :opensolaris?].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/deprecation/chef_windows_platform_helper.rb

@@ -43,7 +43,7 @@ module RuboCop
 
           def on_send(node)
             chef_platform_windows?(node) do
-              add_offense(node, message: MSG, severity: :warning) do |corrector|
+              add_offense(node, severity: :warning) do |corrector|
                 corrector.replace(node, "platform?('windows')")
               end
             end

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

@@ -45,7 +45,7 @@ module RuboCop
             # We want to catch calls like ChefCLI::CLI.whatever or places where classes are defined in the ChefDK module
             return unless node.const_name == 'ChefDK' && (node.parent&.module_type? || node.parent&.const_type?)
 
-            add_offense(node, message: MSG, severity: :warning) do |corrector|
+            add_offense(node, severity: :warning) do |corrector|
               corrector.replace(node, node.source.gsub('ChefDK', 'ChefCLI'))
             end
           end

data/lib/rubocop/cop/chef/deprecation/cheffile.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: :warning)
+            add_offense(range, severity: :warning)
           end
         end
       end

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

@@ -37,7 +37,7 @@ module RuboCop
 
           def on_block(node)
             coverage_reporter?(node) do
-              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/chefspec_legacy_runner.rb

@@ -49,7 +49,7 @@ module RuboCop
 
           def on_const(node)
             chefspec_runner?(node) do
-              add_offense(node, message: MSG, severity: :warning) do |corrector|
+              add_offense(node, severity: :warning) do |corrector|
                 corrector.replace(node, 'ChefSpec::ServerRunner')
               end
             end