sfn 3.0.30 → 3.0.32

data/lib/sfn/command_module/planning.rb

@@ -0,0 +1,151 @@
+require "sfn"
+require "sparkle_formation"
+
+module Sfn
+  module CommandModule
+    # Planning helpers
+    module Planning
+      # Create a new planner instance
+      #
+      # @param [Miasma::Models::Orchestration::Stack]
+      # @return [Sfn::Planner]
+      def build_planner(stack)
+        klass_name = stack.api.class.to_s.split("::").last
+        if Planner.const_defined?(klass_name)
+          Planner.const_get(klass_name).new(ui, config, arguments, stack)
+        else
+          warn "Failed to build planner for current provider. No provider implemented. (`#{klass_name}`)"
+          nil
+        end
+      end
+
+      # Display plan result on the UI
+      #
+      # @param result [Miasma::Models::Orchestration::Stack::Plan]
+      def display_plan_information(result)
+        ui.info ui.color("Pre-update resource planning report:", :bold)
+        unless print_plan_result(result, [result.name])
+          ui.info "No resources life cycle changes detected in this update!"
+        end
+        cmd = self.class.to_s.split("::").last.downcase
+        ui.confirm "Apply this stack #{cmd}?" unless config[:plan_only]
+      end
+
+      # Print plan information to the UI
+      #
+      # @param info [Miasma::Models::Orchestration::Stack::Plan]
+      # @param names [Array<String>] nested names
+      def print_plan_result(info, names = [])
+        said_any_things = false
+        unless Array(info.stacks).empty?
+          info.stacks.each do |s_name, s_info|
+            result = print_plan_result(s_info, [*names, s_name].compact)
+            said_any_things ||= result
+          end
+        end
+        if !names.flatten.compact.empty? || info.name
+          said_things = false
+          output_name = names.empty? ? info.name : names.join(" > ")
+          ui.puts
+          ui.puts "  #{ui.color("Update plan for:", :bold)} #{ui.color(names.join(" > "), :blue)}"
+          unless Array(info.unknown).empty?
+            ui.puts "    #{ui.color("!!! Unknown update effect:", :red, :bold)}"
+            print_plan_items(info, :unknown, :red)
+            ui.puts
+            said_any_things = said_things = true
+          end
+          unless Array(info.unavailable).empty?
+            ui.puts "    #{ui.color("Update request not allowed:", :red, :bold)}"
+            print_plan_items(info, :unavailable, :red)
+            ui.puts
+            said_any_things = said_things = true
+          end
+          unless Array(info.replace).empty?
+            ui.puts "    #{ui.color("Resources to be replaced:", :red, :bold)}"
+            print_plan_items(info, :replace, :red)
+            ui.puts
+            said_any_things = said_things = true
+          end
+          unless Array(info.interrupt).empty?
+            ui.puts "    #{ui.color("Resources to be interrupted:", :yellow, :bold)}"
+            print_plan_items(info, :interrupt, :yellow)
+            ui.puts
+            said_any_things = said_things = true
+          end
+          unless Array(info.remove).empty?
+            ui.puts "    #{ui.color("Resources to be removed:", :red, :bold)}"
+            print_plan_items(info, :remove, :red)
+            ui.puts
+            said_any_things = said_things = true
+          end
+          unless Array(info.add).empty?
+            ui.puts "    #{ui.color("Resources to be added:", :green, :bold)}"
+            print_plan_items(info, :add, :green)
+            ui.puts
+            said_any_things = said_things = true
+          end
+          unless said_things
+            ui.puts "    #{ui.color("No resource lifecycle changes detected!", :green)}"
+            ui.puts
+            said_any_things = true
+          end
+        end
+        said_any_things
+      end
+
+      # Print planning items
+      #
+      # @param info [Miasma::Models::Orchestration::Stack::Plan] plan
+      # @param key [Symbol] key of items
+      # @param color [Symbol] color to flag
+      def print_plan_items(info, key, color)
+        collection = info.send(key)
+        max_name = collection.map(&:name).map(&:size).max
+        max_type = collection.map(&:type).map(&:size).max
+        max_p = collection.map(&:diffs).flatten(1).map(&:name).map(&:to_s).map(&:size).max
+        max_o = collection.map(&:diffs).flatten(1).map(&:current).map(&:to_s).map(&:size).max
+        collection.each do |val|
+          name = val.name
+          ui.print " " * 6
+          ui.print ui.color("[#{val.type}]", color)
+          ui.print " " * (max_type - val.type.size)
+          ui.print " " * 4
+          ui.print ui.color(name, :bold)
+          properties = Array(val.diffs).map(&:name)
+          unless properties.empty?
+            ui.print " " * (max_name - name.size)
+            ui.print " " * 4
+            ui.print "Reason: `#{properties.join("`, `")}`"
+          end
+          ui.puts
+          if config[:diffs]
+            unless val.diffs.empty?
+              p_name = nil
+              val.diffs.each do |diff|
+                if !diff.proposed.nil? || !diff.current.nil?
+                  p_name = diff.name
+                  ui.print " " * 8
+                  ui.print "#{p_name}: "
+                  ui.print " " * (max_p - p_name.size)
+                  ui.print ui.color("-#{diff.current}", :red) if diff.current
+                  ui.print " " * (max_o - diff.current.to_s.size)
+                  ui.print " "
+                  if diff.proposed == Sfn::Planner::RUNTIME_MODIFIED
+                    ui.puts ui.color("+#{diff.current} <Dependency Modified>", :green)
+                  else
+                    if diff.proposed.nil?
+                      ui.puts
+                    else
+                      ui.puts ui.color("+#{diff.proposed.to_s.gsub("__MODIFIED_REFERENCE_VALUE__", "<Dependency Modified>")}", :green)
+                    end
+                  end
+                end
+              end
+              ui.puts if p_name
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/sfn/command_module/stack.rb

