buildr 1.3.5 → 1.4.0

data/spec/packaging/archive_spec.rb

@@ -22,6 +22,8 @@ describe 'ArchiveTask', :shared=>true do
     @dir = File.expand_path('test')
     @files = %w{Test1.txt Text2.html}.map { |file| File.expand_path(file, @dir) }.
       each { |file| write file, content_for(file) }
+    @empty_dirs = %w{EmptyDir1 EmptyDir2}.map { |file| File.expand_path(file, @dir) }.
+      each { |file| mkdir file }
   end
 
   # Not too smart, we just create some content based on file name to make sure you read what you write.
@@ -59,6 +61,11 @@ describe 'ArchiveTask', :shared=>true do
     inspect_archive { |archive| archive.should be_empty }
   end
 
+  it 'should raise error when include() is called with nil values' do
+    lambda { archive(@archive).include(nil) }.should raise_error
+    lambda { archive(@archive).include([nil]) }.should raise_error
+  end
+
   it 'should create empty archive if called #clean method' do
     archive(@archive).include(@files).clean.invoke
     inspect_archive { |archive| archive.should be_empty }
@@ -83,6 +90,22 @@ describe 'ArchiveTask', :shared=>true do
     inspect_archive.keys.should include('included')
   end
 
+  it 'should archive artifacts' do
+    write 'library-1.0.txt', 'library-1.0'
+    artifact("org.example:library:txt:1.0").from 'library-1.0.txt'
+    archive(@archive).include("org.example:library:txt:1.0").invoke
+    inspect_archive.keys.should include('library-1.0.txt')
+  end
+
+  it 'should archive project artifacts' do
+    define 'p1' do
+      project.version = '1.0'
+      package(:zip)
+    end
+    archive(@archive).include(project('p1')).invoke
+    inspect_archive.keys.should include('p1-1.0.zip')
+  end
+
   it 'should include entry for directory' do
     archive(@archive).include(@dir).invoke
     inspect_archive { |archive| @files.each { |f| archive['test/' + File.basename(f)].should eql(content_for(f)) } }
@@ -112,12 +135,18 @@ describe 'ArchiveTask', :shared=>true do
     end
   end
 
+  it 'should raise error when using :from with nil value' do
+    lambda {
+      archive(@archive).include(:from=>nil)
+    }.should raise_error
+  end
+
   it 'should exclude entire directory and all its children' do
     mkpath "#{@dir}/sub"
     write "#{@dir}/sub/test"
     archive(@archive).include(@dir).exclude("#{@dir}/sub").invoke
     inspect_archive do |archive|
-      archive.keys.select { |file| file =~ /sub/ }.should be_empty 
+      archive.keys.select { |file| file =~ /sub/ }.should be_empty
     end
   end
 
@@ -171,6 +200,11 @@ describe 'ArchiveTask', :shared=>true do
     archive(@archive).include(@files.first, :as=>'test/sample').invoke
     inspect_archive { |archive| @files.each { |f| archive['test/sample'].should eql(content_for(@files.first)) } }
   end
+  
+  it 'should archive directory into specified alias, without using "."' do
+    archive(@archive).include(@dir, :as=>'.').invoke
+    inspect_archive { |archive| archive.keys.should_not include(".") }
+  end
 
   it 'should archive file into specified name/path' do
     archive(@archive).include(@files.first, :as=>'test/sample', :path=>'path').invoke
@@ -219,19 +253,19 @@ describe 'ArchiveTask', :shared=>true do
     create_for_merge do |src|
       archive(@archive).merge(src).include(File.basename(@files.first))
       archive(@archive).invoke
-      inspect_archive do |archive| 
+      inspect_archive do |archive|
         archive[File.basename(@files.first)].should eql(content_for(@files.first))
         archive[File.basename(@files.last)].should be_nil
       end
     end
   end
-  
+
   it 'should expand another archive file with exclude pattern' do
     create_for_merge do |src|
       archive(@archive).merge(src).exclude(File.basename(@files.first))
       archive(@archive).invoke
-      inspect_archive do |archive| 
-        @files[1..-1].each { |f| archive[File.basename(f)].should eql(content_for(f)) } 
+      inspect_archive do |archive|
+        @files[1..-1].each { |f| archive[File.basename(f)].should eql(content_for(f)) }
         archive[File.basename(@files.first)].should be_nil
       end
     end
@@ -285,7 +319,7 @@ describe 'ArchiveTask', :shared=>true do
     # all included files newer.
     File.utime Time.now - 100, Time.now - 100, @archive
     archive(@archive).include(@files).invoke
-    File.stat(@archive).mtime.should be_close(Time.now, 10) 
+    File.stat(@archive).mtime.should be_close(Time.now, 10)
   end
 
   it 'should do nothing if all files are uptodate' do
@@ -293,7 +327,7 @@ describe 'ArchiveTask', :shared=>true do
     # By touching all files in the past, there's nothing new to update.
     (@files + [@archive]).each { |f| File.utime Time.now - 100, Time.now - 100, f }
     archive(@archive).include(@files).invoke
