Merge pull request #9186 from mhashizume/PUP-11768/main/layout-cops

(PUP-11768) Enable and fix Rubocop Layout Cops

Author: mhashizume

.rubocop.yml

@@ -22,6 +22,12 @@ Layout/ArgumentAlignment:
   Exclude:
     - 'lib/puppet/defaults.rb'
 
+# The confine statements in package providers unexpectedly affect this cop.
+Layout/BeginEndAlignment:
+  Enabled: true
+  Exclude:
+    - 'lib/puppet/provider/package/*.rb'
+
 # puppet uses symbol booleans in types and providers to work around long standing
 # bugs when trying to manage falsey pararameters and properties
 Lint/BooleanSymbol:

.rubocop_todo.yml

@@ -22,87 +22,6 @@ I18n/GetText/DecorateStringFormattingUsingPercent:
 I18n/RailsI18n/DecorateString:
   Enabled: false
 
-# This cop supports safe auto-correction (--auto-correct).
-# Configuration parameters: EnforcedStyle, IndentationWidth.
-# SupportedStyles: outdent, indent
-Layout/AccessModifierIndentation:
-  Exclude:
-    - 'lib/puppet/module_tool/install_directory.rb'
-    - 'lib/puppet/pops/evaluator/collector_transformer.rb'
-    - 'lib/puppet/provider/service/init.rb'
-    - 'lib/puppet/provider/service/upstart.rb'
-    - 'lib/puppet/settings/config_file.rb'
-    - 'lib/puppet/settings/file_setting.rb'
-    - 'lib/puppet/util/command_line/trollop.rb'
-    - 'lib/puppet/util/run_mode.rb'
-
-# This cop supports safe auto-correction (--auto-correct).
-# Configuration parameters: EnforcedStyle, IndentationWidth.
-# SupportedStyles: with_first_element, with_fixed_indentation
-Layout/ArrayAlignment:
-  Exclude:
-    - 'lib/puppet/module/plan.rb'
-    - 'lib/puppet/pops/model/model_tree_dumper.rb'
-    - 'lib/puppet/pops/parser/heredoc_support.rb'
-    - 'lib/puppet/pops/parser/lexer2.rb'
-    - 'lib/puppet/provider/service/init.rb'
-    - 'lib/puppet/util/log/destinations.rb'
-    - 'lib/puppet/util/posix.rb'
-    - 'lib/puppet/util/windows/registry.rb'
-
-# This cop supports safe auto-correction (--auto-correct).
-# Configuration parameters: IndentationWidth.
-Layout/AssignmentIndentation:
-  Exclude:
-    - 'lib/puppet/application/apply.rb'
-    - 'lib/puppet/ffi/windows/constants.rb'
-    - 'lib/puppet/indirector/request.rb'
-    - 'lib/puppet/parser/e4_parser_adapter.rb'
-    - 'lib/puppet/pops/evaluator/evaluator_impl.rb'
-    - 'lib/puppet/pops/model/model_tree_dumper.rb'
-    - 'lib/puppet/pops/parser/lexer_support.rb'
-    - 'lib/puppet/pops/types/type_parser.rb'
-    - 'lib/puppet/resource/type.rb'
-    - 'lib/puppet/settings.rb'
-    - 'lib/puppet/settings/environment_conf.rb'
-    - 'lib/puppet/type/resources.rb'
-    - 'lib/puppet/util.rb'
-    - 'lib/puppet/util/windows/daemon.rb'
-
-# This cop supports safe auto-correction (--auto-correct).
-# Configuration parameters: EnforcedStyleAlignWith, Severity.
-# SupportedStylesAlignWith: start_of_line, begin
-Layout/BeginEndAlignment:
-  Exclude:
-    - 'lib/puppet/provider/package/apt.rb'
-    - 'lib/puppet/provider/package/aptrpm.rb'
-    - 'lib/puppet/provider/package/dnf.rb'
-    - 'lib/puppet/provider/package/rpm.rb'
-    - 'lib/puppet/provider/package/tdnf.rb'
-    - 'lib/puppet/provider/package/yum.rb'
-
-# This cop supports safe auto-correction (--auto-correct).
-# Configuration parameters: EnforcedStyleAlignWith.
-# SupportedStylesAlignWith: either, start_of_block, start_of_line
-Layout/BlockAlignment:
-  Exclude:
-    - 'lib/puppet/defaults.rb'
-    - 'lib/puppet/node/environment.rb'
-    - 'lib/puppet/parser/functions/lookup.rb'
-    - 'lib/puppet/provider/package/dnfmodule.rb'
-    - 'lib/puppet/type/file.rb'
-    - 'lib/puppet/util/windows/principal.rb'
-
-# This cop supports safe auto-correction (--auto-correct).
-# Configuration parameters: EnforcedStyle, IndentOneStep, IndentationWidth.
-# SupportedStyles: case, end
-Layout/CaseIndentation:
-  Enabled: false
-
-# This cop supports safe auto-correction (--auto-correct).
-Layout/ClosingHeredocIndentation:
-  Enabled: false
-
 # This cop supports safe auto-correction (--auto-correct).
 Layout/ClosingParenthesisIndentation:
   Enabled: false