@@ -1,5 +1,5 @@
-require 'sfn'
-require 'sparkle_formation'
+require "sfn"
+require "sparkle_formation"
 
 module Sfn
   module CommandModule
@@ -10,13 +10,13 @@ module Sfn
         # maximum number of attempts to get valid parameter value
         MAX_PARAMETER_ATTEMPTS = 5
         # Template parameter locations
-        TEMPLATE_PARAMETER_LOCATIONS = ['Parameters', 'parameters']
+        TEMPLATE_PARAMETER_LOCATIONS = ["Parameters", "parameters"]
         # Template parameter default value locations
-        TEMPLATE_PARAMETER_DEFAULTS = ['Default', 'defaultValue', 'default']
+        TEMPLATE_PARAMETER_DEFAULTS = ["Default", "defaultValue", "default"]
         # Template parameter no echo locations
-        TEMPLATE_PARAMETER_NOECHO = ['NoEcho']
+        TEMPLATE_PARAMETER_NOECHO = ["NoEcho"]
         # Template parameter no echo custom
-        TEMPLATE_PARAMETER_SFN_NOECHO = ['Quiet', 'quiet']
+        TEMPLATE_PARAMETER_SFN_NOECHO = ["Quiet", "quiet"]
 
         # Apply any defined remote stacks
         #
@@ -25,7 +25,7 @@ module Sfn
         def apply_stacks!(stack)
           remote_stacks = [config[:apply_stack]].flatten.compact
           remote_stacks.each do |stack_name|
-            stack_info = stack_name.split('__')
+            stack_info = stack_name.split("__")
             stack_info.unshift(nil) if stack_info.size == 1
             stack_location, stack_name = stack_info
             remote_stack = provider_for(stack_location).stack(stack_name)
@@ -66,7 +66,7 @@ module Sfn
           if config[:apply_mapping]
             valid_keys = config[:apply_mapping].keys.find_all do |a_key|
               a_key = a_key.to_s
