RubyGems - cookstyle - Versions diffs - 7.5.2 → 7.8.3 - Mend

cookstyle 7.5.2 → 7.8.3

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (226) hide show

data/lib/rubocop/cop/chef/deprecation/foodcritic_testing.rb CHANGED Viewed

@@ -23,7 +23,7 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   gem 'foodcritic'
         #   require 'foodcritic'
         #

data/lib/rubocop/cop/chef/deprecation/hwrp_without_provides.rb CHANGED Viewed

@@ -24,7 +24,7 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   class Chef
         #     class Resource
         #       class UlimitRule < Chef::Resource
@@ -36,7 +36,7 @@ module RuboCop
         #     end
         #   end
         #
-        #   # bad
+        #   #### incorrect
         #   class Chef
         #     class Resource
         #       class UlimitRule < Chef::Resource
@@ -50,7 +50,7 @@ module RuboCop
         #     end
         #   end
         #
-        #  # good when Chef Infra Client < 15 (but compatible with 16+ as well)
+        #  #### correct when Chef Infra Client < 15 (but compatible with 16+ as well)
         #   class Chef
         #     class Resource
         #       class UlimitRule < Chef::Resource
@@ -65,7 +65,7 @@ module RuboCop
         #     end
         #   end
         #
-        #  # good when Chef Infra Client 16+
+        #  #### correct when Chef Infra Client 16+
         #   class Chef
         #     class Resource
         #       class UlimitRule < Chef::Resource

data/lib/rubocop/cop/chef/deprecation/inherits_compat_resource.rb CHANGED Viewed

@@ -24,12 +24,12 @@ module RuboCop
         #
         # @example
         #
-        #  # bad
+        #  #### incorrect
         #  class AptUpdate < ChefCompat::Resource
         #    # some resource code
         #  end
         #
-        #  # good
+        #  #### correct
         #  class AptUpdate < Chef::Resource
         #    # some resource code
         #  end

data/lib/rubocop/cop/chef/deprecation/launchd_deprecated_hash_property.rb CHANGED Viewed

@@ -23,12 +23,12 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   launchd 'foo' do
         #     hash foo: 'bar'
         #   end
         #
-        #   # good
+        #   #### correct
         #   launchd 'foo' do
         #     plist_hash foo: 'bar'
         #   end

data/lib/rubocop/cop/chef/deprecation/legacy_notify_syntax.rb CHANGED Viewed

@@ -23,7 +23,7 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   template '/etc/www/configures-apache.conf' do
         #     notifies :restart, resources(service: 'apache')
         #   end
@@ -40,7 +40,7 @@ module RuboCop
         #     subscribes :restart, resources(service: service_name_variable), :immediately
         #   end
         #
-        #   # good
+        #   #### correct
         #   template '/etc/www/configures-apache.conf' do
         #     notifies :restart, 'service[apache]'
         #   end
@@ -74,7 +74,7 @@ module RuboCop
                                 when :str
                                   "'#{type.source}[#{name.value}]'"
                                 when :dstr
-                                  "\"#{type.source}[#{name.value.source}]\""
+                                  "\"#{type.source}[#{name.value}]\""
                                 else
                                   "\"#{type.source}[\#{#{name.source}}]\""
                                 end

data/lib/rubocop/cop/chef/deprecation/legacy_yum_cookbook_recipes.rb CHANGED Viewed

@@ -25,7 +25,7 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   include_recipe 'yum::elrepo'
         #   include_recipe 'yum::epel'
         #   include_recipe 'yum::ius'

data/lib/rubocop/cop/chef/deprecation/librarian_chefspec.rb CHANGED Viewed

@@ -23,7 +23,7 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   require 'chefspec/librarian'
         #
         class LibrarianChefSpec < Base

data/lib/rubocop/cop/chef/deprecation/locale_lc_all_property.rb CHANGED Viewed