lib/puppet/application/apply.rb

@@ -234,12 +234,12 @@ def main
         # rule and gets logged.
         #
         catalog =
-        begin
-          Puppet::Resource::Catalog.indirection.find(node.name, :use_node => node)
-        rescue Puppet::Error
-          # already logged and handled by the compiler, including Puppet::ParseErrorWithIssue
-          exit(1)
-        end
+          begin
+            Puppet::Resource::Catalog.indirection.find(node.name, :use_node => node)
+          rescue Puppet::Error
+            # already logged and handled by the compiler, including Puppet::ParseErrorWithIssue
+            exit(1)
+          end
 
         # Resolve all deferred values and replace them / mutate the catalog
         Puppet::Pops::Evaluator::DeferredResolver.resolve_and_replace(node.facts, catalog, apply_environment, Puppet[:preprocess_deferred])

lib/puppet/application/doc.rb

@@ -112,7 +112,7 @@ module - see https://github.com/puppetlabs/puppetlabs-strings for more informati
 ---------
 Copyright (c) 2011 Puppet Inc., LLC Licensed under the Apache 2.0 License
 
-HELP
+    HELP
   end
 
   def handle_unknown( opt, arg )

lib/puppet/application/ssl.rb

@@ -83,7 +83,7 @@ def help
 
  * show:
   Print the full-text version of this host's certificate.
-HELP
+    HELP
   end
 
   option('--target CERTNAME') do |arg|

lib/puppet/datatypes/error.rb

@@ -14,7 +14,7 @@
     functions => {
       message => Callable[[], String[1]]
     }
-    PUPPET
+  PUPPET
 
   require_relative '../../puppet/datatypes/impl/error'
 

lib/puppet/defaults.rb

@@ -430,7 +430,7 @@ def self.initialize_default_settings!(settings)
       between the configured or resolved path.
 
       When set to false, the resolved paths are reported instead of the configured paths.
-    EOT
+      EOT
     },
     :use_last_environment => {
       :type     => :boolean,
@@ -441,7 +441,7 @@ def self.initialize_default_settings!(settings)
       environment and skip the node request.
 
       When set to false, we will do the node request and ignore the environment data from the last_run_summary file.
-    EOT
+      EOT
     },
     :always_retry_plugins => {
         :type     => :boolean,
@@ -462,7 +462,7 @@ def self.initialize_default_settings!(settings)
         feature.  This behavior is almost always appropriate for the server,
         and can result in a significant performance improvement for types and
         features that are checked frequently.
-      EOT
+        EOT
     },
     :diff_args => {
         :default  => lambda { default_diffargs },
@@ -493,10 +493,10 @@ def self.initialize_default_settings!(settings)
           currently cannot daemonize).",
         :short    => "D",
         :hook     => proc do |value|
