batali 0.4.10 → 0.5.0

data/lib/batali/source/path.rb

@@ -1,6 +1,6 @@
-require 'batali'
-require 'fileutils'
-require 'tmpdir'
+require "batali"
+require "fileutils"
+require "tmpdir"
 
 module Batali
   # Source of asset
@@ -9,40 +9,44 @@ module Batali
     class Path < Source
 
       # @return [Array<String>] default ignore globs
-      DEFAULT_IGNORE = ['.git*']
+      DEFAULT_IGNORE = [".git*"]
       # @return [Array<String>] valid ignore file names
-      IGNORE_FILE = ['chefignore', '.chefignore']
+      IGNORE_FILE = ["chefignore", ".chefignore"]
 
       include Bogo::Memoization
 
       attribute :path, String, :required => true, :equivalent => true
 
+      def initialize(*_, &block)
+        super
+        self.path = Utility.clean_path(path)
+      end
+
       # @return [String] directory containing contents
       def asset
         memoize(:asset) do
           dir = Dir.mktmpdir
           chefignore = IGNORE_FILE.map do |c_name|
-            c_path = File.join(path, c_name)
+            c_path = Utility.join_path(path, c_name)
             c_path if File.exist?(c_path)
           end.compact.first
           chefignore = chefignore ? File.readlines(chefignore) : []
           chefignore += DEFAULT_IGNORE
           chefignore.uniq!
-          files_to_copy = Dir.glob(File.join(path, '{.[^.]*,**}', '**', '{*,*.*,.*}'))
+          files_to_copy = Dir.glob(File.join(path, "{.[^.]*,**}", "**", "{*,*.*,.*}"))
           files_to_copy = files_to_copy.map do |file_path|
             next unless File.file?(file_path)
-            relative_path = file_path.sub("#{path}#{File::SEPARATOR}", '')
-            relative_path unless chefignore.detect{|ig| File.fnmatch(ig, relative_path)}
+            relative_path = file_path.sub("#{path}#{File::SEPARATOR}", "")
+            relative_path unless chefignore.detect { |ig| File.fnmatch(ig, relative_path) }
           end.compact
           files_to_copy.each do |relative_path|
-            new_path = File.join(dir, relative_path)
+            new_path = Utility.join_path(dir, relative_path)
             FileUtils.mkdir_p(File.dirname(new_path))
-            FileUtils.cp(File.join(path, relative_path), new_path)
+            FileUtils.cp(Utility.join_path(path, relative_path), new_path)
           end
           dir
         end
       end
-
     end
   end
 end

data/lib/batali/source/site.rb

@@ -1,14 +1,13 @@
-require 'batali'
-require 'http'
-require 'tmpdir'
-require 'rubygems/package'
-require 'zlib'
+require "batali"
+require "http"
+require "tmpdir"
+require "rubygems/package"
+require "zlib"
 
 module Batali
   class Source
     # Site based source
     class Site < Source
-
       include Bogo::Memoization
 
       # @return [Array<Hash>] dependency strings
@@ -25,7 +24,7 @@ module Batali
       #
       # @param args [Hash]
       # @return [self]
-      def initialize(args={})
+      def initialize(args = {})
         @deps = args.delete(:dependencies) || {}
         super
       end
@@ -43,30 +42,30 @@ module Batali
       # @return [String] path to cache
       def cache_directory
         memoize(:cache_directory) do
-          @cache ||= File.join(cache_path, 'remote_site')
+          @cache ||= Utility.join_path(cache_path, "remote_site")
           cache
         end
       end
 
       # @return [String] directory
       def asset
-        path = File.join(cache_directory, Base64.urlsafe_encode64(url))
-        if(File.directory?(path))
-          discovered_path = Dir.glob(File.join(path, '*')).reject do |i|
+        path = Utility.join_path(cache_directory, Base64.urlsafe_encode64(url))
+        if File.directory?(path)
+          discovered_path = Dir.glob(Utility.join_path(path, "*")).reject do |i|
             i.end_with?("#{File::SEPARATOR}asset")
           end.first
           FileUtils.rm_rf(path)
         end
-        unless(discovered_path)
+        unless discovered_path
           retried = false
           begin
             FileUtils.mkdir_p(path)