@@ -23,7 +23,7 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   locale 'set locale' do
         #     lang 'en_gb.utf-8'
         #     lc_all 'en_gb.utf-8'

data/lib/rubocop/cop/chef/deprecation/log_resource_notifications.rb CHANGED Viewed

@@ -24,7 +24,7 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   template '/etc/foo' do
         #     source 'bar.erb'
         #     notifies :write, 'log[Aggregate notifications using a single log resource]', :immediately
@@ -34,7 +34,7 @@ module RuboCop
         #     notifies :restart, 'service[foo]', :delayed
         #   end
         #
-        #   # good
+        #   #### correct
         #   template '/etc/foo' do
         #     source 'bar.erb'
         #     notifies :run, 'notify_group[Aggregate notifications using a single notify_group resource]', :immediately

data/lib/rubocop/cop/chef/deprecation/macos_userdefaults_global_property.rb CHANGED Viewed

@@ -24,14 +24,14 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   macos_userdefaults 'set a value' do
         #     global true
         #     key 'key'
         #     value 'value'
         #   end
         #
-        #   # good
+        #   #### correct
         #   macos_userdefaults 'set a value' do
         #     key 'key'
         #     value 'value'

data/lib/rubocop/cop/chef/deprecation/name_property_and_default.rb CHANGED Viewed

@@ -23,11 +23,11 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   property :config_file, String, default: 'foo', name_property: true
         #   attribute :config_file, String, default: 'foo', name_attribute: true
         #
-        #   # good
+        #   #### correct
         #   property :config_file, String, name_property: true
         #   attribute :config_file, String, name_attribute: true
         #

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

@@ -22,16 +22,16 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   node.deep_fetch("foo")
         #
-        #   # good
+        #   #### correct
         #   node.read("foo")
         #
-        #   # bad
+        #   #### incorrect
         #   node.deep_fetch!("foo")
         #
-        #   # good
+        #   #### correct
         #   node.read!("foo")
         #
         class NodeDeepFetch < Base

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

@@ -23,14 +23,14 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   node.fqdn
         #   node.platform
         #   node.platform_family
         #   node.platform_version
         #   node.hostname
         #
-        #   # good
+        #   #### correct
         #   node['fqdn']
         #   node['platform']
         #   node['platform_family']

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

@@ -24,10 +24,10 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   node.set['foo'] = true
         #
-        #   # good
+        #   #### correct
         #   node.normal['foo'] = true
         #
         class NodeSet < Base

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

@@ -24,10 +24,10 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   node.set_unless['foo'] = true
         #
-        #   # good
+        #   #### correct
         #   node.normal_unless['foo'] = true
         #
         class NodeSetUnless < Base

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

@@ -22,13 +22,13 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   node['foo']['bar'] = 1
         #   node['foo']['bar'] << 1
         #   node['foo']['bar'] += 1
         #   node['foo']['bar'] -= 1
         #
-        #   # good
+        #   #### correct
         #   node.default['foo']['bar'] = 1
         #   node.default['foo']['bar'] << 1
         #   node.default['foo']['bar'] += 1

data/lib/rubocop/cop/chef/deprecation/partial_search_class_usage.rb CHANGED Viewed

