rubycut-babushka 0.10.8 → 0.15.6.1

data/spec/babushka/accepts_for_support.rb

@@ -1,23 +1,30 @@
 class AcceptsForTest
   include AcceptsListFor
   include AcceptsValueFor
+
   attr_reader :payload
+
   def initialize name = nil
     @name = name
     @payload = {}
   end
+
   def chooser
-    :macports
+    :brew
   end
+
   def chooser_choices
-    [:apt, :macports]
+    [:apt, :brew]
   end
+
   def default_formats
     %w[html xml js json]
   end
+
   def default_format
     "json"
   end
+
   accepts_value_for :package, :choose_with => :via
   accepts_value_for :renders, "a default response", :choose_with => :via
   accepts_value_for :format, :default_format, :choose_with => :via
@@ -44,11 +51,11 @@ def test_value_lambdas
       via :apt, "git-core"
     } => nil,
     L{
-      via :macports, 'ruby'
+      via :brew, 'ruby'
       via :apt, 'git-core'
     } => 'ruby',
     L{
-      via :macports, 'something else'
+      via :brew, 'something else'
       via :apt, 'some apt packages'
     } => 'something else'
   }
@@ -61,11 +68,11 @@ def test_list_lambdas
       via :apt, %w[ruby irb ri rdoc]
     } => [],
     L{
-      via :macports, 'ruby'
+      via :brew, 'ruby'
       via :apt, %w[ruby irb ri rdoc]
     } => ['ruby'],
     L{
-      via :macports, 'something else'
+      via :brew, 'something else'
       via :apt, 'some apt packages'
     } => ['something else']
   }

data/spec/babushka/asset_spec.rb