-            result = HTTP.get(url)
-            while(result.code == 302)
-              result = HTTP.get(result.headers['Location'])
+            result = HTTP.get(url.end_with?("/") ? url : url + "/")
+            while result.code == 302
+              result = HTTP.get(result.headers["Location"])
             end
-            File.open(a_path = File.join(path, 'asset'), 'wb') do |file|
-              while(content = result.body.readpartial(2048))
+            File.open(a_path = Utility.join_path(path, "asset"), "wb") do |file|
+              while content = result.body.readpartial(2048)
                 file.write content
               end
             end
@@ -76,10 +75,10 @@ module Batali
             ext.rewind
             ext.each do |entry|
               next unless entry.file?
-              n_path = File.join(path, entry.full_name)
+              n_path = Utility.join_path(path, entry.full_name)
               FileUtils.mkdir_p(File.dirname(n_path))
-              File.open(n_path, 'wb') do |file|
-                while(content = entry.read(2048))
+              File.open(n_path, "wb") do |file|
+                while content = entry.read(2048)
                   file.write(content)
                 end
               end
@@ -91,18 +90,18 @@ module Batali
             end
           rescue => e
             FileUtils.rm_rf(path)
-            unless(retried)
+            unless retried
               FileUtils.mkdir_p(path)
               retried = true
               retry
             end
             raise
           end
-          discovered_path = Dir.glob(File.join(path, '*')).reject do |i|
+          discovered_path = Dir.glob(Utility.join_path(path, "*")).reject do |i|
             i.end_with?("#{File::SEPARATOR}asset")
           end.first
         end
-        unless(discovered_path)
+        unless discovered_path
           raise Errno::ENOENT.new "Failed to locate asset within `#{path}`"
         end
         discovered_path
@@ -110,13 +109,12 @@ module Batali
 
       # @return [TrueClass, FalseClass]
       def clean_asset(asset_path)
-        if(asset_path)
+        if asset_path
           super File.dirname(asset_path)
         else
           false
         end
       end
-
     end
   end
 end

data/lib/batali/tag_lines.rb

@@ -1,4 +1,4 @@
-require 'batali'
+require "batali"
 
 # Batali namespace
 module Batali
@@ -7,9 +7,8 @@ module Batali
   TAG_LINES = [
     'I got 99 problems but a policyfile ain\'t one',
     'Solve the way you want, not the way you\'re told',
-    'Build environments, not applications',
-    'DRY should be the rule, not the exception',
-    'Solve for the forest, not just a tree'
+    "Build environments, not applications",
+    "DRY should be the rule, not the exception",
+    "Solve for the forest, not just a tree",
   ]
-
 end

data/lib/batali/unit.rb

@@ -1,23 +1,21 @@
-require 'batali'
+require "batali"
 
 # Batali namespace
 module Batali
   # Customized Unit
   class Unit < Grimoire::Unit
-    attribute :source, Source, :coerce => lambda{|v| Batali::Source.build(v)}
+    attribute :source, Source, :coerce => lambda { |v| Batali::Source.build(v) }
     attribute(:dependencies, [Batali::UnitDependency, Grimoire::DEPENDENCY_CLASS],
-      :multiple => true,
-      :default => [],
-      :coerce => lambda{ |val|
-        Batali::UnitDependency.new(val.first, *val.last)
-      }
-    )
+              :multiple => true,
+              :default => [],
+              :coerce => lambda { |val|
+                Batali::UnitDependency.new(val.first, *val.last)
+              })
     attribute(:version, [Batali::UnitVersion, Grimoire::VERSION_CLASS],
-      :required => true,
-      :coerce => lambda{ |val|
-        Batali::UnitVersion.new(val)
-      }
-    )
+              :required => true,
+              :coerce => lambda { |val|
+                Batali::UnitVersion.new(val)
+              })
 
     # @return [TrueClass, FalseClass]
     def diff?(u)
@@ -33,15 +31,14 @@ module Batali
     def diff(u)
       Smash.new.tap do |_diff|
         [:name, :version].each do |k|
-          unless(send(k) == u.send(k))
+          unless send(k) == u.send(k)
             _diff[k] = [send(k), u.send(k)]
           end
         end
-        if(source)
+        if source
           _diff.merge!(source.diff(u.source))
         end
       end
     end
-
   end
 end

data/lib/batali/unit_loader.rb