-          if value and Puppet::Util::Platform.windows?
-            raise "Cannot daemonize on Windows"
-          end
-      end
+                       if value and Puppet::Util::Platform.windows?
+                         raise "Cannot daemonize on Windows"
+                       end
+                     end
     },
     :maximum_uid => {
         :default  => 4294967290,
@@ -709,7 +709,7 @@ def self.initialize_default_settings!(settings)
 releases this value will only determine if file content is cached.
 
 Valid values are 0 (never cache) and 15 (15 second minimum wait time).
-            WARNING
+          WARNING
         end
       end
     },

lib/puppet/ffi/windows/constants.rb

@@ -47,25 +47,25 @@ module Constants
     FILE_ALL_ACCESS = STANDARD_RIGHTS_REQUIRED | SYNCHRONIZE | 0x1FF
 
     FILE_GENERIC_READ =
-       STANDARD_RIGHTS_READ |
-       FILE_READ_DATA |
-       FILE_READ_ATTRIBUTES |
-       FILE_READ_EA |
-       SYNCHRONIZE
+      STANDARD_RIGHTS_READ |
+      FILE_READ_DATA |
+      FILE_READ_ATTRIBUTES |
+      FILE_READ_EA |
+      SYNCHRONIZE
 
     FILE_GENERIC_WRITE =
-       STANDARD_RIGHTS_WRITE |
-       FILE_WRITE_DATA |
-       FILE_WRITE_ATTRIBUTES |
-       FILE_WRITE_EA |
-       FILE_APPEND_DATA |
-       SYNCHRONIZE
+      STANDARD_RIGHTS_WRITE |
+      FILE_WRITE_DATA |
+      FILE_WRITE_ATTRIBUTES |
+      FILE_WRITE_EA |
+      FILE_APPEND_DATA |
+      SYNCHRONIZE
 
     FILE_GENERIC_EXECUTE =
-       STANDARD_RIGHTS_EXECUTE |
-       FILE_READ_ATTRIBUTES |
-       FILE_EXECUTE |
-       SYNCHRONIZE
+      STANDARD_RIGHTS_EXECUTE |
+      FILE_READ_ATTRIBUTES |
+      FILE_EXECUTE |
+      SYNCHRONIZE
 
     REPLACEFILE_WRITE_THROUGH         = 0x1
     REPLACEFILE_IGNORE_MERGE_ERRORS   = 0x2

lib/puppet/file_system/uniquefile.rb

@@ -100,14 +100,14 @@ def path
 
   def make_tmpname(prefix_suffix, n)
     case prefix_suffix
-      when String
-        prefix = prefix_suffix
-        suffix = ""
-      when Array
-        prefix = prefix_suffix[0]
-        suffix = prefix_suffix[1]
-      else
-        raise ArgumentError, _("unexpected prefix_suffix: %{value}") % { value: prefix_suffix.inspect }
+    when String
+      prefix = prefix_suffix
+      suffix = ""
+    when Array
+      prefix = prefix_suffix[0]
+      suffix = prefix_suffix[1]
+    else
+      raise ArgumentError, _("unexpected prefix_suffix: %{value}") % { value: prefix_suffix.inspect }
     end
     t = Time.now.strftime("%Y%m%d")
     path = "#{prefix}#{t}-#{$$}-#{rand(0x100000000).to_s(36)}"

lib/puppet/http/dns.rb

@@ -49,9 +49,9 @@ def each_srv_record(domain, service_name = :puppet, &block)
       Puppet.debug "Searching for SRV records for domain: #{domain}"
 
       case service_name
-        when :puppet then service = '_x-puppet'
-        when :file   then service = '_x-puppet-fileserver'
-        else              service = "_x-puppet-#{service_name}"
+      when :puppet then service = '_x-puppet'
+      when :file   then service = '_x-puppet-fileserver'
+      else              service = "_x-puppet-#{service_name}"
       end
       record_name = "#{service}._tcp.#{domain}"