cookstyle 6.15.9 → 6.16.4

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

@@ -29,24 +29,22 @@ module RuboCop
         #   # good
         #   resource_name :foo
         #
-        class RespondToResourceName < Cop
+        class RespondToResourceName < Base
+          extend AutoCorrector
+
           MSG = 'respond_to?(:resource_name) in resources is no longer necessary in Chef Infra Client 12.5+'
 
           def on_if(node)
             if_respond_to_resource_name?(node) do
-              add_offense(node, location: :expression, message: MSG, severity: :refactor)
+              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+                corrector.replace(node.loc.expression, node.children[1].source)
+              end
             end
           end
 
           def_node_matcher :if_respond_to_resource_name?, <<~PATTERN
           (if (send nil? :respond_to? ( :sym :resource_name ) ) ... )
           PATTERN
-
-          def autocorrect(node)
-            lambda do |corrector|
-              corrector.replace(node.loc.expression, node.children[1].source)
-            end
-          end
         end
       end
     end

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

@@ -41,7 +41,7 @@ module RuboCop
           MSG = 'Chef Infra Client 14.0 and later includes :create, :delete, and :configure actions without the need for the sc cookbook dependency. See the windows_service documentation at https://docs.chef.io/resource_windows_service.html for additional details.'
 
           def on_send(node)
-            add_offense(node, location: :expression, message: MSG, severity: :refactor) if node.method_name == :sc_windows
+            add_offense(node, message: MSG, severity: :refactor) if node.method_name == :sc_windows
           end
         end
       end

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

@@ -29,7 +29,7 @@ module RuboCop
         #     path 'C:\expand_here'
         #   end
         #
-        class SevenZipArchiveResource < Cop
+        class SevenZipArchiveResource < Base
           extend TargetChefVersion
 
           minimum_target_chef_version '15.0'
@@ -37,7 +37,7 @@ module RuboCop
           MSG = 'Use the archive_file resource built into Chef Infra Client 15+ instead of the seven_zip_archive'
 
           def on_send(node)
-            add_offense(node, location: :expression, message: MSG, severity: :refactor) if node.method_name == :seven_zip_archive
+            add_offense(node, message: MSG, severity: :refactor) if node.method_name == :seven_zip_archive
           end
         end
       end

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

@@ -30,7 +30,8 @@ module RuboCop
         #   # good
         #   shell_out('foo')
         #
-        class ShellOutHelper < Cop
+        class ShellOutHelper < Base
+          extend AutoCorrector
           extend TargetChefVersion
 
           minimum_target_chef_version '12.11'
@@ -46,14 +47,8 @@ module RuboCop
           PATTERN
 
           def on_send(node)
-            mixlib_shellout_run_cmd?(node) do
-              add_offense(node, location: :expression, message: MSG, severity: :refactor)
-            end
-          end
-
-          def autocorrect(node)
             mixlib_shellout_run_cmd?(node) do |cmd|
-              lambda do |corrector|
+              add_offense(node, message: MSG, severity: :refactor) do |corrector|
                 corrector.replace(node.loc.expression, "shell_out(#{cmd.loc.expression.source})")
               end
             end

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

@@ -33,7 +33,7 @@ module RuboCop
         #    not_if 'choco source list | findstr artifactory'
         #  end
         #
-        class ShellOutToChocolatey < Cop
+        class ShellOutToChocolatey < Base
           include RuboCop::Chef::CookbookHelpers
 
           MSG = 'Use the Chocolatey resources built into Chef Infra Client instead of shelling out to the choco command'
@@ -41,16 +41,14 @@ module RuboCop
           def on_block(node)
             match_property_in_resource?(:powershell_script, 'code', node) do |code_property|
               property_data = method_arg_ast_to_string(code_property)
-              if property_data && property_data.match?(/^choco /i)
-                add_offense(node, location: :expression, message: MSG, severity: :refactor)
-              end
+              next unless property_data && property_data.match?(/^choco /i)
+              add_offense(node, message: MSG, severity: :refactor)
             end
 
             match_property_in_resource?(:execute, 'command', node) do |code_property|
               property_data = method_arg_ast_to_string(code_property)
-              if property_data && property_data.match?(/^choco /i)
-                add_offense(node, location: :expression, message: MSG, severity: :refactor)
-              end
+              next unless property_data && property_data.match?(/^choco /i)
+              add_offense(node, message: MSG, severity: :refactor)
             end
           end
         end

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