@@ -0,0 +1,165 @@
+require 'spec_helper'
+
+def archive_path
+  __FILE__.p.parent.parent / 'archives'
+end
+
+describe Babushka::Asset do
+  it "should detect file types" do
+    Asset.type(archive_path / 'archive.zip').should == :zip
+    Asset.type(archive_path / 'archive.tgz').should == :gzip
+  end
+  it "should first attempt to detect type using file extension" do
+    Asset.type(archive_path / 'really_a_gzip.zip').should == :zip
+  end
+  it "should attempt to detect type via `file` when there is no extension" do
+    Asset.should_receive(:shell).with("file '#{archive_path / 'zip_without_extension'}'").any_number_of_times.and_return('Zip archive data')
+    Asset.type(archive_path / 'zip_without_extension').should == :zip
+  end
+  it "should detect supported archive types" do
+    Asset.for(archive_path / 'archive.tgz').should be_supported
+    Asset.for(archive_path / 'archive.tbz2').should be_supported
+  end
+  it "should raise an error on unsupported types" do
+    Asset.should_receive(:shell).with("file '#{archive_path / 'invalid_archive'}'").any_number_of_times.and_return('ASCII text')
+    L{
+      Asset.for(archive_path / 'invalid_archive')
+    }.should raise_error("Don't know how to extract invalid_archive.")
+  end
+  it "should set the name" do
+    Asset.for(archive_path / 'archive.tar').name.should == 'archive'
+    Asset.for(archive_path / 'archive.tar.gz').name.should == 'archive'
+  end
+  it "should generate the proper command to extract the archive" do
+    {
+      'tar' => "tar -xf '#{archive_path / 'archive.tar'}'",
+      'tgz' => "tar -zxf '#{archive_path / 'archive.tgz'}'",
+      'tbz2' => "tar -jxf '#{archive_path / 'archive.tbz2'}'",
+      'zip' => "unzip -o '#{archive_path / 'archive.zip'}'"
+    }.each_pair {|ext,command|
+      Asset.for(archive_path / "archive.#{ext}").extract_command.should == command
+    }
+  end
+  it "should yield" do
+    yielded = false
+    Asset.for(archive_path / "archive.tar").extract {
+      yielded = true
+    }
+    yielded.should be_true
+  end
+  it "should yield in the extracted dir" do
+    Asset.for(archive_path / "archive.tar").extract {
+      Dir.pwd.should == (tmp_prefix / 'archives/archive')
+    }
+  end
+  it "should yield in the nested dir if there is one" do
+    Asset.for(archive_path / "nested_archive.tar").extract {
+      Dir.pwd.should == (tmp_prefix / 'archives/nested_archive/nested archive')
+    }
+  end
+  it "should find a standard content dir as a nested dir" do
+    Asset.for(archive_path / "test-0.3.1.tgz").extract {
+      Dir.pwd.should == (tmp_prefix / 'archives/test-0.3.1/test-0.3.1')
+      Dir.glob('*').should == ['content.txt']
+    }
+  end
+  it "shouldn't descend into some dirs" do
+    Asset.for(archive_path / "Blah.app.zip").extract {
+      Dir.pwd.should == (tmp_prefix / 'archives/Blah.app')
+      Dir.glob('**/*').should == ['Blah.app', 'Blah.app/content.txt']
+    }
+  end
+
+  describe 'cleanup' do
+    it "should remove the build dir on success" do
+      Asset.for(archive_path / "archive.tar").extract {
+        (tmp_prefix / 'archives/archive').exists?.should be_true
+        true
+      }
+      (tmp_prefix / 'archives/archive').exists?.should be_false
+    end
+    it "should not remove the build dir on failure" do
+      Asset.for(archive_path / "archive.tar").extract {
+        (tmp_prefix / 'archives/archive').exists?.should be_true
+        false
+      }
+      (tmp_prefix / 'archives/archive').exists?.should be_true
+    end
+  end
+
+  describe '#content_subdir' do
+    let(:resource) { Asset.new('test.zip') }
+
+    context "when there is just a single file inside the archive" do
+      before {
+        Dir.stub!(:glob).and_return(['a dir'])
+        File.should_receive(:directory?).with('a dir').and_return(false)
+      }
+      it "should choose it, whatever it's called" do
+        resource.content_subdir.should be_nil
+      end
+    end
+    context "when there is just a single non-descendable dir inside the archive" do
+      before {
+        Dir.stub!(:glob).and_return(['a dir.app'])
+        File.should_receive(:directory?).with('a dir.app').and_return(true)
+      }
+      it "should choose it, whatever it's called" do
+        resource.content_subdir.should be_nil
+      end
+    end
+    context "when there is just a single dir inside the archive" do
+      before {
+        Dir.stub!(:glob).and_return(['a dir'])
+        File.should_receive(:directory?).with('a dir').and_return(true)
+      }
+      it "should choose it, whatever it's called" do
+        resource.content_subdir.should == 'a dir'
+      end
+    end
+    context "when there is more than one dir" do
+      context "and none are named after the archive" do
+        before {
+          Dir.stub!(:glob).and_return(['contents', 'another'])
+        }
+        it "should return nil (so the original extraction dir is used)" do
+          resource.content_subdir.should be_nil
+        end
+      end
+      context "and one is named after the archive" do
+        before {
+          Dir.stub!(:glob).and_return(['contents', 'test'])
+        }
+        it "should choose the directory named after the archive" do
+          resource.content_subdir.should == 'test'
+        end
+      end
+    end
+    context "when there are non-descendable dirs" do
+      context "and none are named after the archive" do
+        before {
+          Dir.stub!(:glob).and_return(['contents', 'LaunchBar.app', 'RSpec.tmbundle'])
+        }
+        it "should not choose the non-descendable dir" do
+          resource.content_subdir.should be_nil
+        end
+      end
+      context "and one is named after the archive" do
+        before {
+          Dir.stub!(:glob).and_return(['contents', 'test.app'])
+        }
+        it "should not choose the non-descendable dir" do
+          resource.content_subdir.should be_nil
+        end
+      end
+      context "one is named after the archive, and a descendable dir is present too" do
+        before {
+          Dir.stub!(:glob).and_return(['contents', 'test.app', 'test'])
+        }
+        it "should choose the descendable dir" do
+          resource.content_subdir.should == 'test'
+        end
+      end
+    end
+  end
+end

data/spec/babushka/cmdline/help_spec.rb

@@ -4,7 +4,7 @@ describe "help" do
   context "with no verb" do
     before {
       [
-        "Babushka v#{Babushka::VERSION}, (c) 2011 Ben Hoskings <ben@hoskings.net>",
+        "Babushka v#{Babushka::VERSION} (#{Babushka::Base.ref}), (c) 2012 Ben Hoskings <ben@hoskings.net>",
         "\nThe gist:",
         "  #{Base.program_name} <command> [options]",
         "\nAlso:",
@@ -22,10 +22,10 @@ describe "help" do
         "  edit       Load the file containing the specified dep in $EDITOR",
         "\nCommands can be abbrev'ed, as long as they remain unique.",
         "  e.g. '#{Base.program_name} l' is short for '#{Base.program_name} list'.",
-        "\n"
       ].each {|line|
         Cmdline::Helpers.should_receive(:log).with(line)
       }
+      Cmdline.should_receive(:log).with("\n")
     }
     it "should print the verb help information" do
       Cmdline::Parser.for(%w[help]).run
