Rubocop: resolve Layout

features/step_definitions/asume_role_steps.rb

@@ -1,7 +1,9 @@
 Then(/^I expect the role "([^"]*)" is assumed in account "([^"]*)"$/) do |role, account|
-  expect(Aws::AssumeRoleCredentials).to receive(:new).with({
-    region: instance_of(String),
-    role_arn: "arn:aws:iam::#{account}:role/#{role}",
-    role_session_name: instance_of(String)
-  })
+  expect(Aws::AssumeRoleCredentials).to receive(:new).with(
+    {
+      region: instance_of(String),
+      role_arn: "arn:aws:iam::#{account}:role/#{role}",
+      role_session_name: instance_of(String)
+    }
+  )
 end

features/step_definitions/stack_steps.rb

@@ -28,7 +28,6 @@ def extract_hash_from_kv_string(string)
   end
 end
 
-
 Given(/^I stub the following stacks:$/) do |table|
   table.hashes.each do |row|
     row.symbolize_keys!
@@ -62,7 +61,9 @@ def extract_hash_from_kv_string(string)
 end
 
 Given(/^I stub CloudFormation validate calls to fail validation with message "([^"]*)"$/) do |message|
-  allow(StackMaster.cloud_formation_driver).to receive(:validate_template).and_raise(Aws::CloudFormation::Errors::ValidationError.new('', message))
+  allow(StackMaster.cloud_formation_driver)
+    .to receive(:validate_template)
+    .and_raise(Aws::CloudFormation::Errors::ValidationError.new('', message))
 end
 
 Given(/^I stub the CloudFormation driver$/) do

lib/stack_master.rb

@@ -137,6 +137,7 @@ def debug?
 
   def debug(message)
     return unless debug?
+
     stderr.puts Rainbow("[DEBUG] #{message}").color(:green)
   end
 
@@ -164,6 +165,7 @@ def skip_account_check?
   end
 
   attr_accessor :non_interactive_answer
+
   @non_interactive_answer = 'y'
 
   def base_dir

lib/stack_master/aws_driver/cloud_formation.rb

@@ -14,31 +14,31 @@ def set_region(value)
         end
       end
 
-      def_delegators :cf, :create_change_set,
-                          :describe_change_set,
-                          :execute_change_set,
-                          :delete_change_set,
-                          :delete_stack,
-                          :cancel_update_stack,
-                          :describe_stack_resources,
-                          :get_template,
-                          :get_stack_policy,
-                          :set_stack_policy,
-                          :describe_stack_events,
-                          :update_stack,
-                          :create_stack,
-                          :validate_template,
-                          :describe_stacks,
-                          :detect_stack_drift,
-                          :describe_stack_drift_detection_status,
-                          :describe_stack_resource_drifts
+      def_delegators :cf,
+                     :create_change_set,
+                     :describe_change_set,
+                     :execute_change_set,
+                     :delete_change_set,
+                     :delete_stack,
+                     :cancel_update_stack,
+                     :describe_stack_resources,
+                     :get_template,
+                     :get_stack_policy,
+                     :set_stack_policy,
+                     :describe_stack_events,
+                     :update_stack,
+                     :create_stack,
+                     :validate_template,
+                     :describe_stacks,
+                     :detect_stack_drift,
+                     :describe_stack_drift_detection_status,
+                     :describe_stack_resource_drifts
 
       private
 
       def cf
         @cf ||= Aws::CloudFormation::Client.new({ region: region, retry_limit: 10 })
       end
-
     end
   end
 end

lib/stack_master/aws_driver/s3.rb

@@ -11,18 +11,19 @@ def set_region(region)
       end
 
       def upload_files(bucket: nil, prefix: nil, region: nil, files: {})
-        raise StackMaster::AwsDriver::S3ConfigurationError, 'A bucket must be specified in order to use S3' unless bucket
+        unless bucket
+          raise StackMaster::AwsDriver::S3ConfigurationError, 'A bucket must be specified in order to use S3'
+        end
 
         return if files.empty?
 
         s3 = new_s3_client(region: region)
 