@@ -1,11 +1,10 @@
-require 'batali'
+require "batali"
 
 # Batali namespace
 module Batali
 
   # Load cookbook units
   class UnitLoader < Utility
-
     include Bogo::Memoization
 
     attribute :file, BFile, :required => true
@@ -20,7 +19,7 @@ module Batali
       memoize(:populate) do
         (file.source + file.chef_server).each do |src|
           src.units.find_all do |unit|
-            if(restrictions[unit.name])
+            if restrictions[unit.name]
               restrictions[unit.name] == src.identifier
             else
               true
@@ -30,22 +29,22 @@ module Batali
           end
         end
         file.cookbook.each do |ckbk|
-          if(ckbk.git)
+          if ckbk.git
             source = Origin::Git.new(
               :name => ckbk.name,
               :url => ckbk.git,
               :subdirectory => ckbk.path,
-              :ref => ckbk.ref || 'master',
-              :cache_path => cache
+              :ref => ckbk.ref || "master",
+              :cache_path => cache,
             )
-          elsif(ckbk.path)
+          elsif ckbk.path
             source = Origin::Path.new(
               :name => ckbk.name,
               :path => ckbk.path,
-              :cache_path => cache
+              :cache_path => cache,
             )
           end
-          if(source)
+          if source
             system.add_unit(source.units.first)
           end
         end
@@ -57,13 +56,13 @@ module Batali
       memoize(:restrictions) do
         rest = (file.restrict || Smash.new).to_smash
         file.cookbook.each do |ckbk|
-          if(auto_path_restrict || ckbk.restrict)
-            if(ckbk.path)
+          if auto_path_restrict || ckbk.restrict
+            if ckbk.path
               rest[ckbk.name] = Smash.new(:path => ckbk.path).checksum
-            elsif(ckbk.git)
+            elsif ckbk.git
               rest[ckbk.name] = Smash.new(
                 :url => ckbk.git,
-                :ref => ckbk.ref
+                :ref => ckbk.ref,
               ).checksum
             end
           end
@@ -71,7 +70,5 @@ module Batali
         rest
       end
     end
-
   end
-
 end

data/lib/batali/utility.rb

@@ -1,8 +1,33 @@
-require 'batali'
+require "batali"
 
 module Batali
+  # Utility class to provide helper methods
   class Utility < Grimoire::Utility
 
+    # Prefix for building UNC paths on Windows
+    UNC_PREFIX = "//?/".freeze
+
+    # Properly format and expand path based
+    # on platform in use
+    def self.clean_path(path)
+      path = File.expand_path(path.to_s)
+      if RUBY_PLATFORM =~ /mswin|mingw|windows/ &&
+         path.downcase.match(/^[a-z]:/) &&
+         ENV["BATALI_DISABLE_UNC"].nil?
+        path = UNC_PREFIX + path
+      end
+      path
+    end
+
+    # Join arguments to base path and clean
+    #
+    # @param base [String] base path
+    # @param args [Array<String>]
+    # @return [String]
+    def self.join_path(base, *args)
+      clean_path(File.join(base, *args))
+    end
+
     # Helper module for enabling chef server support
     module Chef
 
@@ -19,28 +44,33 @@ module Batali
 
       # Load and configure chef
       def init_chef!
-        debug 'Loading chef into the runtime'
+        debug "Loading chef into the runtime"
         begin
-          require 'chef'
-          require 'chef/rest'
-          debug 'Successfully loaded chef into the runtime'
+          require "chef"
+          begin
+            require "chef/rest"
+            @_api_klass = ::Chef::REST
+          rescue LoadError
+            # Newer versions of chef do not include REST
+            require "chef/server_api"
+            @_api_klass = ::Chef::ServerAPI
+          end
+          debug "Successfully loaded chef into the runtime"
         rescue LoadError => e
           debug "Failed to load the chef gem: #{e.class}: #{e}"
-          raise 'The `chef` gem was not found. Please `gem install chef` or add `chef` to your bundle.'
+          raise "The `chef` gem was not found. Please `gem install chef` or add `chef` to your bundle."
         end
         Smash.new(
           :endpoint => :chef_server_url,
           :c_name => :node_name,
-          :c_key => :client_key
+          :c_key => :client_key,
         ).each do |local_attr, config_key|
-          unless(self.send(local_attr)) # rubocop:disable Style/RedundantSelf
+          unless self.send(local_attr)
             memoize(:knife_configure, :global) do