@@ -34,25 +34,27 @@ describe "help" do
   context "with a verb" do
     let(:parser) { Cmdline::Parser.for(%w[help meet]) }
     before {
-      [
-        "Babushka v#{Babushka::VERSION}, (c) 2011 Ben Hoskings <ben@hoskings.net>",
+      Cmdline::Helpers.should_receive(:log).with(
+        "Babushka v#{Babushka::VERSION} (#{Babushka::Base.ref}), (c) 2012 Ben Hoskings <ben@hoskings.net>"
+      )
+      Cmdline.should_receive(:log).with(
         "\nmeet - The main one: run a dep and all its dependencies."
-      ].each {|line|
-        Cmdline::Helpers.should_receive(:log).with(line)
-      }
+      )
 
       parser.should_receive(:log).with("
     -v, --version                    Print the current version
     -h, --help                       Show this information
     -d, --debug                      Show more verbose logging, and realtime shell command output
+    -s, --silent                     Only log errors, running silently on success
         --[no-]color, --[no-]colour  Disable color in the output
     -n, --dry-run                    Discover the curent state without making any changes
     -y, --defaults                   Assume the default value for all vars without prompting, where possible
+    -u, --update                     Update referenced sources before loading deps from them
         --show-args                  Show the arguments being passed between deps as they're run
-        --track-blocks               Track deps' blocks in TextMate as they're run
+        --profile                    Print a per-line timestamp to the debug log.
 ")
 
-      Cmdline::Helpers.should_receive(:log).with("\n")
+      Cmdline.should_receive(:log).with("\n")
     }
     it "should print the help information for the verb" do
       parser.run

data/spec/babushka/cmdline/meet_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe Cmdline, 'meet' do
+  describe "var parsing" do
+    before {
+      Base.task.should_receive(:process).with(
+        %w[git],
+        {'version' => '1.7.7', 'source' => 'http://git.org/git.tgz'}
+      )
+    }
+    it "should recognise vars" do
+      Cmdline::Parser.for(%w[git version=1.7.7 source=http://git.org/git.tgz]).run
+    end
+  end
+end

data/spec/babushka/cmdline/version_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 describe "version" do
   before {
-    Cmdline::Helpers.should_receive(:log).with(Babushka::VERSION)
+    Cmdline::Helpers.should_receive(:log).with("#{Babushka::VERSION} (#{Babushka::Base.ref})")
   }
   it "should print the version" do
     Cmdline::Parser.for(%w[version]).run

data/spec/babushka/core_patches_spec.rb

@@ -169,3 +169,12 @@ describe String, "val_for" do
     "/dev/disk1s2        	Apple_HFS                      	/Volumes/TextMate 1.5.9".val_for(/^\/dev\/disk\d+s\d+\s+Apple_HFS\s+/).should == "/Volumes/TextMate 1.5.9"
   end
 end
+
+describe String, '#colorized?' do
+  it "should return false for regular strings" do
+    "babushka".should_not be_colorized
+  end
+  it "should return true for strings containing ANSI color sequences" do
+    "\e[0;32;29mbabushka\e[0m".should be_colorized
+  end
+end

data/spec/babushka/current_ruby_spec.rb

@@ -0,0 +1,73 @@
+require 'spec_helper'
+
+describe Babushka::CurrentRuby do
+  let(:current_ruby) { Babushka::CurrentRuby.new }
+  before {
+    current_ruby.stub!(:gem_env).and_return(%{
+RubyGems Environment:
+  - RUBYGEMS VERSION: 1.8.23
+  - INSTALLATION DIRECTORY: /usr/local/lib/ruby/gems/1.9.1
+  - EXECUTABLE DIRECTORY: /usr/local/bin
+      })
+  }
+
+  describe '#path' do
+    it "should return the path to the ruby binary" do
+      Babushka::ShellHelpers.stub!(:which).with('ruby').and_return('/usr/local/bin/ruby')
+      current_ruby.path.should == '/usr/local/bin/ruby'
+    end
+  end
+
+  describe '#rbenv?' do
+    it "should return true when ruby is running via rbenv" do
+      Babushka::ShellHelpers.stub!(:which).with('ruby').and_return('/Users/steve/.rbenv/shims/ruby')
+      current_ruby.rbenv?.should be_true
+    end
+    it "should return false otherwise" do
+      Babushka::ShellHelpers.stub!(:which).with('ruby').and_return('/usr/local/bin/ruby')
+      current_ruby.rbenv?.should be_false
+    end
+  end
+
+  describe '#rvm?' do
+    it "should return true when ruby is running via rvm" do
+      Babushka::ShellHelpers.stub!(:which).with('ruby').and_return('/Users/steve/.rvm/rubies/ruby-1.9.3-p194/bin/ruby')
+      current_ruby.rvm?.should be_true
+    end
+    it "should return false otherwise" do
+      Babushka::ShellHelpers.stub!(:which).with('ruby').and_return('/usr/local/bin/ruby')
+      current_ruby.rvm?.should be_false
+    end
+  end
+
+  describe '#bin_dir' do
+    it "should return the path containing the ruby binary" do
+      current_ruby.bin_dir.should == '/usr/local/bin'
+    end
+  end
+
+  describe '#gem_dir' do
+    it "should return the directory containing installed gems" do
+      current_ruby.gem_dir.should == '/usr/local/lib/ruby/gems/1.9.1/gems'
+    end
+  end
+
+  describe '#gemspec_dir' do
+    it "should return the directory containing installed gems' specs" do
+      current_ruby.gemspec_dir.should == '/usr/local/lib/ruby/gems/1.9.1/specifications'
+    end
+  end
+
+  describe '#version' do
+    it "should return the version of the ruby installation" do
+      Babushka::ShellHelpers.stub!(:shell).with('ruby --version').and_return('ruby 1.9.3p194 (2012-04-20 revision 35410) [x86_64-linux]')
+      current_ruby.version.should == Babushka::VersionStr.new('1.9.3p194')
+    end
+  end
+
+  describe '#gem_version' do
+    it "should return the version of the ruby installation's rubygems" do
+      current_ruby.gem_version.should == Babushka::VersionStr.new('1.8.23')
+    end
+  end
+end

data/spec/babushka/dep_context_spec.rb

@@ -1,42 +1,51 @@
 require 'spec_helper'
-require 'dep_definer_support'
 
 describe "accepts_block_for behaviour" do
+  let(:lambda_hello) { L{ "hello world!" } }
+
+  def test_accepts_block_for_response accepter_name, lambda, value, opts = {}
+    DepContext.accepts_block_for accepter_name
+    dep 'accepts_block_for' do
+      send accepter_name, opts, &lambda
+    end
+    on = opts[:on].nil? ? :all : Babushka.host.system
+    Dep('accepts_block_for').context.define!.payload[accepter_name][on].should == value
+  end
+
   before {
-    Dep.stub!(:base_template).and_return(TestTemplate)
-    setup_test_lambdas
+    Babushka.host.stub!(:match_list).and_return([:osx])
     dep 'default'
   }
 
   it "should define a declarer" do
     Dep('default').context.should_not respond_to(:test_defining)
-    TestDepContext.accepts_block_for :test_defining
+    DepContext.accepts_block_for :test_defining
     Dep('default').context.should respond_to(:test_defining)
   end
 
   it "should return lambda" do
-    TestDepContext.accepts_block_for :test_defining
+    DepContext.accepts_block_for :test_defining
     lambda = L{ 'blah' }
     value_from_block = nil
     dep 'returning test' do
-      value_from_block = test_defining &lambda
+      value_from_block = test_defining(&lambda)
     end.met?
     value_from_block.should == lambda
   end
 
   it "should accept and return a block" do
-    test_accepts_block_for_response :test_response, @lambda_hello, @lambda_hello
+    test_accepts_block_for_response :test_response, lambda_hello, lambda_hello
   end
   it "should accept and return a block for this system" do
-    test_accepts_block_for_response :test_this_system, @lambda_hello, @lambda_hello, :on => Base.host.system
+    test_accepts_block_for_response :test_this_system, lambda_hello, lambda_hello, :on => Babushka.host.system
   end
   it "should return nothing on a non-specified system" do
-    test_accepts_block_for_response :test_other_system, @lambda_hello, nil, :on => :nonexistent
+    test_accepts_block_for_response :test_other_system, lambda_hello, nil, :on => :missing
   end
 
   it "should use default blocks when no specific one is specified" do
     lambda = L{ 'default value' }
-    TestDepContext.accepts_block_for :test_defaults, &lambda
+    DepContext.accepts_block_for :test_defaults, &lambda
     value_from_block = nil
     dep 'default test' do
       value_from_block = test_defaults
@@ -49,10 +58,15 @@ end
 
 describe "accepts_list_for behaviour" do
   before {
-    Babushka::Base.stub!(:host).and_return FakeOSXSystemProfile.new
-    make_test_deps
+    Babushka.host.stub!(:match_list).and_return([:osx])
+    dep 'test build tools' do
+      requires {
+        on :osx, 'xcode tools'
+        on :linux, 'build-essential', 'autoconf'
+      }
+    end
   }
   it "should choose requires for the correct system" do
-    Dep('test build tools').context.requires.should == ['xcode tools']
+    Dep('test build tools').context.define!.requires.should == ['xcode tools']
   end
 end