cookstyle 8.6.4 → 8.7.6

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

@@ -1,65 +1,65 @@
-# frozen_string_literal: true
-#
-# Copyright:: Copyright 2019, Chef Software Inc.
-#
-# 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 Deprecations
-        # The node.deep_fetch method has been removed from Chef-Sugar, and must be replaced by the node.read API.
-        #
-        # @example
-        #
-        #   # bad
-        #   node.deep_fetch("foo")
-        #
-        #   # good
-        #   node.read("foo")
-        #
-        #   # bad
-        #   node.deep_fetch!("foo")
-        #
-        #   # good
-        #   node.read!("foo")
-        #
-        class NodeDeepFetch < Base
-          extend RuboCop::Cop::AutoCorrector
-
-          RESTRICT_ON_SEND = [:deep_fetch, :deep_fetch!].freeze
-
-          def_node_matcher :node_deep_fetch?, <<-PATTERN
-            (send (send _ :node) ${:deep_fetch :deep_fetch!} _)
-          PATTERN
-
-          def on_send(node)
-            node_deep_fetch?(node) do
-              add_offense(node.loc.selector, message: "Do not use node.#{node.method_name}. Replace with node.#{fix_name(node.method_name)} to keep identical behavior.", severity: :warning) do |corrector|
-                corrector.replace(node.loc.selector, fix_name(node.method_name))
-              end
-            end
-          end
-
-          private
-
-          def fix_name(name)
-            return 'read!' if name == :deep_fetch!
-            return 'read' if name == :deep_fetch
-            name.to_s
-          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.
+#
+# 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 Deprecations
+        # The node.deep_fetch method has been removed from Chef-Sugar, and must be replaced by the node.read API.
+        #
+        # @example
+        #
+        #   # bad
+        #   node.deep_fetch("foo")
+        #
+        #   # good
+        #   node.read("foo")
+        #
+        #   # bad
+        #   node.deep_fetch!("foo")
+        #
+        #   # good
+        #   node.read!("foo")
+        #
+        class NodeDeepFetch < Base
+          extend RuboCop::Cop::AutoCorrector
+
+          RESTRICT_ON_SEND = [:deep_fetch, :deep_fetch!].freeze
+
+          def_node_matcher :node_deep_fetch?, <<-PATTERN
+            (send (send _ :node) ${:deep_fetch :deep_fetch!} _)
+          PATTERN
+
+          def on_send(node)
+            node_deep_fetch?(node) do
+              add_offense(node.loc.selector, message: "Do not use node.#{node.method_name}. Replace with node.#{fix_name(node.method_name)} to keep identical behavior.", severity: :warning) do |corrector|
+                corrector.replace(node.loc.selector, fix_name(node.method_name))
+              end
+            end
+          end
+
+          private
+
+          def fix_name(name)
+            return 'read!' if name == :deep_fetch!
+            return 'read' if name == :deep_fetch
+            name.to_s
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -1,80 +1,80 @@
-# 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 Deprecations
-        # Use node attributes to access data provided by Ohai instead of using node methods to access that data.
-        #
-        # @example
-        #
-        #   # bad
-        #   node.fqdn
-        #   node.platform
-        #   node.platform_family
-        #   node.platform_version
-        #   node.hostname
-        #
-        #   # good
-        #   node['fqdn']
-        #   node['platform']
-        #   node['platform_family']
-        #   node['platform_version']
-        #   node['hostname']
-        #
-        class NodeMethodsInsteadofAttributes < Base
-          extend AutoCorrector
-
-          MSG = 'Use node attributes to access Ohai data instead of node methods, which were deprecated in Chef Infra Client 13.'
-          RESTRICT_ON_SEND = %i(
-            current_user
-            domain
-            fqdn
-            hostname
-            ip6address
-            ipaddress
-            macaddress
-            machinename
-            ohai_time
-            os
-            os_version
-            platform
-            platform_build
-            platform_family
-            platform_version
-            root_group
-            shard_seed
-            uptime
-            uptime_seconds).freeze
-
-          def_node_matcher :node_ohai_methods?, <<-PATTERN
-            (send (send nil? :node) _)
-          PATTERN
-
-          def on_send(node)
-            node_ohai_methods?(node) do
-              add_offense(node.loc.selector, severity: :warning) do |corrector|
-                corrector.replace(node, "node['#{node.method_name}']")
-              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 Deprecations
+        # Use node attributes to access data provided by Ohai instead of using node methods to access that data.
+        #
+        # @example
+        #
+        #   # bad
+        #   node.fqdn
+        #   node.platform
+        #   node.platform_family
+        #   node.platform_version
+        #   node.hostname
+        #
+        #   # good
+        #   node['fqdn']
+        #   node['platform']
+        #   node['platform_family']
+        #   node['platform_version']
+        #   node['hostname']
+        #
+        class NodeMethodsInsteadofAttributes < Base
+          extend AutoCorrector
+
+          MSG = 'Use node attributes to access Ohai data instead of node methods, which were deprecated in Chef Infra Client 13.'
+          RESTRICT_ON_SEND = %i(
+            current_user
+            domain
+            fqdn
+            hostname
+            ip6address
+            ipaddress
+            macaddress
+            machinename
+            ohai_time
+            os
+            os_version
+            platform
+            platform_build
+            platform_family
+            platform_version
+            root_group
+            shard_seed
+            uptime
+            uptime_seconds).freeze
+
+          def_node_matcher :node_ohai_methods?, <<-PATTERN
+            (send (send nil? :node) _)
+          PATTERN
+
+          def on_send(node)
+            node_ohai_methods?(node) do
+              add_offense(node.loc.selector, severity: :warning) do |corrector|
+                corrector.replace(node, "node['#{node.method_name}']")
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -1,54 +1,54 @@
-# frozen_string_literal: true
-#
-# Copyright:: Copyright 2019, Chef Software Inc.
-#
-# 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 Deprecations
-        # The `node.set` method has been removed in Chef Infra Client 13 and usage must be replaced with `node.normal`.
-        #
-        # This cop will autocorrect code to use node.normal, which is functionally identical to node.set, but we also discourage the use of that method as normal level attributes persist on the node even if the code setting the attribute is later removed.
-        #
-        # @example
-        #
-        #   # bad
-        #   node.set['foo'] = true
-        #
-        #   # good
-        #   node.normal['foo'] = true
-        #
-        class NodeSet < Base
-          extend AutoCorrector
-
-          MSG = 'Do not use node.set. Replace with node.normal to keep identical behavior.'
-          RESTRICT_ON_SEND = [:set].freeze
-
-          def_node_matcher :node_set?, <<-PATTERN
-            (send (send _ :node) $:set)
-          PATTERN
-
-          def on_send(node)
-            node_set?(node) do
-              add_offense(node.loc.selector, severity: :warning) do |corrector|
-                corrector.replace(node.loc.selector, 'normal')
-              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.
+#
+# 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 Deprecations
+        # The `node.set` method has been removed in Chef Infra Client 13 and usage must be replaced with `node.normal`.
+        #
+        # This cop will autocorrect code to use node.normal, which is functionally identical to node.set, but we also discourage the use of that method as normal level attributes persist on the node even if the code setting the attribute is later removed.
+        #
+        # @example
+        #
+        #   # bad
+        #   node.set['foo'] = true
+        #
+        #   # good
+        #   node.normal['foo'] = true
+        #
+        class NodeSet < Base
+          extend AutoCorrector
+
+          MSG = 'Do not use node.set. Replace with node.normal to keep identical behavior.'
+          RESTRICT_ON_SEND = [:set].freeze
+
+          def_node_matcher :node_set?, <<-PATTERN
+            (send (send _ :node) $:set)
+          PATTERN
+
+          def on_send(node)
+            node_set?(node) do
+              add_offense(node.loc.selector, severity: :warning) do |corrector|
+                corrector.replace(node.loc.selector, 'normal')
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -1,54 +1,54 @@
-# frozen_string_literal: true
-#
-# Copyright:: Copyright 2019, Chef Software Inc.
-#
-# 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 Deprecations
-        # The node.set_unless method has been removed in Chef Infra Client 13 and usage must be replaced with node.normal_unless.
-        #
-        # This cop will autocorrect code to use node.normal_unless, which is functionally identical to node.set_unless, but we also discourage the use of that method as normal level attributes persist on the node even if the code setting the attribute is later removed.
-        #
-        # @example
-        #
-        #   # bad
-        #   node.set_unless['foo'] = true
-        #
-        #   # good
-        #   node.normal_unless['foo'] = true
-        #
-        class NodeSetUnless < Base
-          extend AutoCorrector
-
-          MSG = 'Do not use node.set_unless. Replace with node.normal_unless to keep identical behavior.'
-          RESTRICT_ON_SEND = [:set_unless].freeze
-
-          def_node_matcher :node_set_unless?, <<-PATTERN
-            (send (send _ :node) $:set_unless)
-          PATTERN
-
-          def on_send(node)
-            node_set_unless?(node) do
-              add_offense(node.loc.selector, severity: :warning) do |corrector|
-                corrector.replace(node.loc.selector, 'normal_unless')
-              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.
+#
+# 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 Deprecations
+        # The node.set_unless method has been removed in Chef Infra Client 13 and usage must be replaced with node.normal_unless.
+        #
+        # This cop will autocorrect code to use node.normal_unless, which is functionally identical to node.set_unless, but we also discourage the use of that method as normal level attributes persist on the node even if the code setting the attribute is later removed.
+        #
+        # @example
+        #
+        #   # bad
+        #   node.set_unless['foo'] = true
+        #
+        #   # good
+        #   node.normal_unless['foo'] = true
+        #
+        class NodeSetUnless < Base
+          extend AutoCorrector
+
+          MSG = 'Do not use node.set_unless. Replace with node.normal_unless to keep identical behavior.'
+          RESTRICT_ON_SEND = [:set_unless].freeze
+
+          def_node_matcher :node_set_unless?, <<-PATTERN
+            (send (send _ :node) $:set_unless)
+          PATTERN
+
+          def on_send(node)
+            node_set_unless?(node) do
+              add_offense(node.loc.selector, severity: :warning) do |corrector|
+                corrector.replace(node.loc.selector, 'normal_unless')
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -1,70 +1,70 @@
-# frozen_string_literal: true
-#
-# Copyright:: Copyright 2019, Chef Software Inc.
-#
-# 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 Deprecations
-        # When setting a node attribute in Chef Infra Client 11 and later you must specify the precedence level.
-        #
-        # @example
-        #
-        #   # bad
-        #   node['foo']['bar'] = 1
-        #   node['foo']['bar'] << 1
-        #   node['foo']['bar'] += 1
-        #   node['foo']['bar'] -= 1
-        #
-        #   # good
-        #   node.default['foo']['bar'] = 1
-        #   node.default['foo']['bar'] << 1
-        #   node.default['foo']['bar'] += 1
-        #   node.default['foo']['bar'] -= 1
-        #
-        class NodeSetWithoutLevel < Base
-          MSG = 'When setting a node attribute in Chef Infra Client 11 and later you must specify the precedence level.'
-          RESTRICT_ON_SEND = [:[]=, :<<].freeze
-
-          def on_op_asgn(node)
-            # make sure it was a += or -=
-            if %i(- +).include?(node.node_parts[1])
-              add_offense_for_bare_assignment(node.children&.first)
-            end
-          end
-
-          def on_send(node)
-            # make sure the method being send is []= and then make sure the receiver is a send
-            if %i([]= <<).include?(node.method_name) && node.receiver.send_type?
-              add_offense_for_bare_assignment(node)
-            end
-          end
-
-          private
-
-          def add_offense_for_bare_assignment(sub_node)
-            if sub_node.receiver == s(:send, nil, :node) # node['foo'] scenario
-              add_offense(sub_node.receiver.loc.selector, severity: :warning)
-            elsif sub_node.receiver &&
-                  sub_node.receiver&.node_parts.first == s(:send, nil, :node) &&
-                  sub_node.receiver&.node_parts[1] == :[] # node['foo']['bar'] scenario
-              add_offense(sub_node.receiver.node_parts.first.loc.selector, severity: :warning)
-            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.
+#
+# 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 Deprecations
+        # When setting a node attribute in Chef Infra Client 11 and later you must specify the precedence level.
+        #
+        # @example
+        #
+        #   # bad
+        #   node['foo']['bar'] = 1
+        #   node['foo']['bar'] << 1
+        #   node['foo']['bar'] += 1
+        #   node['foo']['bar'] -= 1
+        #
+        #   # good
+        #   node.default['foo']['bar'] = 1
+        #   node.default['foo']['bar'] << 1
+        #   node.default['foo']['bar'] += 1
+        #   node.default['foo']['bar'] -= 1
+        #
+        class NodeSetWithoutLevel < Base
+          MSG = 'When setting a node attribute in Chef Infra Client 11 and later you must specify the precedence level.'
+          RESTRICT_ON_SEND = [:[]=, :<<].freeze
+
+          def on_op_asgn(node)
+            # make sure it was a += or -=
+            if %i(- +).include?(node.node_parts[1])
+              add_offense_for_bare_assignment(node.children&.first)
+            end
+          end
+
+          def on_send(node)
+            # make sure the method being send is []= and then make sure the receiver is a send
+            if %i([]= <<).include?(node.method_name) && node.receiver.send_type?
+              add_offense_for_bare_assignment(node)
+            end
+          end
+
+          private
+
+          def add_offense_for_bare_assignment(sub_node)
+            if sub_node.receiver == s(:send, nil, :node) # node['foo'] scenario
+              add_offense(sub_node.receiver.loc.selector, severity: :warning)
+            elsif sub_node.receiver &&
+                  sub_node.receiver&.node_parts.first == s(:send, nil, :node) &&
+                  sub_node.receiver&.node_parts[1] == :[] # node['foo']['bar'] scenario
+              add_offense(sub_node.receiver.node_parts.first.loc.selector, severity: :warning)
+            end
+          end
+        end
+      end
+    end
+  end
+end