@@ -40,7 +40,8 @@ module RuboCop
         #      key 'C2518248EEA14886'
         #    end
         #
-        class SimplifyAptPpaSetup < Cop
+        class SimplifyAptPpaSetup < Base
+          extend AutoCorrector
           include RangeHelp
           include RuboCop::Chef::CookbookHelpers
 
@@ -49,15 +50,10 @@ module RuboCop
           def on_block(node)
             match_property_in_resource?(:apt_repository, 'uri', node) do |uri|
               if %r{http(s)*://ppa.launchpad.net/(.*)/ubuntu$}.match?(uri.arguments&.first&.str_content)
-                add_offense(uri, location: :expression, message: MSG, severity: :refactor)
-              end
-            end
-          end
-
-          def autocorrect(node)
-            if (replacement_val = %r{http(s)*://ppa.launchpad.net/(.*)/ubuntu}.match(node.source)[2])
-              lambda do |corrector|
-                corrector.replace(node.loc.expression, "uri 'ppa:#{replacement_val}'")
+                add_offense(uri, message: MSG, severity: :refactor) do |corrector|
+                  next unless (replacement_val = %r{http(s)*://ppa.launchpad.net/(.*)/ubuntu}.match(node.source)[2])
+                  corrector.replace(uri.loc.expression, "uri 'ppa:#{replacement_val}'")
+                end
               end
             end
           end

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

@@ -31,20 +31,18 @@ module RuboCop
         #     value '1048576'
         #   end
         #
-        class SysctlParamResource < Cop
+        class SysctlParamResource < Base
           include RuboCop::Chef::CookbookHelpers
           extend TargetChefVersion
+          extend AutoCorrector
 
           minimum_target_chef_version '14.0'
 
           MSG = 'The sysctl_param resource was renamed to sysctl when it was added to Chef Infra Client 14.0. The new resource name should be used.'
 
           def on_send(node)
-            add_offense(node, location: :expression, message: MSG, severity: :refactor) if node.method_name == :sysctl_param
-          end
-
-          def autocorrect(node)
-            lambda do |corrector|
+            return unless node.method_name == :sysctl_param
+            add_offense(node, message: MSG, severity: :refactor) do |corrector|
               corrector.replace(node.loc.expression, node.source.gsub(/^sysctl_param/, 'sysctl'))
             end
           end

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

@@ -27,7 +27,8 @@ module RuboCop
         #   # bad
         #   require 'mixlib/shellout'
         #
-        class UnnecessaryMixlibShelloutRequire < Cop
+        class UnnecessaryMixlibShelloutRequire < Base
+          extend AutoCorrector
           include RangeHelp
 
           MSG = 'Chef Infra Client 12.4+ includes mixlib/shellout automatically in resources and providers.'
@@ -38,13 +39,9 @@ module RuboCop
 
           def on_send(node)
             require_mixlibshellout?(node) do
-              add_offense(node, location: :expression, message: MSG, severity: :refactor)
-            end
-          end
-
-          def autocorrect(node)
-            lambda do |corrector|
-              corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
+              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+                corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
+              end
             end
           end
         end

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

@@ -33,7 +33,9 @@ module RuboCop
         #   # good
         #   package %w(bmon htop vim curl)
         #
-        class UseMultipackageInstalls < Cop
+        class UseMultipackageInstalls < Base
+          extend AutoCorrector
+
           MSG = 'Pass an array of packages to package resources instead of iterating over an array of packages when using multi-package capable package subsystem such as apt, yum, chocolatey, dnf, or zypper. Multi-package installs are faster and simplify logs.'
           MULTIPACKAGE_PLATS = %w(debian redhat suse amazon fedora scientific oracle rhel ubuntu centos redhat).freeze
 
@@ -81,22 +83,21 @@ module RuboCop
           end
 
           def on_when(node)
-            if platform_or_platform_family?(node.parent.condition) &&
-               package_array_install?(node.body) &&
-               multipackage_platforms?(node.conditions)
-              add_offense(node.body, location: :expression, message: MSG, severity: :refactor)
-            end
+            return unless platform_or_platform_family?(node.parent.condition) &&
+                          package_array_install?(node.body) &&
+                          multipackage_platforms?(node.conditions)
+            check_offense(node.body)
           end
 
           def on_if(node)
             platform_helper?(node) do |plats, blk, _pkgs|
-              add_offense(blk, location: :expression, message: MSG, severity: :refactor) if multipackage_platforms?(plats)
+              check_offense(blk) if multipackage_platforms?(plats)
             end
           end
 
-          def autocorrect(node)
-            package_array_install?(node) do |vals|
-              lambda do |corrector|
+          def check_offense(node)
+            add_offense(node, message: MSG, severity: :refactor) do |corrector|
+              package_array_install?(node) do |vals|
                 corrector.replace(node.loc.expression, "package #{vals.source}")
               end
             end

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

@@ -29,7 +29,9 @@ module RuboCop
         #   # good
         #   require_relative '../libraries/helpers'
         #
-        class UseRequireRelative < Cop
+        class UseRequireRelative < Base
+          extend AutoCorrector
+
           MSG = 'Instead of using require with a File.expand_path and __FILE__ use the simpler require_relative method.'
 
           def_node_matcher :require_with_expand_path?, <<-PATTERN
@@ -41,14 +43,9 @@ module RuboCop
           PATTERN
 
           def on_send(node)
-            require_with_expand_path?(node) do |_file, path|
-              add_offense(node, location: :expression, message: MSG, severity: :refactor) if path.source == '__FILE__'
-            end
-          end
-
-          def autocorrect(node)
-            lambda do |corrector|
-              require_with_expand_path?(node) do |file, _path|
+            require_with_expand_path?(node) do |file, path|
+              return unless path.source == '__FILE__'
+              add_offense(node, message: MSG, severity: :refactor) do |corrector|
                 corrected_value = file.value
                 corrected_value.slice!(%r{^../}) # take the first ../ off the path
                 corrector.replace(node.loc.expression, "require_relative '#{corrected_value}'")

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

@@ -28,8 +28,9 @@ module RuboCop
         #    true
         #   end
         #
-        class WhyRunSupportedTrue < Cop
+        class WhyRunSupportedTrue < Base
           extend TargetChefVersion
+          extend AutoCorrector
           include RangeHelp
 
           minimum_target_chef_version '13.0'
@@ -45,13 +46,9 @@ module RuboCop
 
           def on_def(node)
             whyrun_true?(node) do
-              add_offense(node, location: :expression, message: MSG, severity: :refactor)
-            end
-          end
-
-          def autocorrect(node)
-            lambda do |corrector|
-              corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
+              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+                corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
+              end
             end
           end
         end

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

@@ -27,8 +27,9 @@ module RuboCop
         #   include_recipe 'windows::default'
         #   include_recipe 'windows'
         #
-        class IncludingWindowsDefaultRecipe < Cop
+        class IncludingWindowsDefaultRecipe < Base
           include RangeHelp
+          extend AutoCorrector
 
           MSG = 'Do not include the Windows default recipe, which only installs win32 gems already included in Chef Infra Client'
 
@@ -38,13 +39,9 @@ module RuboCop
 
           def on_send(node)
             windows_recipe_usage?(node) do
-              add_offense(node, location: :expression, message: MSG, severity: :refactor)
-            end
-          end
-
-          def autocorrect(node)
-            lambda do |corrector|
-              corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
+              add_offense(node, message: MSG, severity: :refactor) do |corrector|
+                corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
+              end
             end
           end
         end

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

@@ -47,22 +47,21 @@ module RuboCop
 
           # non block execute resources
           def on_send(node)
-            return unless node.method_name == :registry_key
+            return unless node.method_name == :registry_key &&
+                          node&.arguments.first&.source.match?(/(HKLM|HKEY_LOCAL_MACHINE)\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Policies\\System/i) &&
+                          node.parent&.method_name != :describe
 
             # use source instead of .value in case there's string interpolation which adds a complex dstr type
             # with a nested string and a begin. Source allows us to avoid a lot of defensive programming here
-            if node&.arguments.first&.source.match?(/(HKLM|HKEY_LOCAL_MACHINE)\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Policies\\System/i) && node.parent&.method_name != :describe
-              add_offense(node, location: :expression, message: MSG, severity: :refactor)
-            end
+            add_offense(node, location: :expression, message: MSG, severity: :refactor)
           end
 
           # block execute resources
           def on_block(node)
             match_property_in_resource?(:registry_key, 'key', node) do |key_prop|
               property_data = method_arg_ast_to_string(key_prop)
-              if property_data && property_data.match?(/(HKLM|HKEY_LOCAL_MACHINE)\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Policies\\System/i)
-                add_offense(node, location: :expression, message: MSG, severity: :refactor)
-              end
+              return unless property_data && property_data.match?(/(HKLM|HKEY_LOCAL_MACHINE)\\SOFTWARE\\Microsoft\\Windows\\CurrentVersion\\Policies\\System/i)
+              add_offense(node, location: :expression, message: MSG, severity: :refactor)
             end
           end
         end

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

@@ -28,7 +28,7 @@ module RuboCop
         #     source 'C:\\Temp\\file.zip'
         #   end
         #
-        class WindowsZipfileUsage < Cop
+        class WindowsZipfileUsage < Base
           extend TargetChefVersion
 
           minimum_target_chef_version '15.0'
@@ -36,7 +36,7 @@ module RuboCop
           MSG = 'Use the archive_file resource built into Chef Infra Client 15+ instead of the windows_zipfile from the Windows cookbook'
 
           def on_send(node)
-            add_offense(node, location: :expression, message: MSG, severity: :refactor) if node.method_name == :windows_zipfile
+            add_offense(node, message: MSG, severity: :refactor) if node.method_name == :windows_zipfile
           end
         end
       end

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

@@ -29,7 +29,7 @@ module RuboCop
         #     path 'C:\expand_here'
         #   end
         #
-        class ZipfileResource < Cop
+        class ZipfileResource < Base
           include RuboCop::Chef::CookbookHelpers
           extend TargetChefVersion
 
@@ -43,13 +43,13 @@ module RuboCop
 
           def on_send(node)
             depends_zipfile?(node) do
-              add_offense(node, location: :expression, message: MSG, severity: :refactor)
+              add_offense(node, message: MSG, severity: :refactor)
             end
           end
 
           def on_block(node)
             match_resource_type?(:zipfile, node) do
-              add_offense(node, location: :expression, message: MSG, severity: :refactor)
+              add_offense(node, message: MSG, severity: :refactor)
             end
           end
         end

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

@@ -39,19 +39,17 @@ module RuboCop
         #     priority '100'
         #   end
         #
-        class UsesZypperRepo < Cop
+        class UsesZypperRepo < Base
           extend TargetChefVersion
+          extend AutoCorrector
 
           minimum_target_chef_version '13.3'
 
           MSG = 'The zypper_repo resource was renamed zypper_repository when it was added to Chef Infra Client 13.3.'
 
           def on_send(node)
-            add_offense(node, location: :expression, message: MSG, severity: :refactor) if node.method_name == :zypper_repo
-          end
-
-          def autocorrect(node)
-            lambda do |corrector|
+            return unless node.method_name == :zypper_repo
+            add_offense(node, message: MSG, severity: :refactor) do |corrector|
               corrector.replace(node.loc.expression, node.source.gsub(/^zypper_repo/, 'zypper_repository'))
             end
           end

data/lib/rubocop/cop/chef/redundant/apt_repository_distribution_default.rb

@@ -36,9 +36,10 @@ module RuboCop
         #     deb_src false
         #   end
         #
-        class AptRepositoryDistributionDefault < Cop
+        class AptRepositoryDistributionDefault < Base
           include RuboCop::Chef::CookbookHelpers
           include RangeHelp
+          extend AutoCorrector
 
           MSG = "There is no need to pass `distribution node['lsb']['codename']` to an apt_repository resource as this is done automatically by the apt_repository resource."
 
@@ -49,16 +50,12 @@ module RuboCop
           def on_block(node)
             match_property_in_resource?(:apt_repository, 'distribution', node) do |dist|
               default_dist?(dist) do
-                add_offense(dist, location: :expression, message: MSG, severity: :refactor)
+                add_offense(dist, message: MSG, severity: :refactor) do |corrector|
+                  corrector.remove(range_with_surrounding_space(range: dist.loc.expression, side: :left))
+                end
               end
             end
           end
-
-          def autocorrect(node)
-            lambda do |corrector|
-              corrector.remove(range_with_surrounding_space(range: node.loc.expression, side: :left))
-            end
-          end
         end
       end
     end