cookstyle 8.6.4 → 8.7.6

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

@@ -1,94 +1,94 @@
-# frozen_string_literal: true
-#
-# Copyright:: 2020, Chef Software, Inc.
-# Author:: Tim Smith (<tsmith84@gmail.com>)
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-module RuboCop
-  module Cop
-    module Chef
-      module Modernize
-        # There is no need to check if the chef_gem resource supports compile_time as Chef Infra Client 12.1 and later support the compile_time property.
-        #
-        # @example
-        #
-        #   # bad
-        #   chef_gem 'ultradns-sdk' do
-        #     compile_time true if Chef::Resource::ChefGem.method_defined?(:compile_time)
-        #     action :nothing
-        #   end
-        #
-        #   chef_gem 'ultradns-sdk' do
-        #     compile_time true if Chef::Resource::ChefGem.instance_methods(false).include?(:compile_time)
-        #     action :nothing
-        #   end
-        #
-        #   chef_gem 'ultradns-sdk' do
-        #     compile_time true if respond_to?(:compile_time)
-        #     action :nothing
-        #   end
-        #
-        #   # good
-        #   chef_gem 'ultradns-sdk' do
-        #     compile_time true
-        #     action :nothing
-        #   end
-        #
-        class RespondToCompileTime < Base
-          include RuboCop::Chef::CookbookHelpers
-          extend TargetChefVersion
-          extend AutoCorrector
-
-          minimum_target_chef_version '12.1'
-
-          MSG = 'There is no need to check if the chef_gem resource supports compile_time as Chef Infra Client 12.1 and later support the compile_time property.'
-
-          def_node_matcher :compile_time_method_defined?, <<-PATTERN
-          (if
-            {
-              (send
-              (const
-                (const
-                  (const nil? :Chef) :Resource) :ChefGem) :method_defined?
-              (sym :compile_time))
-
-              (send
-                (send
-                  (const
-                    (const
-                      (const nil? :Chef) :Resource) :ChefGem) :instance_methods
-                  (false)) :include?
-                (sym :compile_time))
-
-              (send nil? :respond_to?
-                (sym :compile_time))
-            }
-            (send nil? :compile_time
-              $(_)) nil?)
-          PATTERN
-
-          def on_block(node)
-            match_property_in_resource?(:chef_gem, 'compile_time', node) do |compile_time_property|
-              compile_time_method_defined?(compile_time_property.parent) do |val|
-                add_offense(compile_time_property.parent, severity: :refactor) do |corrector|
-                  corrector.replace(compile_time_property.parent, "compile_time #{val.source}")
-                end
-              end
-            end
-          end
-        end
-      end
-    end
-  end
-end
+# frozen_string_literal: true
+#
+# Copyright:: Copyright (c) 2016-2025 Progress Software Corporation and/or its subsidiaries or affiliates. All Rights Reserved.
+# Author:: Tim Smith (<tsmith84@gmail.com>)
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+module RuboCop
+  module Cop
+    module Chef
+      module Modernize
+        # There is no need to check if the chef_gem resource supports compile_time as Chef Infra Client 12.1 and later support the compile_time property.
+        #
+        # @example
+        #
+        #   # bad
+        #   chef_gem 'ultradns-sdk' do
+        #     compile_time true if Chef::Resource::ChefGem.method_defined?(:compile_time)
+        #     action :nothing
+        #   end
+        #
+        #   chef_gem 'ultradns-sdk' do
+        #     compile_time true if Chef::Resource::ChefGem.instance_methods(false).include?(:compile_time)
+        #     action :nothing
+        #   end
+        #
+        #   chef_gem 'ultradns-sdk' do
+        #     compile_time true if respond_to?(:compile_time)
+        #     action :nothing
+        #   end
+        #
+        #   # good
+        #   chef_gem 'ultradns-sdk' do
+        #     compile_time true
+        #     action :nothing
+        #   end
+        #
+        class RespondToCompileTime < Base
+          include RuboCop::Chef::CookbookHelpers
+          extend TargetChefVersion
+          extend AutoCorrector
+
+          minimum_target_chef_version '12.1'
+
+          MSG = 'There is no need to check if the chef_gem resource supports compile_time as Chef Infra Client 12.1 and later support the compile_time property.'
+
+          def_node_matcher :compile_time_method_defined?, <<-PATTERN
+          (if
+            {
+              (send
+              (const
+                (const
+                  (const nil? :Chef) :Resource) :ChefGem) :method_defined?
+              (sym :compile_time))
+
+              (send
+                (send
+                  (const
+                    (const
+                      (const nil? :Chef) :Resource) :ChefGem) :instance_methods
+                  (false)) :include?
+                (sym :compile_time))
+
+              (send nil? :respond_to?
+                (sym :compile_time))
+            }
+            (send nil? :compile_time
+              $(_)) nil?)
+          PATTERN
+
+          def on_block(node)
+            match_property_in_resource?(:chef_gem, 'compile_time', node) do |compile_time_property|
+              compile_time_method_defined?(compile_time_property.parent) do |val|
+                add_offense(compile_time_property.parent, severity: :refactor) do |corrector|
+                  corrector.replace(compile_time_property.parent, "compile_time #{val.source}")
+                end
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -1,77 +1,77 @@
-# frozen_string_literal: true
-#
-# Copyright:: 2019, Chef Software, Inc.
-# Author:: Tim Smith (<tsmith84@gmail.com>)
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-module RuboCop
-  module Cop
-    module Chef
-      module Modernize
-        # It is no longer necessary to use respond_to?(:foo) or defined?(foo) in metadata. This was used to support new metadata methods in Chef 11 and early versions of Chef 12.
-        #
-        # @example
-        #
-        #   # bad
-        #   chef_version '>= 13' if respond_to?(:chef_version)
-        #   chef_version '>= 13' if defined?(chef_version)
-        #   chef_version '>= 13' unless defined?(Ridley::Chef::Cookbook::Metadata)
-        #   if defined(chef_version)
-        #     chef_version '>= 13'
-        #   end
-        #
-        #   # good
-        #   chef_version '>= 13'
-        #
-        class RespondToInMetadata < Base
-          extend AutoCorrector
-          extend TargetChefVersion
-
-          minimum_target_chef_version '12.15'
-
-          MSG = 'It is no longer necessary to use respond_to? or defined? in metadata.rb in Chef Infra Client 12.15 and later'
-
-          def on_if(node)
-            if_respond_to?(node) do
-              add_offense(node, severity: :refactor) do |corrector|
-                # When the if statement is if modifier like `foo if respond_to?(:foo)` then
-                # node.if_branch is the actual method call we want to extract.
-                # If a series of metadata methods are wrapped in an if statement then the content we want
-                # is a block under the if statement and node.parent.if_branch can get us that block
-                node = node.parent unless node.if_type?
-
-                corrector.replace(node, node.if_branch.source)
-              end
-            end
-          end
-
-          def on_defined?(node)
-            # When the if statement is if modifier like `foo if respond_to?(:foo)` then
-            # node.if_branch is the actual method call we want to extract.
-            # If a series of metadata methods are wrapped in an if statement then the content we want
-            # is a block under the if statement and node.parent.if_branch can get us that block
-            node = node.parent if node.parent.conditional? # we want the whole conditional statement
-            add_offense(node, severity: :refactor) do |corrector|
-              corrector.replace(node, node.if_branch.source)
-            end
-          end
-
-          def_node_matcher :if_respond_to?, <<~PATTERN
-            (if (send nil? :respond_to? _ ) ... )
-          PATTERN
-        end
-      end
-    end
-  end
-end
+# frozen_string_literal: true
+#
+# Copyright:: Copyright (c) 2016-2025 Progress Software Corporation and/or its subsidiaries or affiliates. All Rights Reserved.
+# Author:: Tim Smith (<tsmith84@gmail.com>)
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+module RuboCop
+  module Cop
+    module Chef
+      module Modernize
+        # It is no longer necessary to use respond_to?(:foo) or defined?(foo) in metadata. This was used to support new metadata methods in Chef 11 and early versions of Chef 12.
+        #
+        # @example
+        #
+        #   # bad
+        #   chef_version '>= 13' if respond_to?(:chef_version)
+        #   chef_version '>= 13' if defined?(chef_version)
+        #   chef_version '>= 13' unless defined?(Ridley::Chef::Cookbook::Metadata)
+        #   if defined(chef_version)
+        #     chef_version '>= 13'
+        #   end
+        #
+        #   # good
+        #   chef_version '>= 13'
+        #
+        class RespondToInMetadata < Base
+          extend AutoCorrector
+          extend TargetChefVersion
+
+          minimum_target_chef_version '12.15'
+
+          MSG = 'It is no longer necessary to use respond_to? or defined? in metadata.rb in Chef Infra Client 12.15 and later'
+
+          def on_if(node)
+            if_respond_to?(node) do
+              add_offense(node, severity: :refactor) do |corrector|
+                # When the if statement is if modifier like `foo if respond_to?(:foo)` then
+                # node.if_branch is the actual method call we want to extract.
+                # If a series of metadata methods are wrapped in an if statement then the content we want
+                # is a block under the if statement and node.parent.if_branch can get us that block
+                node = node.parent unless node.if_type?
+
+                corrector.replace(node, node.if_branch.source)
+              end
+            end
+          end
+
+          def on_defined?(node)
+            # When the if statement is if modifier like `foo if respond_to?(:foo)` then
+            # node.if_branch is the actual method call we want to extract.
+            # If a series of metadata methods are wrapped in an if statement then the content we want
+            # is a block under the if statement and node.parent.if_branch can get us that block
+            node = node.parent if node.parent.conditional? # we want the whole conditional statement
+            add_offense(node, severity: :refactor) do |corrector|
+              corrector.replace(node, node.if_branch.source)
+            end
+          end
+
+          def_node_matcher :if_respond_to?, <<~PATTERN
+            (if (send nil? :respond_to? _ ) ... )
+          PATTERN
+        end
+      end
+    end
+  end
+end

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

