rubycut-babushka 0.10.8 → 0.15.6.1

data/spec/babushka/vars_spec.rb

@@ -2,7 +2,8 @@ require 'spec_helper'
 
 describe Vars do
   before {
-    Base.task.vars.define_var :username, :default => shell('whoami')
+    Base.task.process [], {} # a hack to assign Base.task.vars
+    Base.task.vars.define_var :username, :default => ShellHelpers.shell('whoami')
     Base.task.vars.define_var :domain, :default => :username
     Base.task.vars.define_var :db_name
     Base.task.vars.define_var :test_user, :default => :db_name

data/spec/babushka/version_of_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 def version_of *args
-  Babushka::VersionOf::Helpers.VersionOf *args
+  Babushka.VersionOf(*args)
 end
 
 describe "creation" do
@@ -25,6 +25,12 @@ describe "creation" do
     version_of(version_of('ruby', '1.8')).should == version_of('ruby', '1.8')
     version_of(version_of('ruby', '1.8'), '1.9').should == version_of('ruby', '1.9')
   end
+  it "should accept name with space" do
+    version_of('Google Chrome.app').name.should == 'Google Chrome.app'
+    version_of('Google Chrome.app').version.should == nil
+  end
+  # TODO: This is tricky, e.g. splitting "Sublime Text 2 >= 2.0.1".
+  # it "should accept name with space and version"
 end
 
 describe "to_s" do
@@ -39,9 +45,30 @@ describe "to_s" do
     end
   end
   describe "versioned" do
-    it "should be separated with -" do
+    it "should be separated with - when no operator is specified" do
       version_of('ruby', '1.8').to_s.should == 'ruby-1.8'
     end
+    it "should be separated with - when the operator is ==" do
+      version_of('ruby', '== 1.8').to_s.should == 'ruby-1.8'
+    end
+    it "should be separated with - when no version is specified" do
+      version_of('ruby', '>= 1.8').to_s.should == 'ruby >= 1.8'
+    end
+  end
+end
+
+describe '#exact?' do
+  it "should be false when there is no version" do
+    version_of('ruby').should_not be_exact
+  end
+  it "should be true when there is a just version number" do
+    version_of('ruby', '1.8').should be_exact
+  end
+  it "should be true when the operator is ==" do
+    version_of('ruby', '== 1.8').should be_exact
+  end
+  it "should be false when the operator is not ==" do
+    version_of('ruby', '>= 1.8').should_not be_exact
   end
 end
 

data/spec/babushka/version_str_spec.rb

@@ -1,56 +1,6 @@
 require 'spec_helper'
 require 'version_str_support'
 
-def compare_with operator
-  pairs.zip(results[operator]).each {|pair,expected|
-    result = VersionStr.new(pair.first).send operator, VersionStr.new(pair.last)
-    it "#{pair.first} #{operator} #{pair.last}: #{result}" do
-      result.should == expected
-    end
-  }
-end
-
-%w[== != > < >= <= ~>].each do |operator|
-  describe operator do
-    compare_with operator
-  end
-end
-
-describe "comparing" do
-  it "should work with other VersionStrs" do
-    (VersionStr.new('0.3.1') > VersionStr.new('0.2.9')).should be_true
-  end
-  
-  it "should work with strings" do
-    (VersionStr.new('0.3.1') > '0.2.9').should be_true
-  end
-  
-  it "should treat word pieces as less than no piece" do
-    (VersionStr.new('3.0.0') > VersionStr.new('3.0.0.beta')).should be_true
-    (VersionStr.new('3.0.0') > VersionStr.new('3.0.0.beta1')).should be_true
-    (VersionStr.new('1.0.0') > VersionStr.new('1.0.0.rc.5')).should be_true
-  end
-
-  it "should compare word pieces alphabetically" do
-    (VersionStr.new('3.0.0.beta') < VersionStr.new('3.0.0.pre')).should be_true
-    (VersionStr.new('3.0.0.pre') < VersionStr.new('3.0.0.rc')).should be_true
-  end
-
-  it "should treat word pieces with a number as more than without one" do
-    (VersionStr.new('3.0.0.beta1') > VersionStr.new('3.0.0.beta')).should be_true
-  end
-
-  it "should compare number parts of word pieces numerically" do
-    (VersionStr.new('3.0.0.beta2') > VersionStr.new('3.0.0.beta1')).should be_true
-    (VersionStr.new('3.0.0.beta10') > VersionStr.new('3.0.0.beta1')).should be_true
-  end
-  
-  it "should allow for integers in strings and sort correctly" do
-    (VersionStr.new('3.0.0.beta12') > VersionStr.new('3.0.0.beta2')).should be_true
-    (VersionStr.new('R13B04') > VersionStr.new('R2B9')).should be_true
-  end
-end
-
 describe "parsing" do
   it "should parse the version number" do
     VersionStr.new('0.2').pieces.should == [0, 2]
@@ -59,6 +9,7 @@ describe "parsing" do
     VersionStr.new('3.0.0.beta').pieces.should == [3, 0, 0, 'beta']
     VersionStr.new('3.0.0.beta3').pieces.should == [3, 0, 0, 'beta', 3]
     VersionStr.new('R13B04').pieces.should == ['R', 13, 'B', 4]
+    VersionStr.new('HEAD').pieces.should == ['HEAD']
   end
   it "should parse the operator if supplied" do
     v = VersionStr.new('>0.2')
@@ -91,23 +42,48 @@ describe "parsing" do
     v.pieces.should == [1, 9, 2, 'p', 180]
     v.operator.should == '>='
   end
-  it "should reject invalid operators" do
-    L{
-      VersionStr.new('~ 0.2')
-    }.should raise_error(InvalidVersionOperator, "VersionStr.new('~ 0.2'): invalid operator '~'.")
-
-    L{
-      VersionStr.new('>> 0.2')
-    }.should raise_error(InvalidVersionOperator, "VersionStr.new('>> 0.2'): invalid operator '>>'.")
+  describe "invalid operators" do
+    it "should reject them" do
+      L{
+        VersionStr.new('~ 0.2')
+      }.should raise_error(InvalidVersionOperator, "VersionStr.new('~ 0.2'): invalid operator '~'.")
+      L{
+        VersionStr.new('>> 0.2')
+      }.should raise_error(InvalidVersionOperator, "VersionStr.new('>> 0.2'): invalid operator '>>'.")
+    end
+  end
+  describe "invalid version numbers" do
+    it "should reject version numbers that don't contain any digits" do
+      L{
+        VersionStr.new('nginx')
+      }.should raise_error(InvalidVersionStr, "VersionStr.new('nginx'): couldn't parse a version number.")
+    end
+    it "should reject numbers containing spaces" do
+      L{
+        VersionStr.new('0. 2')
+      }.should raise_error(InvalidVersionStr, "VersionStr.new('0. 2'): couldn't parse a version number.")
+    end
+    it "should reject numbers containing unexpected chars" do
+      L{
+        VersionStr.new('0.2!')
+      }.should raise_error(InvalidVersionStr, "VersionStr.new('0.2!'): couldn't parse a version number.")
+    end
   end
-  it "should reject bad version numbers" do
-    L{
-      VersionStr.new('0. 2')
-    }.should raise_error(InvalidVersionStr, "VersionStr.new('0. 2'): couldn't parse a version number.")
+end
 
-    L{
-      VersionStr.new('0.2!')
-    }.should raise_error(InvalidVersionStr, "VersionStr.new('0.2!'): couldn't parse a version number.")
+describe '#parseable_version?' do
+  it 'should not report emptyness as parseable' do
+    VersionStr.parseable_version?(nil).should be_false
+    VersionStr.parseable_version?('').should be_false
+    VersionStr.parseable_version?('  ').should be_false
+  end
+  it "should not report digitless input as parseable" do
+    VersionStr.parseable_version?('nginx').should be_false
+  end
+  it "should not report input with digits as parseable" do
+    VersionStr.parseable_version?('3').should be_true
+    VersionStr.parseable_version?('R13B04').should be_true
+    VersionStr.parseable_version?('1.9.3-p0').should be_true
   end
 end
 
@@ -128,3 +104,53 @@ describe 'rendering' do
     VersionStr.new('3.0.0-beta').to_s.should == '3.0.0-beta'
   end
 end
+
+def compare_with operator
+  pairs.zip(results[operator]).each {|pair,expected|
+    result = VersionStr.new(pair.first).send operator, VersionStr.new(pair.last)
+    it "#{pair.first} #{operator} #{pair.last}: #{result}" do
+      result.should == expected
+    end
+  }
+end
+
+%w[== != > < >= <= ~>].each do |operator|
+  describe operator do
+    compare_with operator
+  end
+end
+
+describe "comparing" do
+  it "should work with other VersionStrs" do
+    (VersionStr.new('0.3.1') > VersionStr.new('0.2.9')).should be_true
+  end
+
+  it "should work with strings" do
+    (VersionStr.new('0.3.1') > '0.2.9').should be_true
+  end
+
+  it "should treat word pieces as less than no piece" do
+    (VersionStr.new('3.0.0') > VersionStr.new('3.0.0.beta')).should be_true
+    (VersionStr.new('3.0.0') > VersionStr.new('3.0.0.beta1')).should be_true
+    (VersionStr.new('1.0.0') > VersionStr.new('1.0.0.rc.5')).should be_true
+  end
+
+  it "should compare word pieces alphabetically" do
+    (VersionStr.new('3.0.0.beta') < VersionStr.new('3.0.0.pre')).should be_true
+    (VersionStr.new('3.0.0.pre') < VersionStr.new('3.0.0.rc')).should be_true
+  end
+
+  it "should treat word pieces with a number as more than without one" do
+    (VersionStr.new('3.0.0.beta1') > VersionStr.new('3.0.0.beta')).should be_true
+  end
+
+  it "should compare number parts of word pieces numerically" do
+    (VersionStr.new('3.0.0.beta2') > VersionStr.new('3.0.0.beta1')).should be_true
+    (VersionStr.new('3.0.0.beta10') > VersionStr.new('3.0.0.beta1')).should be_true
+  end
+
+  it "should allow for integers in strings and sort correctly" do
+    (VersionStr.new('3.0.0.beta12') > VersionStr.new('3.0.0.beta2')).should be_true
+    (VersionStr.new('R13B04') > VersionStr.new('R2B9')).should be_true
+  end
+end

data/spec/babushka/xml_string_spec.rb

@@ -95,4 +95,4 @@ def sample_xml
     </dict>
     </plist>
   }