-              require 'chef/knife'
+              require "chef/knife"
               ::Chef::Knife.new.configure_chef
             end
-            # rubocop:disable Style/RedundantSelf
             debug "Settting #{config_key} from knife configuration file for #{self.class} <#{endpoint}>"
-            # rubocop:disable Style/RedundantSelf
             self.send("#{local_attr}=", ::Chef::Config[config_key])
           end
         end
@@ -54,15 +84,13 @@ module Batali
       # @return [Object] result
       def api_service
         memoize(:api_service) do
-          ::Chef::REST.new(
+          @_api_klass.new(
             endpoint,
             c_name,
             c_key
           )
         end
       end
-
     end
-
   end
 end

data/lib/batali/version.rb

@@ -1,5 +1,5 @@
 # Batali namespace
 module Batali
   # Current version
-  VERSION = Gem::Version.new('0.4.10')
+  VERSION = Gem::Version.new("0.5.0")
 end

data/lib/chef/knife/batali_sync.rb

@@ -1,92 +1,87 @@
-require 'batali'
+require "batali"
 
 class Chef
   class Knife
     # Batali manifest to chef server sync
     class BataliSync < Knife
-
-      banner 'knife batali sync'
+      banner "knife batali sync"
 
       option(:blacklist,
-        :short => '-B COOKBOOK_NAME[,COOKBOOK_NAME]',
-        :long => '--blacklist COOKBOOK_NAME[,COOKBOOK_NAME]',
-        :description => 'Cookbooks to ignore from sync',
-        :proc => lambda{|val|
-          Chef::Config[:knife][:batali_blacklist] ||= []
-          Chef::Config[:knife][:batali_blacklist] += val.split(',')
-        }
-      )
+             :short => "-B COOKBOOK_NAME[,COOKBOOK_NAME]",
+             :long => "--blacklist COOKBOOK_NAME[,COOKBOOK_NAME]",
+             :description => "Cookbooks to ignore from sync",
+             :proc => lambda { |val|
+               Chef::Config[:knife][:batali_blacklist] ||= []
+               Chef::Config[:knife][:batali_blacklist] += val.split(",")
+             })
 
       option(:details,
-        :long => '--[no-]details',
-        :boolean => true,
-        :default => true,
-        :description => 'Show details of cookbooks to be removed / added'
-      )
+             :long => "--[no-]details",
+             :boolean => true,
+             :default => true,
+             :description => "Show details of cookbooks to be removed / added")
 
       option(:show_remaining,
-        :long => '--[no-]show-remaining',
-        :description => 'Display cookbook details of expected final server state',
-        :boolean => true,
-        :default => false,
-        :proc => lambda{|val|
-          Chef::Config[:knife][:batali_show_remaining] = val
-        }
-      )
+             :long => "--[no-]show-remaining",
+             :description => "Display cookbook details of expected final server state",
+             :boolean => true,
+             :default => false,
+             :proc => lambda { |val|
+               Chef::Config[:knife][:batali_show_remaining] = val
+             })
 
       option(:dry_run,
-        :long => '--[no-]dry-run',
-        :description => 'Display information but perform no action',
-        :boolean => true,
-        :default => false
-      )
+             :long => "--[no-]dry-run",
+             :description => "Display information but perform no action",
+             :boolean => true,
+             :default => false)
 
       def run
         Chef::Config[:knife][:batali_blacklist] ||= []
         config[:verbose] = config[:verbosity].to_i > 0
-        ui.info "#{ui.color('[Batali]', :green, :bold)}: Chef Server Batali Manifest Sync"
-        valid_cookbooks = run_task('Generating valid cookbook versions from manifest') do
+        ui.info "#{ui.color("[Batali]", :green, :bold)}: Chef Server Batali Manifest Sync"
+        valid_cookbooks = run_task("Generating valid cookbook versions from manifest") do
           generate_manifest_cookbooks
         end
-        remote_cookbooks = run_task('Generating remote cookbook versions from chef server') do
+        remote_cookbooks = run_task("Generating remote cookbook versions from chef server") do
           generate_remote_cookbooks
         end
-        to_remove = run_task('Building cookbook removal list') do
+        to_remove = run_task("Building cookbook removal list") do
           locate_removals(
             :manifest => valid_cookbooks,
-            :server => remote_cookbooks
+            :server => remote_cookbooks,
           )
         end