-        current_objects = s3.list_objects({
-          prefix: prefix,
-          bucket: bucket
-        }).map(&:contents).flatten.inject({}){|h,obj|
-          h.merge(obj.key => obj)
-        }
+        current_objects =
+          s3.list_objects({ prefix: prefix, bucket: bucket })
+            .map(&:contents)
+            .flatten
+            .inject({}) { |h, obj| h.merge(obj.key => obj) }
 
         StackMaster.stdout.puts "Uploading files to S3:"
 
@@ -35,15 +36,18 @@ def upload_files(bucket: nil, prefix: nil, region: nil, files: {})
           s3_md5 = current_objects[object_key] ? current_objects[object_key].etag.gsub("\"", '') : nil
 
           next if compiled_template_md5 == s3_md5
+
           s3_uri = "s3://#{bucket}/#{object_key}"
           StackMaster.stdout.print "- #{File.basename(path)} => #{s3_uri} "
 
-          s3.put_object({
-            bucket: bucket,
-            key: object_key,
-            body: body,
-            metadata: { md5: compiled_template_md5 }
-          })
+          s3.put_object(
+            {
+              bucket: bucket,
+              key: object_key,
+              body: body,
+              metadata: { md5: compiled_template_md5 }
+            }
+          )
           StackMaster.stdout.puts "done."
         end
       end

lib/stack_master/change_set.rb

@@ -42,15 +42,15 @@ def initialize(describe_change_set_response)
     end
 
     def display(io)
-      io.puts <<-EOL
+      io.puts <<~EOL
 
-========================================
-Proposed change set:
-EOL
+        ========================================
+        Proposed change set:
+      EOL
       @response.changes.each do |change|
         display_resource_change(io, change.resource_change)
       end
-io.puts "========================================"
+      io.puts "========================================"
     end
 
     def failed?

lib/stack_master/cli.rb

@@ -5,7 +5,7 @@ module StackMaster
   class CLI
     include Commander::Methods
 
-    def initialize(argv, stdin=STDIN, stdout=STDOUT, stderr=STDERR, kernel=Kernel)
+    def initialize(argv, stdin = STDIN, stdout = STDOUT, stderr = STDERR, kernel = Kernel)
       @argv, @stdin, @stdout, @stderr, @kernel = argv, stdin, stdout, stderr, kernel
       Commander::Runner.instance_variable_set('@instance', Commander::Runner.new(argv))
       StackMaster.stdout = @stdout
@@ -41,9 +41,13 @@ def execute!
       command :apply do |c|
         c.syntax = 'stack_master apply [region_or_alias] [stack_name]'
         c.summary = 'Creates or updates a stack'
-        c.description = "Creates or updates a stack. Shows a diff of the proposed stack's template and parameters. Tails stack events until CloudFormation has completed."
+        c.description = "Creates or updates a stack. Shows a diff of the proposed stack's template and parameters. " \
+                        "Tails stack events until CloudFormation has completed."
         c.example 'update a stack named myapp-vpc in us-east-1', 'stack_master apply us-east-1 myapp-vpc'
-        c.option '--on-failure ACTION', String, "Action to take on CREATE_FAILURE. Valid Values: [ DO_NOTHING | ROLLBACK | DELETE ]. Default: ROLLBACK\nNote: You cannot use this option with Serverless Application Model (SAM) templates."
+        c.option '--on-failure ACTION', String, "Action to take on CREATE_FAILURE. Valid Values: " \
+                                                "[ DO_NOTHING | ROLLBACK | DELETE ]. Default: ROLLBACK\n" \
+                                                "Note: You cannot use this option with Serverless Application Model " \
+                                                "(SAM) templates."
         c.option '--yes-param PARAM_NAME', String, "Auto-approve stack updates when only parameter PARAM_NAME changes"
         c.action do |args, options|
           options.default config: default_config_file
@@ -171,11 +175,13 @@ def execute!
       command :status do |c|
         c.syntax = 'stack_master status'
         c.summary = 'Check the current status stacks.'