-end
+end

data/spec/deps/bad/broken.rb

@@ -1,7 +1,7 @@
 dep 'broken test dep 1' do
-  syntax error
 end
 
+syntax } error
+
 dep 'test dep 2' do
-  
 end

data/spec/deps/bad/working.rb

@@ -1,3 +1,2 @@
 dep 'test dep 1' do
-  
 end

data/spec/deps/good/test.rb

@@ -1,11 +1,8 @@
 meta 'test_meta_1' do
-  
 end
 
 dep 'test dep 1' do
-  
 end
 
 dep 'test dep 2' do
-  
 end

data/spec/deps/outer/deps.rb

@@ -1,9 +1,7 @@
 dep 'test dep 1' do
-  
 end
 
 dep 'externally templated', :template => 'nested source:test_template' do
-  
 end
 
 meta 'local_template' do

data/spec/fancypath/fancypath_spec.rb

@@ -111,6 +111,36 @@ describe Fancypath do
     it('removes directory') { @dir.create.remove.should_not exist }
   end
 
+  describe "#grep" do
+    before {
+      @file.write("some\ncontent\ncontentedness")
+    }
+    it("returns nil when the file doesn't exist") {
+      (@dir / 'missing').grep(/test/).should be_nil
+    }
+    it('returns nil on no match') {
+      @file.grep(/test/).should be_nil
+    }
+    it('returns the matches with string parameter') {
+      @file.grep("content").should == ['content']
+    }
+    it('returns the matches with regexp parameter') {
+      @file.grep(/cont/).should == ['content', 'contentedness']
+    }
+  end
+
+  describe "#yaml" do
+    before {
+      @file.write("
+        guise:
+          seriously: guise
+      ")
+    }
+    it('returns the file contents as yaml') {
+      @file.yaml.should == {'guise' => {'seriously' => 'guise'}}
+    }
+  end
+
   describe '#readlink' do
     before {
       @file.touch

data/spec/inkan/inkan_spec.rb

@@ -1,5 +1,7 @@
 require 'spec_helper'
 
+require 'digest/sha1'
+
 describe "Inkan" do
   describe '.legitimate?' do
     context "without a hashbang" do
@@ -63,13 +65,13 @@ describe "Inkan" do
       end
     end
   end
-  
+
   describe '.seal' do
     it "should write the file after closing the block" do
       Inkan.seal('/tmp/spec.txt') do |inkan|
         inkan.print 'foo bar baz'
       end
-      
+
       contents = open('/tmp/spec.txt').read
       contents.should match(/foo bar baz$/)
       contents.should match(/^# Generated by Inkan/)
@@ -81,49 +83,49 @@ describe "Inkan" do
       result = Inkan.render do |inkan|
         inkan.print 'foo bar baz'
       end
-      
+
       result.should match(/foo bar baz$/)
       result.should match(/^# Generated by Inkan/)
     end
   end
-  
+
   describe '#seal' do
     let(:inkan) { Inkan.new('/tmp/spec.txt') }
-    
+
     it "writes out the contents of the buffer" do
       inkan.print "foo bar baz"
       inkan.seal
-      
+
       open('/tmp/spec.txt').read.should match(/foo bar baz$/)
     end
-    
+
     it "adds the credit line to the top of the file" do
       inkan.print "foo bar baz"
       inkan.seal
-      
+
       open('/tmp/spec.txt').read.should match(/^# Generated by Inkan/)
     end
-    
+
     it "adds the SHA to the top of the file" do
       inkan.print "foo bar baz"
       inkan.seal
-      
+
       sha = Digest::SHA1.hexdigest("foo bar baz")
-      
+
       open('/tmp/spec.txt').read.should match(/#{sha}/)
     end
   end
 
   describe '#render' do
     let(:inkan) { Inkan.new(nil) }
-    
-    before { shell "rm -f /tmp/spec.txt" }
-    
+
+    before { ShellHelpers.shell "rm -f /tmp/spec.txt" }
+
     it "writes out the contents of the buffer" do
       inkan.print "foo bar baz"
       inkan.render.should match(/foo bar baz$/)
     end
-    
+
     context "when there is no hashbang" do
       before { inkan.print "foo bar baz" }
       it "adds the credit line to the top of the output" do
@@ -134,7 +136,7 @@ describe "Inkan" do
         inkan.render.split("\n").first.should match(/#{sha}/)
       end
     end
-    
+
     context "when there is a hashbang" do
       before { inkan.print "#!/bin/sh\nfoo bar baz" }
       it "leaves the hashbang at the top of the file" do
@@ -148,69 +150,69 @@ describe "Inkan" do
         inkan.render.split("\n")[1].should match(/#{sha}/)
       end
     end
-    
+
     it "doesn't write the file" do
       inkan.print "foo bar baz"
       inkan.render
       File.exists?('/tmp/spec.txt').should be_false
     end
   end
-  
+
   describe '#puts' do
     let(:inkan) { Inkan.new('/tmp/spec.txt') }
-    
+
     it "adds an extra new line to the output" do
       inkan.puts "foo bar baz"
       inkan.seal
-      
+
       open('/tmp/spec.txt').read.should match(/foo bar baz\n$/)
     end
   end
-  
+
   describe '#credit' do
     let(:inkan) { Inkan.new('/tmp/spec.txt') }
-    
+
     it "defaults to 'Generated by Inkan'" do
       inkan.credit.should == 'Generated by Inkan'
     end
-    
+
     it "should pass through changes to the file comment" do
       inkan.credit = "Pat's Magic Code"
       inkan.print "foo bar baz"
       inkan.seal
-      
+
       open('/tmp/spec.txt').read.should match(/^# Pat's Magic Code/)
     end
   end
-  
+
   describe '#comment' do
     let(:inkan) { Inkan.new('/tmp/spec.txt') }
-    
+
     it "defaults to a hash symbol" do
       inkan.comment.should == '#'
     end
-    
+
     it "should pass through changes to the file credit" do
       inkan.comment = "//"
       inkan.print "foo bar baz"
       inkan.seal
-      
+
       open('/tmp/spec.txt').read.should match(/^\/\/ Generated by Inkan/)
     end
   end
-  
+
   describe '#comment_suffix' do
     let(:inkan) { Inkan.new('/tmp/spec.txt') }
-    
+
     it "defaults to a blank string" do
       inkan.comment_suffix.should == ''
     end
-    
+
     it "should pass through changes to the file comment" do
       inkan.comment_suffix = "*/"
       inkan.print "foo bar baz"
       inkan.seal
-      
+
       open('/tmp/spec.txt').read.should match(/\*\/\n/)
     end
   end