-              key_parts = a_key.split('__')
+              key_parts = a_key.split("__")
               case key_parts.size
               when 3
                 provider_stack.api.data[:location] == key_parts[0] &&
@@ -85,7 +85,7 @@ module Sfn
             valid_keys -= to_remove
             Hash[
               valid_keys.map do |a_key|
-                cut_key = a_key.split('__').last
+                cut_key = a_key.split("__").last
                 [cut_key, config[:apply_mapping][a_key]]
               end
             ]
@@ -148,16 +148,16 @@ module Sfn
         # @param parameter_name [String] parameter name
         # @return [Array<String>] [expected_template_key, configuration_used_key]
         def locate_config_parameter_key(parameter_prefix, parameter_name, root_name)
-          check_name = parameter_name.downcase.tr('-_', '')
-          check_prefix = parameter_prefix.map { |i| i.downcase.tr('-_', '') }
+          check_name = parameter_name.downcase.tr("-_", "")
+          check_prefix = parameter_prefix.map { |i| i.downcase.tr("-_", "") }
           key_match = config[:parameters].keys.detect do |cp_key|
-            cp_key = cp_key.to_s.downcase.split('__').map { |i| i.tr('-_', '') }.join('__')
-            non_root_matcher = (check_prefix + [check_name]).join('__')
-            root_matcher = ([root_name] + check_prefix + [check_name]).join('__')
+            cp_key = cp_key.to_s.downcase.split("__").map { |i| i.tr("-_", "") }.join("__")
+            non_root_matcher = (check_prefix + [check_name]).join("__")
+            root_matcher = ([root_name] + check_prefix + [check_name]).join("__")
             cp_key == non_root_matcher ||
               cp_key == root_matcher
           end
-          actual_key = (parameter_prefix + [parameter_name]).compact.join('__')
+          actual_key = (parameter_prefix + [parameter_name]).compact.join("__")
           if key_match
             ui.debug "Remapping configuration runtime parameter `#{key_match}` -> `#{actual_key}`"
             config[:parameters][actual_key] = config[:parameters].delete(key_match)
@@ -179,9 +179,9 @@ module Sfn
           attempt = 0
           if !valid && !param_banner
             if sparkle.is_a?(SparkleFormation)
-              ui.info "#{ui.color('Stack runtime parameters:', :bold)} - template: #{ui.color(sparkle.root_path.map(&:name).map(&:to_s).join(' > '), :green, :bold)}"
+              ui.info "#{ui.color("Stack runtime parameters:", :bold)} - template: #{ui.color(sparkle.root_path.map(&:name).map(&:to_s).join(" > "), :green, :bold)}"
             else
-              ui.info ui.color('Stack runtime parameters:', :bold)
+              ui.info ui.color("Stack runtime parameters:", :bold)
             end
             param_banner = true
           end
@@ -194,17 +194,17 @@ module Sfn
             )
             if config[:interactive_parameters]
               no_echo = !!TEMPLATE_PARAMETER_NOECHO.detect { |loc_key|
-                param_value[loc_key].to_s.downcase == 'true'
+                param_value[loc_key].to_s.downcase == "true"
               }
               sfn_no_echo = TEMPLATE_PARAMETER_SFN_NOECHO.map do |loc_key|
                 res = param_value.delete(loc_key).to_s.downcase
-                res if !res.empty? && res != 'false'
+                res if !res.empty? && res != "false"
               end.compact.first
               no_echo = sfn_no_echo if sfn_no_echo
               answer = ui.ask_question(
-                "#{param_name.split(/([A-Z]+[^A-Z]*)/).find_all { |s| !s.empty? }.join(' ')}",
+                "#{param_name.split(/([A-Z]+[^A-Z]*)/).find_all { |s| !s.empty? }.join(" ")}",
                 :default => default,
-                :hide_default => sfn_no_echo == 'all',
+                :hide_default => sfn_no_echo == "all",
                 :no_echo => !!no_echo,
               )
             else