@@ -23,7 +23,7 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   ::Chef::PartialSearch.new.search((:node, 'role:web',
         #     keys: { 'name' => [ 'name' ],
         #             'ip' => [ 'ipaddress' ],
@@ -35,7 +35,7 @@ module RuboCop
         #     puts result['kernel_version']
         #   end
         #
-        #   # good
+        #   #### correct
         #   search(:node, 'role:web',
         #     filter_result: { 'name' => [ 'name' ],
         #                      'ip' => [ 'ipaddress' ],

data/lib/rubocop/cop/chef/deprecation/partial_search_helper_usage.rb CHANGED Viewed

@@ -23,7 +23,7 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   partial_search(:node, 'role:web',
         #     keys: { 'name' => [ 'name' ],
         #             'ip' => [ 'ipaddress' ],
@@ -35,7 +35,7 @@ module RuboCop
         #     puts result['kernel_version']
         #   end
         #
-        #   # good
+        #   #### correct
         #   search(:node, 'role:web',
         #     filter_result: { 'name' => [ 'name' ],
         #                      'ip' => [ 'ipaddress' ],

data/lib/rubocop/cop/chef/deprecation/poise_archive.rb CHANGED Viewed

@@ -23,12 +23,12 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   poise_archive 'https://example.com/myapp.tgz' do
         #     destination '/opt/my_app'
         #   end
         #
-        #   # good
+        #   #### correct
         #   archive_file 'https://example.com/myapp.tgz' do
         #     destination '/opt/my_app'
         #   end

data/lib/rubocop/cop/chef/deprecation/powershell_cookbook_helpers.rb CHANGED Viewed

@@ -23,10 +23,10 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   Powershell::VersionHelper.powershell_version?('4.0')
         #
-        #   # good
+        #   #### correct
         #   node['powershell']['version'].to_f == 4.0
         #
         #   # better (Chef Infra Client 15.8+)

data/lib/rubocop/cop/chef/deprecation/require_recipe.rb CHANGED Viewed

@@ -24,10 +24,10 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   require_recipe 'foo'
         #
-        #   # good
+        #   #### correct
         #   include_recipe 'foo'
         #
         class RequireRecipe < Base

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

@@ -23,12 +23,12 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   def provides?
         #    true
         #   end
         #
-        #   # good
+        #   #### correct
         #   provides :SOME_PROVIDER_NAME
         #
         class ResourceOverridesProvidesMethod < Base

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

@@ -23,10 +23,10 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   my_resource = MyResource.dsl_name
         #
-        #   # good
+        #   #### correct
         #   my_resource = MyResource.resource_name
         #
         class ResourceUsesDslNameMethod < Base

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

@@ -23,7 +23,7 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   mycookbook/resources/myresource.rb:
         #   resource_name :mycookbook_myresource
         #

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

@@ -23,7 +23,7 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   provider_base ::Chef::Provider::SomethingSomething
         #
         class ResourceUsesProviderBaseMethod < Base

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

@@ -19,16 +19,16 @@ module RuboCop
   module Cop
     module Chef
       module Deprecations
-        # Don't call the deprecated updated= method in a resource to set the resource to updated. This method was removed from Chef Infra Client 13 and this will now cause an error. Instead wrap code that updated the state of the node in a converge_by block. Documentation on using the converge_by block can be found at https://docs.chef.io/custom_resources.html.
+        # Don't call the deprecated `updated=` method in a resource to set the resource to updated. This method was removed from Chef Infra Client 13 and this will now cause an error. Instead wrap code that updated the state of the node in a `converge_by` block. Documentation on using the `converge_by` block can be found at https://docs.chef.io/custom_resources/.
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   action :foo do
         #     updated = true
         #   end
         #
-        #   # good
+        #   #### correct
         #   action :foo do
         #     converge_by('resource did something) do
         #       # code that causes the resource to converge

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

@@ -24,11 +24,11 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   shell_out!('hostnamectl status', { returns: [0, 1] })
         #   shell_out('hostnamectl status', { returns: [0, 1] })
         #
-        #   # good
+        #   #### correct
         #   shell_out!('hostnamectl status', returns: [0, 1])
         #   shell_out('hostnamectl status', returns: [0, 1])
         #

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

@@ -24,7 +24,7 @@ module RuboCop
         #
         # @example
         #
-        #   # bad
+        #   #### incorrect
         #   ruby_block 'my special ruby block' do
         #     block do
         #       puts 'running'
@@ -32,7 +32,7 @@ module RuboCop
         #     action :create
         #   end
         #
-        #   # good
+        #   #### correct
         #   ruby_block 'my special ruby block' do
         #     block do
         #       puts 'running'