-        to_add = run_task('Building cookbook upload list') do
+        to_add = run_task("Building cookbook upload list") do
           locate_additions(
             :manifest => valid_cookbooks,
-            :server => remote_cookbooks
+            :server => remote_cookbooks,
           )
         end
-        if(to_add.empty? && to_remove.empty?)
-          ui.info "#{ui.color('[Batali]', :green, :bold)}: Chef Server Batali Manifest Sync - #{ui.color('No Changes Detected!', :green, :bold)}" # rubocop:disable Metrics/LineLength
+        if to_add.empty? && to_remove.empty?
+          ui.info "#{ui.color("[Batali]", :green, :bold)}: Chef Server Batali Manifest Sync - #{ui.color("No Changes Detected!", :green, :bold)}" # rubocop:disable Metrics/LineLength
         else
           display_sync_info(
             :additions => to_add,
             :removals => to_remove,
-            :manifest => valid_cookbooks
+            :manifest => valid_cookbooks,
           )
-          unless(config[:dry_run])
-            ui.confirm 'Sync remote cookbooks with Batali manifest'
+          unless config[:dry_run]
+            ui.confirm "Sync remote cookbooks with Batali manifest"
             remove_cookbooks(to_remove) unless to_remove.empty?
             add_cookbooks(to_add) unless to_add.empty?
-            ui.info "#{ui.color('[Batali]', :green, :bold)}: Chef Server Batali Manifest Sync - #{ui.color('Sync Complete!', :green, :bold)}" # rubocop:disable Metrics/LineLength
+            ui.info "#{ui.color("[Batali]", :green, :bold)}: Chef Server Batali Manifest Sync - #{ui.color("Sync Complete!", :green, :bold)}" # rubocop:disable Metrics/LineLength
           else
-            ui.warn 'Dry run requested. No action taken.'
+            ui.warn "Dry run requested. No action taken."
           end
         end
       end
 
       def remove_cookbooks(ckbks)
-        run_task('Removing cookbooks') do
+        run_task("Removing cookbooks") do
           ckbks.each do |c_name, vers|
             vers.each do |version|
-              if(config[:verbose])
+              if config[:verbose]
                 ui.warn "Deleting cookbook #{c_name} @ #{version}"
               end
               rest.delete("/cookbooks/#{c_name}/#{version}")
@@ -97,13 +92,13 @@ class Chef
 
       def add_cookbooks(ckbks)
         Batali::Command::Install.new({}, []).execute!
-        ui.info "#{ui.color('[Batali]', :green, :bold)}: Adding cookbooks to Chef server."
+        ui.info "#{ui.color("[Batali]", :green, :bold)}: Adding cookbooks to Chef server."
         Knife::Upload.load_deps
         ckbks.each do |c_name, vers|
           vers.each do |version|
             c_path = [
-              File.join('cookbooks', c_name),
-              File.join('cookbooks', "#{c_name}-#{version}")
+              File.join("cookbooks", c_name),
+              File.join("cookbooks", "#{c_name}-#{version}"),
             ].detect do |_path|
               File.directory?(_path)
             end
@@ -111,44 +106,44 @@ class Chef
             uploader.configure_chef
             uploader.config = config
             uploader.name_args = [c_path]
-            if(config[:verbose])
+            if config[:verbose]
               ui.warn "Unloading cookbook #{c_name} @ #{version} - `#{c_path}`"
             end
             uploader.run
           end
         end
-        ui.info "#{ui.color('[Batali]', :green, :bold)}: Chef server cookbook additions complete."
+        ui.info "#{ui.color("[Batali]", :green, :bold)}: Chef server cookbook additions complete."
       end
 
       def display_sync_info(opts)
         num_remove = ui.color(opts[:removals].size.to_s, :red, :bold)
         num_add = ui.color(opts[:additions].size.to_s, :green, :bold)
-        ui.info "#{ui.color('[Batali]', :green, :bold)}: Removals - #{num_remove} Additions: #{num_add}"
-        if(config[:details])
-          unless(opts[:removals].empty?)
-            ui.info "#{ui.color('[Batali]', :green, :bold)}: Cookbooks to be #{ui.color('removed', :red, :bold)}:"
+        ui.info "#{ui.color("[Batali]", :green, :bold)}: Removals - #{num_remove} Additions: #{num_add}"
+        if config[:details]
+          unless opts[:removals].empty?
+            ui.info "#{ui.color("[Batali]", :green, :bold)}: Cookbooks to be #{ui.color("removed", :red, :bold)}:"
             opts[:removals].sort.each do |name, versions|
               vers = versions.map do |v|
                 Gem::Version.new(v)