@@ -216,11 +216,11 @@ module Sfn
               valid = true
             else
               validation.each do |validation_error|
-                ui.error validation_error.last
+                ui.error "#{param_name}: #{validation_error.last}"
               end
             end
             if attempt > MAX_PARAMETER_ATTEMPTS
-              ui.fatal 'Failed to receive allowed parameter!'
+              ui.fatal "Failed to receive allowed parameter!"
               exit 1
             end
           end
@@ -279,8 +279,8 @@ module Sfn
           end
           Smash[
             config.fetch(:parameters, {}).map do |k, v|
-              strip_key = parameter_prefix ? k.sub(/#{parameter_prefix.join('__')}_{2}?/, '') : k
-              unless strip_key.include?('__')
+              strip_key = parameter_prefix ? k.sub(/#{parameter_prefix.join("__")}_{2}?/, "") : k
+              unless strip_key.include?("__")
                 [strip_key, v]
               end
             end.compact
@@ -299,7 +299,7 @@ module Sfn
         def config_root_parameters
           Hash[
             config.fetch(:parameters, {}).find_all do |k, v|
-              !k.include?('__')
+              !k.include?("__")
             end
           ]
         end
@@ -318,13 +318,13 @@ module Sfn
         def validate_stack_parameter(c_stack, p_key, p_ns_key, c_value)
           stack_value = c_stack.parameters[p_key]
           p_stack = c_stack.data[:parent_stack]
-          unless config[:parameter_validation] == 'none'
+          unless config[:parameter_validation] == "none"
             if c_value.is_a?(Hash)
               case c_value.keys.first
-              when 'Ref'
+              when "Ref"
                 current_value = p_stack.parameters[c_value.values.first]
-              when 'Fn::Att'
-                resource_name, output_name = c_value.values.first.split('.', 2)
+              when "Fn::Att"
+                resource_name, output_name = c_value.values.first.split(".", 2)
                 ref_stack = p_stack.nested_stacks.detect { |i| i.data[:logical_id] == resource_name }
                 if ref_stack
                   output = ref_stack.outputs.detect do |o|
@@ -339,14 +339,14 @@ module Sfn
               current_value = c_value
             end
             if current_value && current_value.to_s != stack_value.to_s
-              if config[:parameter_validation] == 'default'
-                ui.warn 'Nested stack has been altered directly! This update may cause unexpected modifications!'
+              if config[:parameter_validation] == "default"
+                ui.warn "Nested stack has been altered directly! This update may cause unexpected modifications!"
                 ui.warn "Stack name: #{c_stack.name}. Parameter: #{p_key}. Current value: #{stack_value}. Expected value: #{current_value} (via: #{c_value.inspect})"
-                answer = ui.ask_question("Use current value or expected value for #{p_key} [current/expected]?", :valid => ['current', 'expected'])
+                answer = ui.ask_question("Use current value or expected value for #{p_key} [current/expected]?", :valid => ["current", "expected"])
               else
                 answer = config[:parameter_validation]
               end
-              answer == 'expected'
+              answer == "expected"
             else
               true
             end

data/lib/sfn/command_module/template.rb

@@ -1,7 +1,7 @@
-require 'sfn'
-require 'sparkle_formation'
+require "sfn"
+require "sparkle_formation"
 
-require 'pathname'
+require "pathname"
 
 module Sfn
   module CommandModule
@@ -48,7 +48,7 @@ module Sfn
         def request_compile_parameter(p_name, p_config, cur_val, nested = false)
           result = nil
           attempts = 0
-          parameter_type = p_config.fetch(:type, 'string').to_s.downcase.to_sym
+          parameter_type = p_config.fetch(:type, "string").to_s.downcase.to_sym
           if parameter_type == :complex
             ui.debug "Compile time parameter `#{p_name}` is a complex type. Not requesting value from user."
             if cur_val.nil?
@@ -61,13 +61,13 @@ module Sfn
               cur_val = p_config[:default]
             end
             if cur_val.is_a?(Array)
-              cur_val = cur_val.map(&:to_s).join(',')
+              cur_val = cur_val.map(&:to_s).join(",")
             end
             until result && (!result.respond_to?(:empty?) || !result.empty?)
               attempts += 1
               if config[:interactive_parameters] && (!nested || !p_config.key?(:prompt_when_nested) || p_config[:prompt_when_nested] == true)
                 result = ui.ask_question(
-                  p_name.to_s.split('_').map(&:capitalize).join,
+                  p_name.to_s.split("_").map(&:capitalize).join,
                   :default => cur_val.to_s.empty? ? nil : cur_val.to_s,
                 )
               else
@@ -76,11 +76,11 @@ module Sfn
               case parameter_type
               when :string
                 if p_config[:multiple]
-                  result = result.split(',').map(&:strip)
+                  result = result.split(",").map(&:strip)
                 end
               when :number
                 if p_config[:multiple]
-                  result = result.split(',').map(&:strip)
+                  result = result.split(",").map(&:strip)
                   new_result = result.map do |item|
                     new_item = item.to_i
                     new_item if new_item.to_s == item
@@ -97,7 +97,7 @@ module Sfn
               unless valid == true
                 result = nil
                 valid.each do |invalid_msg|
-                  ui.error invalid_msg.last
+                  ui.error "#{p_name}: #{invalid_msg.last}"
                 end
               end
               if result.nil? || (result.respond_to?(:empty?) && result.empty?)
@@ -150,7 +150,7 @@ module Sfn
               end
               collection.set_root(root_pack)
             rescue Errno::ENOENT
-              ui.warn 'No local SparkleFormation files detected'
+              ui.warn "No local SparkleFormation files detected"
             end
             sparkle_packs.each do |pack|
               collection.add_sparkle(pack)
@@ -185,8 +185,8 @@ module Sfn
               end
               sf.compile_time_parameter_setter do |formation|
                 f_name = formation.root_path.map(&:name).map(&:to_s)
-                pathed_name = f_name.join(' > ')
-                f_name = f_name.join('__')
+                pathed_name = f_name.join(" > ")
+                f_name = f_name.join("__")
                 if formation.root? && compile_state[f_name].nil?
                   current_state = compile_state
                 else
@@ -200,7 +200,7 @@ module Sfn
                   current_state = current_state.merge(formation.compile_state)
                 end
                 unless formation.parameters.empty?
-                  ui.info "#{ui.color('Compile time parameters:', :bold)} - template: #{ui.color(pathed_name, :green, :bold)}" unless config[:print_only]
+                  ui.info "#{ui.color("Compile time parameters:", :bold)} - template: #{ui.color(pathed_name, :green, :bold)}" unless config[:print_only]
                   formation.parameters.each do |k, v|
                     valid_keys = [
                       "#{f_name}__#{k}",
@@ -251,7 +251,7 @@ module Sfn
               template
             end
           else
-            raise ArgumentError.new 'Failed to locate template for processing!'
+            raise ArgumentError.new "Failed to locate template for processing!"
           end
         end
 
@@ -262,7 +262,7 @@ module Sfn
           ui.debug "Initial compile parameters - #{compile_state}"
           compile_state.keys.each do |cs_key|
             unless cs_key.to_s.start_with?("#{arguments.first}__")
-              named_cs_key = [arguments.first, cs_key].compact.join('__')
+              named_cs_key = [arguments.first, cs_key].compact.join("__")
               non_named = compile_state.delete(cs_key)
               if non_named && !compile_state.key?(named_cs_key)
                 ui.debug "Setting non-named compile parameter `#{cs_key}` into `#{named_cs_key}`"
@@ -282,8 +282,8 @@ module Sfn
         # Force user friendly error if nesting bucket is not set within configuration
         def validate_nesting_bucket!
           if config[:nesting_bucket].to_s.empty?
-            ui.error 'Missing required configuration value for `nesting_bucket`. Cannot generated nested templates!'
-            raise ArgumentError.new 'Required configuration value for `nesting_bucket` not provided.'
+            ui.error "Missing required configuration value for `nesting_bucket`. Cannot generated nested templates!"
+            raise ArgumentError.new "Required configuration value for `nesting_bucket` not provided."
           end
         end
 
@@ -307,13 +307,13 @@ module Sfn
               end
               file = bucket.files.build
               file.name = "#{name_args.first}_#{stack_name}.json"
-              file.content_type = 'text/json'
+              file.content_type = "text/json"
               file.body = MultiJson.dump(parameter_scrub!(stack_definition))
               file.save
               url = URI.parse(file.url)
               template_url = "#{url.scheme}://#{url.host}#{url.path}"
             end
-            resource.properties.set!('TemplateURL', template_url)
+            resource.properties.set!("TemplateURL", template_url)
           end
         end
 
@@ -331,16 +331,16 @@ module Sfn
             if current_stack && current_stack.data[:parent_stack]
               current_parameters.merge!(
                 current_stack.data[:parent_stack].template.fetch(
-                  'Resources', stack_name, 'Properties', 'Parameters', current_stack.data[:parent_stack].template.fetch(
-                    'resources', stack_name, 'properties', 'parameters', Smash.new
+                  "Resources", stack_name, "Properties", "Parameters", current_stack.data[:parent_stack].template.fetch(
+                    "resources", stack_name, "properties", "parameters", Smash.new
                   )
                 )
               )
             end
             full_stack_name = [
               config[:nesting_prefix],
-              stack.root_path.map(&:name).map(&:to_s).join('_'),
-            ].compact.join('/')
+              stack.root_path.map(&:name).map(&:to_s).join("_"),
+            ].compact.join("/")
             unless config[:print_only]
               result = Smash.new(
                 :parameters => populate_parameters!(stack,
@@ -390,7 +390,7 @@ module Sfn
           end
           file = bucket.files.build
           file.name = "#{full_stack_name}.json"
-          file.content_type = 'text/json'
+          file.content_type = "text/json"
           file.body = MultiJson.dump(parameter_scrub!(stack_definition))
           file.save
           result.merge!(
@@ -422,9 +422,9 @@ module Sfn
         # @return [Hash]
         def scrub_template(template)
           template = parameter_scrub!(template)
-          (template['Resources'] || {}).each do |r_name, r_content|
-            if valid_stack_types.include?(r_content['Type'])
-              result = (r_content['Properties'] || {}).delete('Stack')
+          (template["Resources"] || {}).each do |r_name, r_content|
+            if valid_stack_types.include?(r_content["Type"])
+              result = (r_content["Properties"] || {}).delete("Stack")
             end
           end
           template
@@ -439,11 +439,11 @@ module Sfn
           case provider
           when :aws
             if results[:parameters]
-              results['Parameters'] = results.delete(:parameters)
+              results["Parameters"] = results.delete(:parameters)
             end
             if results[:url]
               url = URI.parse(results.delete(:url))
-              results['TemplateURL'] = "#{url.scheme}://#{url.host}#{url.path}"
+              results["TemplateURL"] = "#{url.scheme}://#{url.host}#{url.path}"
             end
             results
           when :heat, :rackspace
@@ -461,10 +461,10 @@ module Sfn
             if results[:url]
               results[:templateLink] = Smash.new(
                 :uri => results.delete(:url),
-                :contentVersion => '1.0.0.0',
+                :contentVersion => "1.0.0.0",
               )
             end
-            results[:mode] = 'Incremental'
+            results[:mode] = "Incremental"
             results
           else
             raise "Unknown stack provider value given! `#{provider}`"
@@ -491,7 +491,7 @@ module Sfn
             translator = klass.new(template, args)
             translator.translate!
             template = translator.translated
-            ui.info "#{ui.color('Translation applied:', :bold)} #{ui.color(klass_name, :yellow)}"
+            ui.info "#{ui.color("Translation applied:", :bold)} #{ui.color(klass_name, :yellow)}"
           end
           template
         end
@@ -530,9 +530,9 @@ module Sfn
         # @return [String] path to template
         def prompt_for_template(prefix = nil)
           if prefix
-            collection_name = prefix.split('__').map do |c_name|
-              c_name.split('_').map(&:capitalize).join(' ')
-            end.join(' / ')
+            collection_name = prefix.split("__").map do |c_name|
+              c_name.split("_").map(&:capitalize).join(" ")
+            end.join(" / ")
             ui.info "Viewing collection: #{ui.color(collection_name, :bold)}"
             template_names = sparkle_collection.templates.fetch(provider.connection.provider, {}).keys.find_all do |t_name|
               t_name.to_s.start_with?(prefix.to_s)
@@ -541,46 +541,46 @@ module Sfn
             template_names = sparkle_collection.templates.fetch(provider.connection.provider, {}).keys
           end
           collections = template_names.map do |t_name|
-            t_name = t_name.to_s.sub(/^#{Regexp.escape(prefix.to_s)}/, '')
-            if t_name.include?('__')
-              c_name = t_name.split('__').first
-              [[prefix, c_name].compact.join('') + '__', c_name]
+            t_name = t_name.to_s.sub(/^#{Regexp.escape(prefix.to_s)}/, "")
+            if t_name.include?("__")
+              c_name = t_name.split("__").first
+              [[prefix, c_name].compact.join("") + "__", c_name]
             end
           end.compact.uniq(&:first)
           templates = template_names.map do |t_name|
-            t_name = t_name.to_s.sub(/^#{Regexp.escape(prefix.to_s)}/, '')
-            unless t_name.include?('__')
-              [[prefix, t_name].compact.join(''), t_name]
+            t_name = t_name.to_s.sub(/^#{Regexp.escape(prefix.to_s)}/, "")
+            unless t_name.include?("__")
+              [[prefix, t_name].compact.join(""), t_name]
             end
           end.compact
           if collections.empty? && templates.empty?
-            ui.error 'Failed to locate any templates!'
+            ui.error "Failed to locate any templates!"
             return nil
           end
-          ui.info "Please select an entry#{'(or collection to list)' unless collections.empty?}:"
+          ui.info "Please select an entry#{"(or collection to list)" unless collections.empty?}:"
           output = []
           idx = 1
           valid = {}
           unless collections.empty?
-            output << ui.color('Collections:', :bold)
+            output << ui.color("Collections:", :bold)
             collections.each do |full_name, part_name|
               valid[idx] = {:name => full_name, :type => :collection}
-              output << [idx, part_name.split('_').map(&:capitalize).join(' ')]
+              output << [idx, part_name.split("_").map(&:capitalize).join(" ")]
               idx += 1
             end
           end
           unless templates.empty?
-            output << ui.color('Templates:', :bold)
+            output << ui.color("Templates:", :bold)
             templates.each do |full_name, part_name|
               valid[idx] = {:name => full_name, :type => :template}
-              output << [idx, part_name.split('_').map(&:capitalize).join(' ')]
+              output << [idx, part_name.split("_").map(&:capitalize).join(" ")]
               idx += 1
             end
           end
           max = idx.to_s.length
           output.map! do |line|
             if line.is_a?(Array)
-              "  #{line.first}.#{' ' * (max - line.first.to_s.length)} #{line.last}"
+              "  #{line.first}.#{" " * (max - line.first.to_s.length)} #{line.last}"
             else
               line
             end
@@ -588,7 +588,7 @@ module Sfn
           ui.puts "#{output.join("\n")}\n"
           response = nil
           until valid[response]
-            response = ui.ask_question('Enter selection').to_i
+            response = ui.ask_question("Enter selection").to_i
           end
           entry = valid[response]
           if entry[:type] == :collection