-    File.stat(@archive).mtime.should be_close(Time.now - 100, 10) 
+    File.stat(@archive).mtime.should be_close(Time.now - 100, 10)
   end
 
   it 'should update if one of the files is recent' do
@@ -342,7 +376,7 @@ describe TarTask, ' gzipped' do
 
   def inspect_archive
     entries = {}
-    Zlib::GzipReader.open @archive do |gzip| 
+    Zlib::GzipReader.open @archive do |gzip|
       Archive::Tar::Minitar.open gzip, 'r' do |reader|
         reader.each { |entry| entries[entry.directory ? "#{entry.name}/" : entry.name] = entry.read }
       end
@@ -370,23 +404,50 @@ describe ZipTask do
     entries
   end
 
+  it 'should include empty dirs' do
+    archive(@archive).include(@dir)
+    archive(@archive).invoke
+    inspect_archive do |archive|
+      archive.keys.should include('test/EmptyDir1/')
+    end
+  end
+
+  it 'should include empty dirs from Dir' do
+    archive(@archive).include(Dir["#{@dir}/*"])
+    archive(@archive).invoke
+    inspect_archive do |archive|
+      archive.keys.should include('EmptyDir1/')
+    end
+  end
+
   it 'should work with path object' do
     archive(@archive).path('code').include(@files)
     archive(@archive).invoke
     inspect_archive { |archive| archive.keys.should include('code/') }
   end
+  
+  it 'should have path object that includes empty dirs' do
+    archive(@archive).path('code').include(Dir["#{@dir}/*"])
+    archive(@archive).invoke
+    inspect_archive do |archive|
+      archive.keys.should include('code/EmptyDir1/')
+    end
+  end
 
-  it 'should preserve file permissions' do
-    # with JRuby it's important to use absolute paths with File.chmod()
-    # http://jira.codehaus.org/browse/JRUBY-3300
-    hello = File.expand_path('src/main/bin/hello')
-    write hello, 'echo hi'
-    File.chmod(0777,  hello) || 
-    fail("Failed to set permission on #{hello}") unless (File.stat(hello).mode & 0777) == 0777
+  # chmod is not reliable on Windows
+  unless Buildr::Util.win_os?
+    it 'should preserve file permissions' do
+      # with JRuby it's important to use absolute paths with File.chmod()
+      # http://jira.codehaus.org/browse/JRUBY-3300
+      hello = File.expand_path('src/main/bin/hello')
+      write hello, 'echo hi'
+      File.chmod(0777,  hello)
+      fail("Failed to set permission on #{hello}") unless (File.stat(hello).mode & 0777) == 0777
 
-    zip('foo.zip').include('src/main/bin/*').invoke
-    unzip('target' => 'foo.zip').extract
-    (File.stat('target/hello').mode & 0777).should == 0777
+      zip('foo.zip').include('src/main/bin/*').invoke
+      unzip('target' => 'foo.zip').extract
+      (File.stat('target/hello').mode & 0777).should == 0777
+    end
   end
 
 end
@@ -399,6 +460,8 @@ describe Unzip do
     @files = %w{Test1.txt Text2.html}.map { |file| File.join(@dir, file) }.
       each { |file| write file, content_for(file) }
     @target = File.expand_path('target')
+    @targz = File.expand_path('test.tar.gz')
+    @targz2 = File.expand_path('test.tgz')
   end
 
   # Not too smart, we just create some content based on file name to
@@ -407,11 +470,21 @@ describe Unzip do
     "Content for #{File.basename(file)}"
   end
 
+  def with_tar(*args)
+    tar(@targz).include(*args.empty? ? @files : args).invoke
+    yield
+  end
+
+  def with_tar_too(*args)
+    tar(@targz2).include(*args.empty? ? @files : args).invoke
+    yield
+  end
+
   def with_zip(*args)
     zip(@zip).include(*args.empty? ? @files : args).invoke
     yield
   end
-    
+
   it 'should touch target directory' do
     with_zip do
       mkdir @target
@@ -428,6 +501,20 @@ describe Unzip do
     end
   end
 
+  it 'should expand files from a tar.gz file' do
+    with_tar do
+      unzip(@target=>@targz).target.invoke
+      @files.each { |f| File.read(File.join(@target, File.basename(f))).should eql(content_for(f)) }
+    end
+  end
+
+  it 'should expand files from a .tgz file' do
+    with_tar_too do
+      unzip(@target=>@targz2).target.invoke
+      @files.each { |f| File.read(File.join(@target, File.basename(f))).should eql(content_for(f)) }
+    end
+  end
+
   it 'should expand all files' do
     with_zip do
       unzip(@target=>@zip).target.invoke
@@ -435,6 +522,13 @@ describe Unzip do
     end
   end
 