@@ -1,63 +1,63 @@
-# frozen_string_literal: true
-#
-# Copyright:: 2019-2020, Chef Software, Inc.
-# Author:: Tim Smith (<tsmith84@gmail.com>)
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-module RuboCop
-  module Cop
-    module Chef
-      module Modernize
-        # In Chef Infra Client 12+ it is no longer necessary to gate the use of the provides methods in resources with `if respond_to?(:provides)` or `if defined? provides`.
-        #
-        # @example
-        #
-        #   # bad
-        #   provides :foo if respond_to?(:provides)
-        #
-        #   provides :foo if defined? provides
-        #
-        #   # good
-        #   provides :foo
-        #
-        class RespondToProvides < Base
-          extend AutoCorrector
-
-          MSG = 'Using `respond_to?(:provides)` or `if defined? provides` in resources is no longer necessary in Chef Infra Client 12+.'
-
-          def on_if(node)
-            if_respond_to_provides?(node) do
-              add_offense(node, severity: :refactor) do |corrector|
-                corrector.replace(node, node.children[1].source)
-              end
-            end
-          end
-
-          def_node_matcher :if_respond_to_provides?, <<~PATTERN
-            (if
-              {
-              (send nil? :respond_to?
-                (sym :provides))
-
-              (:defined?
-                (send nil? :provides))
-              }
-              (send nil? :provides
-                (sym _)) ... )
-          PATTERN
-        end
-      end
-    end
-  end
-end
+# frozen_string_literal: true
+#
+# Copyright:: Copyright (c) 2016-2025 Progress Software Corporation and/or its subsidiaries or affiliates. All Rights Reserved.
+# Author:: Tim Smith (<tsmith84@gmail.com>)
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+module RuboCop
+  module Cop
+    module Chef
+      module Modernize
+        # In Chef Infra Client 12+ it is no longer necessary to gate the use of the provides methods in resources with `if respond_to?(:provides)` or `if defined? provides`.
+        #
+        # @example
+        #
+        #   # bad
+        #   provides :foo if respond_to?(:provides)
+        #
+        #   provides :foo if defined? provides
+        #
+        #   # good
+        #   provides :foo
+        #
+        class RespondToProvides < Base
+          extend AutoCorrector
+
+          MSG = 'Using `respond_to?(:provides)` or `if defined? provides` in resources is no longer necessary in Chef Infra Client 12+.'
+
+          def on_if(node)
+            if_respond_to_provides?(node) do
+              add_offense(node, severity: :refactor) do |corrector|
+                corrector.replace(node, node.children[1].source)
+              end
+            end
+          end
+
+          def_node_matcher :if_respond_to_provides?, <<~PATTERN
+            (if
+              {
+              (send nil? :respond_to?
+                (sym :provides))
+
+              (:defined?
+                (send nil? :provides))
+              }
+              (send nil? :provides
+                (sym _)) ... )
+          PATTERN
+        end
+      end
+    end
+  end
+end

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