-        c.description = 'Checks the status of all stacks defined in the stack_master.yml file. Warning this operation can be somewhat slow.'
+        c.description = 'Checks the status of all stacks defined in the stack_master.yml file. ' \
+                        'Warning this operation can be somewhat slow.'
         c.example 'description', 'Check the status of all stack definitions'
         c.action do |args, options|
           options.default config: default_config_file
           say "Invalid arguments. stack_master status" and return unless args.size == 0
+
           config = load_config(options.config)
           StackMaster::Commands::Status.perform(config, nil, options)
         end
@@ -184,11 +190,13 @@ def execute!
       command :tidy do |c|
         c.syntax = 'stack_master tidy'
         c.summary = 'Try to identify extra & missing files.'
-        c.description = 'Cross references stack_master.yml with the template and parameter directories to identify extra or missing files.'
+        c.description = 'Cross references stack_master.yml with the template and parameter directories ' \
+                        'to identify extra or missing files.'
         c.example 'description', 'Check for missing or extra files'
         c.action do |args, options|
           options.default config: default_config_file
           say "Invalid arguments. stack_master tidy" and return unless args.size == 0
+
           config = load_config(options.config)
           StackMaster::Commands::Tidy.perform(config, nil, options)
         end
@@ -269,11 +277,14 @@ def execute_stacks_command(command, args, options)
           success = false
         end
         stack_definitions = stack_definitions.select do |stack_definition|
-          running_in_allowed_account?(stack_definition.allowed_accounts) && StackStatus.new(config, stack_definition).changed?
+          running_in_allowed_account?(stack_definition.allowed_accounts) &&
+            StackStatus.new(config, stack_definition).changed?
         end if options.changed
         stack_definitions.each do |stack_definition|
           StackMaster.cloud_formation_driver.set_region(stack_definition.region)
-          StackMaster.stdout.puts "Executing #{command.command_name} on #{stack_definition.stack_name} in #{stack_definition.region}"
+          StackMaster.stdout.puts(
+            "Executing #{command.command_name} on #{stack_definition.stack_name} in #{stack_definition.region}"
+          )
           success = execute_if_allowed_account(stack_definition.allowed_accounts) do
             command.perform(config, stack_definition, options).success?
           end
@@ -283,20 +294,23 @@ def execute_stacks_command(command, args, options)
     end
 
     def show_other_region_candidates(config, stack_name)
-      candidates = config.filter(region="", stack_name=stack_name)
+      candidates = config.filter(region = "", stack_name = stack_name)
       return if candidates.empty?
 
       StackMaster.stdout.puts "Stack name #{stack_name} exists in regions: #{candidates.map(&:region).join(', ')}"
     end
 
     def execute_if_allowed_account(allowed_accounts, &block)
       raise ArgumentError, "Block required to execute this method" unless block_given?
+
       if running_in_allowed_account?(allowed_accounts)
         block.call
       else
         account_text = "'#{identity.account}'"
         account_text << " (#{identity.account_aliases.join(', ')})" if identity.account_aliases.any?
-        StackMaster.stdout.puts "Account #{account_text} is not an allowed account. Allowed accounts are #{allowed_accounts}."
+        StackMaster.stdout.puts(
+          "Account #{account_text} is not an allowed account. Allowed accounts are #{allowed_accounts}."
+        )
         false
       end
     end

lib/stack_master/commands/apply.rb

@@ -144,6 +144,7 @@ def ask_update_confirmation!
 
       def upload_files
         return unless use_s3?
+
         s3.upload_files(**s3_options)
       end
 
@@ -153,14 +154,20 @@ def template_method
 
       def template_value
         if use_s3?
-          s3.url(bucket: @s3_config['bucket'], prefix: @s3_config['prefix'], region: @s3_config['region'], template: @stack_definition.s3_template_file_name)
+          s3.url(
+            bucket: @s3_config['bucket'],
+            prefix: @s3_config['prefix'],
+            region: @s3_config['region'],
+            template: @stack_definition.s3_template_file_name
+          )
         else
           proposed_stack.template
         end
       end
 
       def files_to_upload
         return {} unless use_s3?