+  it 'should expand all files from a .tar.gz file' do
+    with_tar do
+      unzip(@target=>@targz).target.invoke
+      FileList[File.join(@target, '*')].size.should be(@files.size)
+    end
+  end
+
   it 'should expand only included files' do
     with_zip do
       only = File.basename(@files.first)
@@ -444,6 +538,15 @@ describe Unzip do
     end
   end
 
+  it 'should expand only included files from a .tar.gz file' do
+    with_tar do
+      only = File.basename(@files.first)
+      unzip(@target=>@targz).include(only).target.invoke
+      FileList[File.join(@target, '*')].should include(File.expand_path(only, @target))
+      FileList[File.join(@target, '*')].size.should be(1)
+    end
+  end
+
   it 'should expand all but excluded files' do
     with_zip do
       except = File.basename(@files.first)
@@ -453,6 +556,15 @@ describe Unzip do
     end
   end
 
+  it 'should expand all but excluded files with a .tar.gz file' do
+    with_tar do
+      except = File.basename(@files.first)
+      unzip(@target=>@targz).exclude(except).target.invoke
+      FileList[File.join(@target, '*')].should_not include(File.expand_path(except, @target))
+      FileList[File.join(@target, '*')].size.should be(@files.size - 1)
+    end
+  end
+
   it 'should include with nested path patterns' do
     with_zip @files, :path=>'test/path' do
       only = File.basename(@files.first)
@@ -469,6 +581,22 @@ describe Unzip do
     end
   end
 
+  it 'should include with nested path patterns with a .tar.gz file' do
+    with_tar @files, :path=>'test/path' do
+      only = File.basename(@files.first)
+      unzip(@target=>@targz).include(only).target.invoke
+      FileList[File.join(@target, '*')].should be_empty
+
+      Rake::Task.clear ; rm_rf @target
+      unzip(@target=>@targz).include('test/path/' + only).target.invoke
+      FileList[File.join(@target, 'test/path/*')].size.should be(1)
+
+      Rake::Task.clear ; rm_rf @target
+      unzip(@target=>@targz).include('test/**/*').target.invoke
+      FileList[File.join(@target, 'test/path/*')].size.should be(2)
+    end
+  end
+
   it 'should include with relative path' do
     with_zip @files, :path=>'test/path' do
       only = File.basename(@files.first)
@@ -488,7 +616,27 @@ describe Unzip do
       FileList[File.join(@target, 'path/*')].size.should be(2)
     end
   end
-  
+
+  it 'should include with relative path with a .tar.gz file' do
+    with_tar @files, :path=>'test/path' do
+      only = File.basename(@files.first)
+      unzip(@target=>@targz).tap { |unzip| unzip.from_path('test').include(only) }.target.invoke
+      FileList[File.join(@target, '*')].should be_empty
+
+      Rake::Task.clear ; rm_rf @target
+      unzip(@target=>@targz).tap { |unzip| unzip.from_path('test').include('test/*') }.target.invoke
+      FileList[File.join(@target, 'path/*')].should be_empty
+
+      Rake::Task.clear ; rm_rf @target
+      unzip(@target=>@targz).tap { |unzip| unzip.from_path('test').include('path/*' + only) }.target.invoke
+      FileList[File.join(@target, 'path/*')].size.should be(1)
+
+      Rake::Task.clear ; rm_rf @target
+      unzip(@target=>@targz).tap { |unzip| unzip.from_path('test').include('path/*') }.target.invoke
+      FileList[File.join(@target, 'path/*')].size.should be(2)
+    end
+  end
+
   it 'should exclude with relative path' do
     with_zip @files, :path=>'test' do
       except = File.basename(@files.first)
@@ -498,6 +646,15 @@ describe Unzip do
     end
   end
 
+  it 'should exclude with relative path on a tar.gz file' do
+    with_tar @files, :path=>'test' do
+      except = File.basename(@files.first)
+      unzip(@target=>@targz).tap { |unzip| unzip.from_path('test').exclude(except) }.target.invoke
+      FileList[File.join(@target, '*')].should include(File.join(@target, File.basename(@files[1])))
+      FileList[File.join(@target, '*')].size.should be(@files.size - 1)
+    end
+  end
+
   it "should handle relative paths without any includes or excludes" do
     lib_files = %w{Test3.so Test4.rb}.
       map { |file| File.join(@dir, file) }.
@@ -508,6 +665,16 @@ describe Unzip do
     FileList[File.join(@target, '**/*')].should have(2).files
   end
 
+  it "should handle relative paths without any includes or excludes with a tar.gz file" do
+    lib_files = %w{Test3.so Test4.rb}.
+      map { |file| File.join(@dir, file) }.
+      each { |file| write file, content_for(file) }
+    tar(@targz).include(@files, :path => 'src').include(lib_files, :path => 'lib').invoke
+
+    unzip(@target=>@targz).tap { |unzip| unzip.from_path('lib') }.target.invoke
+    FileList[File.join(@target, '**/*')].should have(2).files
+  end
+
   it 'should return itself from root method' do
     task = unzip(@target=>@zip)
     task.root.should be(task)