@@ -1,52 +1,52 @@
-# frozen_string_literal: true
-#
-# Copyright:: 2019, Chef Software, Inc.
-# Author:: Tim Smith (<tsmith84@gmail.com>)
-#
-# Licensed under the Apache License, Version 2.0 (the "License");
-# you may not use this file except in compliance with the License.
-# You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software
-# distributed under the License is distributed on an "AS IS" BASIS,
-# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-# See the License for the specific language governing permissions and
-# limitations under the License.
-#
-module RuboCop
-  module Cop
-    module Chef
-      module Modernize
-        # Chef Infra Client 12.5 introduced the resource_name method for resources. Many cookbooks used respond_to?(:resource_name) to provide backwards compatibility with older chef-client releases. This backwards compatibility is no longer necessary.
-        #
-        # @example
-        #
-        #   # bad
-        #   resource_name :foo if respond_to?(:resource_name)
-        #
-        #   # good
-        #   resource_name :foo
-        #
-        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, severity: :refactor) do |corrector|
-                corrector.replace(node, 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
-        end
-      end
-    end
-  end
-end
+# frozen_string_literal: true
+#
+# Copyright:: Copyright (c) 2016-2025 Progress Software Corporation and/or its subsidiaries or affiliates. All Rights Reserved.
+# Author:: Tim Smith (<tsmith84@gmail.com>)
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+module RuboCop
+  module Cop
+    module Chef
+      module Modernize
+        # Chef Infra Client 12.5 introduced the resource_name method for resources. Many cookbooks used respond_to?(:resource_name) to provide backwards compatibility with older chef-client releases. This backwards compatibility is no longer necessary.
+        #
+        # @example
+        #
+        #   # bad
+        #   resource_name :foo if respond_to?(:resource_name)
+        #
+        #   # good
+        #   resource_name :foo
+        #
+        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, severity: :refactor) do |corrector|
+                corrector.replace(node, 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
+        end
+      end
+    end
+  end
+end