+
         @stack_definition.s3_files.tap do |files|
           files[@stack_definition.s3_template_file_name] = {
             path: @stack_definition.template_file_path,
@@ -218,6 +225,7 @@ def set_stack_policy
         proposed_policy = proposed_stack.stack_policy_body
         # No need to reset a stack policy if it's nil or not changed
         return if proposed_policy.nil? || proposed_policy == current_policy
+
         StackMaster.stdout.print 'Setting a stack policy...'
         cf.set_stack_policy(
           stack_name: stack_name,

lib/stack_master/commands/delete.rb

@@ -12,7 +12,6 @@ def initialize(region, stack_name, options)
       end
 
       def perform
-
         return unless check_exists
 
         unless ask?("Really delete stack #{@stack_name} (y/n)? ")
@@ -27,11 +26,11 @@ def perform
       private
 
       def delete_stack
-        cf.delete_stack({stack_name: @stack_name})
+        cf.delete_stack({ stack_name: @stack_name })
       end
 
       def check_exists
-        cf.describe_stacks({stack_name: @stack_name})
+        cf.describe_stacks({ stack_name: @stack_name })
         true
       rescue Aws::CloudFormation::Errors::ValidationError
         failed("Stack does not exist")

lib/stack_master/commands/drift.rb

@@ -15,7 +15,10 @@ def perform
         detect_stack_drift_result = cf.detect_stack_drift(stack_name: stack_name)
         drift_results = wait_for_drift_results(detect_stack_drift_result.stack_drift_detection_id)
 
-        puts colorize("Drift Status: #{drift_results.stack_drift_status}", stack_drift_status_color(drift_results.stack_drift_status))
+        puts colorize(
+          "Drift Status: #{drift_results.stack_drift_status}",
+          stack_drift_status_color(drift_results.stack_drift_status)
+        )
         return if drift_results.stack_drift_status == 'IN_SYNC'
 
         failed
@@ -52,7 +55,7 @@ def display_drift(drift)
 
       def display_resource_drift(drift)
         diff = ::StackMaster::Diff.new(before: prettify_json(drift.expected_properties),
-                        after: prettify_json(drift.actual_properties))
+                                       after: prettify_json(drift.actual_properties))
         diff.display_colorized_diff
       end
 

lib/stack_master/commands/init.rb

@@ -28,9 +28,13 @@ def check_files
         @region_parameters_filename = File.join("parameters", @region, "#{@stack_name}.yml")
 
         if !@options.overwrite
-          [@stack_master_filename, @stack_json_filename, @parameters_filename, @region_parameters_filename].each do |filename|
+          [
+            @stack_master_filename, @stack_json_filename, @parameters_filename, @region_parameters_filename
+          ].each do |filename|
             if File.exist?(filename)
-              StackMaster.stderr.puts("Aborting: #{filename} already exists. Use --overwrite to force overwriting file.")
+              StackMaster.stderr.puts(
+                "Aborting: #{filename} already exists. Use --overwrite to force overwriting file."
+              )
               return false
             end
           end

lib/stack_master/commands/lint.rb

@@ -8,11 +8,11 @@ class Lint
 
       def perform
         unless cfn_lint_available
-          failed! 'Failed to run cfn-lint. You may need to install it using'\
-                  '`pip install cfn-lint`, or add it to $PATH.'\
-                  "\n"\
-                  '(See https://github.com/aws-cloudformation/cfn-python-lint'\
-                  ' for package information)'
+          failed! 'Failed to run cfn-lint. You may need to install it using' \
+                  '`pip install cfn-lint`, or add it to $PATH.' \
+                  "\n" \
+                  '(See https://github.com/aws-cloudformation/cfn-python-lint ' \
+                  'for package information)'
         end
 
         Tempfile.open(['stack', ".#{proposed_stack.template_format}"]) do |f|

lib/stack_master/commands/nag.rb

@@ -24,7 +24,6 @@ def stack_definition
       def proposed_stack
         @proposed_stack ||= Stack.generate(stack_definition, @config)
       end
-
     end
   end
 end