-              end.sort.map(&:to_s).join(', ')
+              end.sort.map(&:to_s).join(", ")
               ui.info "  #{ui.color(name, :red, :bold)}: #{ui.color(vers, :red)}"
             end
           end
-          unless(opts[:additions].empty?)
-            ui.info "#{ui.color('[Batali]', :green, :bold)}: Cookbooks to be #{ui.color('added', :green, :bold)}:"
+          unless opts[:additions].empty?
+            ui.info "#{ui.color("[Batali]", :green, :bold)}: Cookbooks to be #{ui.color("added", :green, :bold)}:"
             opts[:additions].sort.each do |name, versions|
               vers = versions.map do |v|
                 Gem::Version.new(v)
-              end.sort.map(&:to_s).join(', ')
+              end.sort.map(&:to_s).join(", ")
               ui.info "  #{ui.color(name, :green, :bold)}: #{ui.color(vers, :green)}"
             end
           end
-          if(Chef::Config[:knife][:batali_show_remaining])
-            ui.info "#{ui.color('[Batali]', :green, :bold)}: Final list of cookbooks to be available on the chef server:" # rubocop:disable Metrics/LineLength
+          if Chef::Config[:knife][:batali_show_remaining]
+            ui.info "#{ui.color("[Batali]", :green, :bold)}: Final list of cookbooks to be available on the chef server:" # rubocop:disable Metrics/LineLength
             opts[:manifest].sort.each do |name, versions|
               vers = versions.map do |v|
                 Gem::Version.new(v)
-              end.sort.map(&:to_s).join(', ')
+              end.sort.map(&:to_s).join(", ")
               ui.info "  #{ui.color(name, :bold)}: #{vers}"
             end
           end
@@ -159,7 +154,7 @@ class Chef
         Smash.new.tap do |rm|
           opts[:server].each do |c_name, c_versions|
             kills = c_versions - opts[:manifest].fetch(c_name, [])
-            unless(kills.empty?)
+            unless kills.empty?
               rm[c_name] = kills
             end
           end
@@ -170,7 +165,7 @@ class Chef
         Smash.new.tap do |add|
           opts[:manifest].each do |c_name, c_versions|
             adds = c_versions - opts[:server].fetch(c_name, [])
-            unless(adds.empty?)
+            unless adds.empty?
               add[c_name] = adds
             end
           end
@@ -178,8 +173,8 @@ class Chef
       end
 
       def generate_manifest_cookbooks
-        path = File.join(Dir.pwd, 'batali.manifest')
-        unless(File.exist?(path))
+        path = File.join(Dir.pwd, "batali.manifest")
+        unless File.exist?(path)
           raise "Failed to locate batali manifest at: #{path}"
         end
         manifest = Batali::Manifest.build(path)
@@ -194,29 +189,28 @@ class Chef
 
       def generate_remote_cookbooks
         Smash.new.tap do |ckbks|
-          rest.get('cookbooks?num_versions=all').map do |c_name, meta|
+          rest.get("cookbooks?num_versions=all").map do |c_name, meta|
             next if Chef::Config[:knife][:batali_blacklist].include?(c_name)
             ckbks[c_name] = []
-            meta['versions'].each do |info|
-              ckbks[c_name] << info['version']
+            meta["versions"].each do |info|
+              ckbks[c_name] << info["version"]
             end
           end
         end
       end
 
       def run_task(task)
-        ui.stdout.print "#{ui.color('[Batali]', :green, :bold)}: #{task}... "
+        ui.stdout.print "#{ui.color("[Batali]", :green, :bold)}: #{task}... "
         begin
           value = yield if block_given?
-          ui.info ui.color('complete', :green)
+          ui.info ui.color("complete", :green)
           value
         rescue => e
-          ui.info ui.color('failed', :red, :bold)
+          ui.info ui.color("failed", :red, :bold)
           puts e.backtrace.join("\n")
           raise e
         end
       end
-
     end
   end
 end