file-dependencies 0.1.2 → 0.1.3

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    ZWFmYzZiYWUyY2Q5Njc4NDE4ZDdmYjAxYzllZjAzZTJiOTAzMTU3Mg==
+    OGVkNDBkYmY4NWM4N2UxOTRiODhiM2QwN2QwZGM2ZjEwM2Q4YzFiZQ==
   data.tar.gz: !binary |-
-    ZTFhNGU3MTAxMmMyMjQxZTVlZTgzZTUzZTE4OTcwZDBkZmFiOGZmOA==
+    YmZjNmFlMWU3YWFiOGYwNWU4NjM0ZWI0Yjc5Y2U1YjBlM2JkNjlkYw==
 SHA512:
   metadata.gz: !binary |-
-    ZThhMzA1NTkwMzY2MjZmZDdkNjlkNjFlNGQ2NDMxNGJhZDBlMDk2ZmZmYzA0
-    ZmVjMTcwNDc4NjNiMGM2NjllY2U2ZWM0NDhhMmM4ZTBiNWMwZmZjYjBlM2Ez
-    MDZiMmRlYzYzNzg0MmFkZjA3ZWQ4ZmI2Y2ZjNTBhM2I4YmE3Yzc=
+    YWJlMmU3NmI5OTdiNTUxNjQ3Y2E2NWZjMzE0YWYwNjQxMTNlZGJkM2E1YjIw
+    MzdiN2IyN2Q2Y2YxNTVhNDAzODU4MWY0ZTNjZDVkZDcxNDY2NTFiZmEzMjEx
+    ZjAxMjY4YWQ4YzU2ZTBkZDRjZWU3MjUxMDY0YmNjMmM1ZTAyNDg=
   data.tar.gz: !binary |-
-    ODFkNTc5Mzg1NDc4YzY1MGEwNzM4MDRlNjJkN2M2N2RjZDU4YmUwYjYzMTUx
-    Nzc0YzdjZDY5YjgyM2U5YTFjMTYzMzhmMjM3OTA4OWEzMGE4MGUxZTFkZjg1
-    OWU3NTFhMWYzODNlNGM0M2E5NTQwODYwZTY1NTEzM2E2Y2E3M2Q=
+    NGMxMDlmMzg0MTgzNjVlYjQxM2Q0MjI1YjE4OWUzNjU4ZWQ0MGI1YjkwYTA5
+    NGVmMzg1MjYxYTZmNDBjYjk1YjMwZmI0MTZiMzEzMDA0MGFiNmVlY2UxMGIw
+    OWVlMjZlMzg5NmU0NWNmNTUwNGM5OTcwNzE0NGNlZWJiYmQ3Mzk=

data/.gitignore

@@ -6,3 +6,4 @@ pkg
 target
 build.log
 *.swp
+coverage/

data/file-dependencies.gemspec

@@ -3,7 +3,7 @@ require File.expand_path('../lib/file-dependencies/version', __FILE__)
 
 Gem::Specification.new do |s|
   s.name = 'file-dependencies'
-  s.version = APP_VERSION
+  s.version = FileDependencies::VERSION
   s.author = 'Richard Pijnenburg'
   s.email = ['richard.pijnenburg@elasticsearch.com']
   s.summary = 'manage file dependencies for gems'

data/lib/file-dependencies.rb

@@ -10,34 +10,34 @@ module FileDependencies
     if ::File.exist?(vendor_file)
       vendor_file_content = IO.read(vendor_file)
       file_list = JSON.load(vendor_file_content)
-      FileDependencies.download(file_list, ::File.join(dir, target), tmpdir)
+      FileDependencies.process_downloads(file_list, ::File.join(dir, target), tmpdir)
     else
       puts "vendor.json not found, looked for the file at #{vendor_file}"
     end
   end # def process_vendor
   module_function :process_vendor
 
-  def download(files, target, tmpdir)
+  def process_downloads(files, target, tmpdir)
     FileUtils.mkdir_p(target) unless ::File.directory?(target)
     files.each do |file|
-      target = ::File.join(target, file['target']) unless file['target'].nil?
+      full_target = file['target'] ? ::File.join(target, file['target']) : target
       download = FileDependencies::File.fetch_file(file['url'], file['sha1'], tmpdir)
-      if (res = download.match(/(\S+?)(\.tar\.gz|\.tgz)/))
+      if (res = download.match(/(\S+?)(\.tar\.gz|\.tgz)$/))
         prefix = res.captures.first.gsub("#{tmpdir}/", '')
         FileDependencies::Archive.untar(download) do |entry|
-          if file['extract'].nil?
-            ::File.join(target, entry.full_name.gsub(prefix, ''))
+          next unless FileDependencies::Archive.extract_file?(entry.full_name, file['extract'], file['exclude'], prefix)
+          if file['flatten'] == true
+            ::File.join(full_target, entry.full_name.split("/").last)
           else
-            next unless FileDependencies::Archive.eval_file(entry.full_name, file['extract'], prefix)
-            ::File.join(target, entry.full_name.split("/").last)
+            ::File.join(full_target, entry.full_name.gsub(prefix, ''))
           end
         end
-      elsif download =~ /.gz/
-        FileDependencies::Archive.ungzip(download, target)
+      elsif download =~ /.gz$/
+        FileDependencies::Archive.ungzip(download, full_target)
       else
-        FileUtils.mv(download, ::File.join(target, download.split("/").last))
+        FileUtils.mv(download, ::File.join(full_target, download.split("/").last))
       end
     end
   end # def download
-  module_function :download
+  module_function :process_downloads
 end

data/lib/file-dependencies/archive.rb

@@ -64,20 +64,42 @@ module FileDependencies
     end # def untar
     module_function :untar
 
-    def eval_file(entry, files, prefix)
+    def extract_file?(entry, extract=nil, exclude=nil, prefix)
+      return true if extract.nil? && exclude.nil?
       return false if tar_header?(entry)
-      if files.is_a?(Array)
-        return true if files.include?(entry.gsub(prefix, ''))
-      elsif files.is_a?(String)
-        return true if entry =~ Regexp.new(files)
+
+      if extract
+        return match?(entry, extract, prefix)
+      elsif exclude
+        return !match?(entry, exclude, prefix)
+      end
+    end
+    module_function :extract_file?
+
+    def match?(entry, list, prefix)
+      return false if list.nil?
+
+      if list.is_a?(Array)
+        list.each do |pattern|
+          if pattern.is_a?(Regexp) or regexp_string?(pattern)
+            return true if entry =~ Regexp.new(pattern)
+          elsif pattern.is_a?(String)
+            return true if pattern == entry.gsub(prefix, '')
+          end
+        end
       end
       false
     end
-    module_function :eval_file
+    module_function :match?
 
     def tar_header?(entry)
       entry =~ /PaxHeaders/
     end
     module_function :tar_header?
+
+    def regexp_string?(pattern)
+      pattern.to_s.match('^\(\?-mix:.+\)$')
+    end
+    module_function :regexp_string?
   end
 end

data/lib/file-dependencies/file.rb

@@ -24,17 +24,17 @@ module FileDependencies
     def validate_sha1(local_file, remote_sha1)
       return true if remote_sha1 == 'none'
       sha1 = fetch_sha1(remote_sha1)
-      local_sha1 = calc_sha1(local_file)
+      local_sha1 = calculate_sha1(local_file)
 
       raise("SHA1 did not match. Expected #{sha1} but computed #{local_sha1}") if sha1 != local_sha1
       true
     end # def validate_sha1
     module_function :validate_sha1
 
-    def calc_sha1(path)
+    def calculate_sha1(path)
       Digest::SHA1.file(path).hexdigest
     end # def calc__sha1
-    module_function :calc_sha1
+    module_function :calculate_sha1
 
     def fetch_file(url, sha1, target)
       puts "Downloading #{url}" if $DEBUG

data/lib/file-dependencies/version.rb

@@ -1,3 +1,5 @@
 # Note to authors: this should not include dashes because 'gem' barfs if
 # you include a dash in the version string.
-APP_VERSION = '0.1.2'
+module FileDependencies
+  VERSION = '0.1.3'
+end

data/spec/file-dependencies/archive_spec.rb

@@ -0,0 +1,138 @@
+require 'spec_helper'
+require 'file-dependencies/archive'
+
+describe FileDependencies::Archive do
+
+  describe ".ungzip" do
+
+    after do
+      FileUtils.remove_entry_secure(gzipfile) if ::File.exist?(gzipfile)
+      FileUtils.remove_entry_secure(expected_file)
+      FileUtils.remove_entry_secure(tmpdir)
+    end
+    let(:gzipfile) { Assist.generate_gzip('some_content') }
+    let(:expected_file) { gzipfile.gsub('.gz', '') }
+    let(:tmpdir) { Stud::Temporary.directory }
+
+    it 'decompresses a gzip file'do
+      expect { FileDependencies::Archive.ungzip(gzipfile, tmpdir) }.to_not(raise_error)
+      expect(File.exist?(expected_file))
+    end
+
+    let(:expected_file) { Assist.generate_file('some_content') }
+    it 'raises error extracting non gzip file' do
+      expect { FileDependencies::Archive.ungzip(expected_file, tmpdir) }.to(raise_error(Zlib::GzipFile::Error))
+    end
+  end
+
+  describe ".untar" do
+
+    after do
+      FileUtils.remove_entry_secure(tmpdir)
+      FileUtils.remove_entry_secure(file)
+    end
+
+    let(:file) { Assist.generate_file('some_content') }
+    let(:tarball) { Assist.generate_tarball('some/file' => 'content1', 'some/other/file' => 'content2', 'other' => 'content3') }
+    let(:tmpdir) { Stud::Temporary.directory }
+
+    it 'extracts a full tarball' do
+      entries = ['some/file', 'some/other/file', 'other']
+
+      FileDependencies::Archive.untar(tarball) do |entry|
+        ::File.join(tmpdir, entry.full_name)
+      end
+      found_files = Dir.glob(File.join(tmpdir, '**', '*')).reject { |entry| File.directory?(entry) }.sort
+      expected_files = entries.map { |k| ::File.join(tmpdir, k) }.sort
+      expect(expected_files).to(eq(found_files))
+    end
+
+    it 'extracts some files' do
+      entries = ['some/file', 'some/other/file']
+
+      FileDependencies::Archive.untar(tarball) do |entry|
+        if entries.include?(entry.full_name)
+          ::File.join(tmpdir, entry.full_name)
+        end
+      end
+      found_files = Dir.glob(File.join(tmpdir, '**', '*')).reject { |entry| File.directory?(entry) }.sort
+      expected_files = entries.map { |k| ::File.join(tmpdir, k) }.sort
+      expect(expected_files).to(eq(found_files))
+    end
+
+    it 'raises error when invalid file is provided' do
+      expect do
+        FileDependencies::Archive.untar(file) do |entry|
+          ::File.join(tmpdir, entry.full_name)
+        end
+      end.to(raise_error(Zlib::GzipFile::Error))
+    end
+  end
+
+  describe ".extract_file?" do
+
+    describe "with extract" do
+
+      let(:entries) { ['sometar/PaxHeaders', 'sometar/some/dir/PaxHeaders', 'sometar/some/dir/somefile', 'sometar/somefile', 'sometar/some/other/file', 'sometar/some/jars/file1.jar', 'sometar/some/jars/file2.jar', 'sometar/other/jars/file3.jar'] }
+      let(:prefix) { 'sometar' }
+
+      it 'returns all files based on a regex' do
+        extract1 = [/.jar/]
+        expect1  = ['file1.jar', 'file2.jar', 'file3.jar']
+
+        filelist = entries.reject { |entry| FileDependencies::Archive.extract_file?(entry, extract1, '', prefix) == false }.map { |entry| entry.gsub(prefix, '').split("/").last }
+        expect(filelist.sort).to(eq(expect1.sort))
+      end
+
+      it 'returns all files based on an array' do
+        extract2 = ['/some/other/file', '/somefile', '/other/jars/file3.jar']
+        expect2  = ['file', 'somefile', 'file3.jar']
+
+        filelist = entries.reject { |entry| FileDependencies::Archive.extract_file?(entry, extract2, '', prefix) == false }.map { |entry| entry.gsub(prefix, '').split("/").last }
+        expect(filelist.sort).to(eq(expect2.sort))
+      end
+
+      it 'returns all files from directories' do
+        extract3 = [/\/some\/jars/, /^sometar\/other/]
+        expect3  = ['file1.jar', 'file2.jar', 'file3.jar']
+
+        filelist = entries.reject { |entry| FileDependencies::Archive.extract_file?(entry, extract3, '',  prefix) == false }.map { |entry| entry.gsub(prefix, '').split("/").last }
+        expect(filelist.sort).to(eq(expect3.sort))
+      end
+
+    end
+
+    describe "with exclude" do
+
+      let(:entries) { ['sometar/PaxHeaders', 'sometar/some/dir/PaxHeaders', 'sometar/some/dir/somefile', 'sometar/somefile', 'sometar/some/other/file', 'sometar/some/jars/file1.jar', 'sometar/some/jars/file2.jar', 'sometar/other/jars/file3.jar'] }
+      let(:prefix) { 'sometar' }
+
+      it 'returns all non matching files based on a regex' do
+        extract1 = [/.jar/]
+        expect1  = ["file", "somefile", "somefile"]
+
+        filelist = entries.reject { |entry| FileDependencies::Archive.extract_file?(entry, nil, extract1, prefix) == false }.map { |entry| entry.gsub(prefix, '').split("/").last }
+        expect(filelist.sort).to(eq(expect1.sort))
+      end
+
+      it 'returns all non matching files based on an array' do
+        extract2 = ['/some/other/file', '/somefile', '/other/jars/file3.jar']
+        expect2  = ['file1.jar', 'file2.jar', 'somefile']
+
+        filelist = entries.reject { |entry| FileDependencies::Archive.extract_file?(entry, nil, extract2, prefix) == false }.map { |entry| entry.gsub(prefix, '').split("/").last }
+        expect(filelist.sort).to(eq(expect2.sort))
+      end
+
+      it 'returns all except the directories' do
+        extract3 = [/^sometar\/some\/jars/, /^sometar\/other/]
+        expect3  = ['somefile', 'somefile', 'file']
+
+        filelist = entries.reject { |entry| FileDependencies::Archive.extract_file?(entry, nil, extract3, prefix) == false }.map { |entry| entry.gsub(prefix, '').split("/").last }
+        expect(filelist.sort).to(eq(expect3.sort))
+      end
+
+    end
+
+  end
+
+end

data/spec/{file_spec.rb → file-dependencies/file_spec.rb}

@@ -3,7 +3,7 @@ require 'file-dependencies/file'
 include WebMock::API
 describe FileDependencies::File do
 
-  describe ".calc_sha1" do
+  describe ".calculate_sha1" do
 
     after do 
       ::File.unlink(file)
@@ -11,11 +11,11 @@ describe FileDependencies::File do
    
     let(:file) { Assist.generate_file('some_content') }
     it 'gives back correct sha1 value' do
-      expect(FileDependencies::File.calc_sha1(file)).to(eq('778164c23fae5935176254d2550619cba8abc262'))
+      expect(FileDependencies::File.calculate_sha1(file)).to(eq('778164c23fae5935176254d2550619cba8abc262'))
     end
 
     it 'raises an error when the file doesnt exist' do
-      expect { FileDependencies::File.calc_sha1('dont_exist') }.to(raise_error(Errno::ENOENT))
+      expect { FileDependencies::File.calculate_sha1('dont_exist') }.to(raise_error(Errno::ENOENT))
     end
   end
 

data/spec/file-dependencies_spec.rb

@@ -5,7 +5,7 @@ require 'file-dependencies'
 include WebMock::API
 describe FileDependencies do
 
-  describe '.download' do
+  describe '.process_downloads' do
     after do
       [tmpdir, target, file1, file2, file3, file4].each do |entry|
         FileUtils.remove_entry_secure(entry)
@@ -18,30 +18,34 @@ describe FileDependencies do
     let(:file1) { Assist.generate_tarball('some/file' => 'content1', 'some/other/file' => 'content2', 'other' => 'content3') }
     let(:file2) { Assist.generate_file('some_content') }
     let(:file3) { Assist.generate_gzip('some_content_for_gzip') }
-    let(:file4) { Assist.generate_tarball('jars/some.jar' => 'content10', 'jars/someother.jar' => 'content11') }
+    let(:file4) { Assist.generate_tarball('jars/some.jar' => 'content10', 'jars/someother.jar' => 'content11', 'somefile.txt' => 'bla') }
+    let(:file5) { Assist.generate_tarball('src/types.db' => 'typesdb') }
 
-    let(:sha1) { FileDependencies::File.calc_sha1(file1) }
-    let(:sha2) { FileDependencies::File.calc_sha1(file2) }
-    let(:sha3) { FileDependencies::File.calc_sha1(file3) }
-    let(:sha4) { FileDependencies::File.calc_sha1(file4) }
+    let(:sha1) { FileDependencies::File.calculate_sha1(file1) }
+    let(:sha2) { FileDependencies::File.calculate_sha1(file2) }
+    let(:sha3) { FileDependencies::File.calculate_sha1(file3) }
+    let(:sha4) { FileDependencies::File.calculate_sha1(file4) }
+    let(:sha5) { FileDependencies::File.calculate_sha1(file5) }
 
     let(:url1) { 'http://www.example.com/somefile1.tar.gz' }
     let(:url2) { 'http://www.example.com/somefile2.txt' }
     let(:url3) { 'http://www.example.com/somefile3.gz' }
     let(:url4) { 'http://www.example.com/somefile4.tar.gz' }
+    let(:url5) { 'http://www.example.com/somefile5.tar.gz' }
 
-    let(:entries) { ['somefile2.txt', 'somefile3', 'some/file', 'some/other/file', 'other', 'jars/some.jar', 'jars/someother.jar'] }
+    let(:entries) { ['somefile2.txt', 'somefile3', 'some/file', 'some/other/file', 'other', 'jarfiles/jars/some.jar', 'jarfiles/jars/someother.jar', 'jarfiles/somefile.txt', 'types.db'] }
 
-    let(:files) { [{ 'url' => url1, 'sha1' => sha1 }, { 'url' => url2, 'sha1' => sha2 }, { 'url' => url3, 'sha1' => sha3 }, { 'url' => url4, 'sha1' => sha4, 'extract' => '.jar', 'target' => 'jars' }] }
+    let(:files) { [{ 'url' => url1, 'sha1' => sha1 }, { 'url' => url2, 'sha1' => sha2 }, { 'url' => url3, 'sha1' => sha3 }, { 'url' => url4, 'sha1' => sha4, 'extract' => [/\.jar/, /\.txt/], 'target' => 'jarfiles' }, { 'url' => url5, 'sha1' => sha5, 'flatten' => true }] }
 
     it 'processes file list' do
       stub_request(:get, url1).to_return(:body => File.new(file1), :status => 200)
       stub_request(:get, url2).to_return(:body => File.new(file2), :status => 200)
       stub_request(:get, url3).to_return(:body => File.new(file3), :status => 200)
       stub_request(:get, url4).to_return(:body => File.new(file4), :status => 200)
+      stub_request(:get, url5).to_return(:body => File.new(file5), :status => 200)
      
       # we should not have any errors
-      expect { FileDependencies.download(files, target, tmpdir) }.to_not(raise_error)
+      expect { FileDependencies.process_downloads(files, target, tmpdir) }.to_not(raise_error)
 
       # check if we got all the expected files
       found_files = Dir.glob(File.join(target, '**', '*')).reject { |entry| File.directory?(entry) }.sort
@@ -64,27 +68,32 @@ describe FileDependencies do
     let(:file1) { Assist.generate_tarball('some/file' => 'content1', 'some/other/file' => 'content2', 'other' => 'content3') }
     let(:file2) { Assist.generate_file('some_content') }
     let(:file3) { Assist.generate_gzip('some_content_for_gzip') }
-    let(:file4) { Assist.generate_tarball('jars/some.jar' => 'content10', 'jars/someother.jar' => 'content11') }
+    let(:file4) { Assist.generate_tarball('jars/some.jar' => 'content10', 'jars/someother.jar' => 'content11', 'somefile.txt' => 'bla') }
+    let(:file5) { Assist.generate_tarball('src/types.db' => 'typesdb') }
 
-    let(:sha1) { FileDependencies::File.calc_sha1(file1) }
-    let(:sha2) { FileDependencies::File.calc_sha1(file2) }
-    let(:sha3) { FileDependencies::File.calc_sha1(file3) }
-    let(:sha4) { FileDependencies::File.calc_sha1(file4) }
+    let(:sha1) { FileDependencies::File.calculate_sha1(file1) }
+    let(:sha2) { FileDependencies::File.calculate_sha1(file2) }
+    let(:sha3) { FileDependencies::File.calculate_sha1(file3) }
+    let(:sha4) { FileDependencies::File.calculate_sha1(file4) }
+    let(:sha5) { FileDependencies::File.calculate_sha1(file5) }
 
     let(:url1) { 'http://www.example.com/somefile1.tar.gz' }
     let(:url2) { 'http://www.example.com/somefile2.txt' }
     let(:url3) { 'http://www.example.com/somefile3.gz' }
     let(:url4) { 'http://www.example.com/somefile4.tar.gz' }
+    let(:url5) { 'http://www.example.com/somefile5.tar.gz' }
+
+    let(:files) { [{ 'url' => url1, 'sha1' => sha1 }, { 'url' => url2, 'sha1' => sha2 }, { 'url' => url3, 'sha1' => sha3 }, { 'url' => url4, 'sha1' => sha4, 'extract' => [/\.jar/, /\.txt/], 'target' => 'jarfiles' }, { 'url' => url5, 'sha1' => sha5, 'flatten' => true }].to_json }
 
-    let(:files) { [{ 'url' => url1, 'sha1' => sha1 }, { 'url' => url2, 'sha1' => sha2 }, { 'url' => url3, 'sha1' => sha3 }, { 'url' => url4, 'sha1' => sha4, 'extract' => '.jar', 'target' => 'jars' }].to_json }
     let(:vendorfile) { File.write(File.join(target, 'vendor.json'), files) }
-    let(:entries) { ['somefile2.txt', 'somefile3', 'some/file', 'some/other/file', 'other', 'jars/some.jar', 'jars/someother.jar'] }
+    let(:entries) { ['somefile2.txt', 'somefile3', 'some/file', 'some/other/file', 'other', 'jarfiles/jars/some.jar', 'jarfiles/jars/someother.jar', 'jarfiles/somefile.txt', 'types.db'] }
 
     it 'processes the vendor.json file' do
       stub_request(:get, url1).to_return(:body => File.new(file1), :status => 200)
       stub_request(:get, url2).to_return(:body => File.new(file2), :status => 200)
       stub_request(:get, url3).to_return(:body => File.new(file3), :status => 200)
       stub_request(:get, url4).to_return(:body => File.new(file4), :status => 200)
+      stub_request(:get, url5).to_return(:body => File.new(file5), :status => 200)
       File.write(File.join(target, 'vendor.json'), files)
      
       # we should not have any errors

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: file-dependencies
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Richard Pijnenburg
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-12-15 00:00:00.000000000 Z
+date: 2015-01-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitar
@@ -127,11 +127,11 @@ files:
 - lib/file-dependencies/file.rb
 - lib/file-dependencies/gem.rb
 - lib/file-dependencies/rake_tasks.rb
-- lib/file-dependencies/rubygems_plugin.rb
 - lib/file-dependencies/version.rb
-- spec/archive_spec.rb
+- lib/rubygems_plugin.rb
+- spec/file-dependencies/archive_spec.rb
+- spec/file-dependencies/file_spec.rb
 - spec/file-dependencies_spec.rb
-- spec/file_spec.rb
 - spec/spec_assist.rb
 - spec/spec_helper.rb
 homepage: https://github.com/electrical/file-dependencies

data/spec/archive_spec.rb

@@ -1,94 +0,0 @@
-require 'spec_helper'
-require 'file-dependencies/archive'
-
-describe FileDependencies::Archive do
-
-  describe ".ungzip" do
-
-    after do
-      FileUtils.remove_entry_secure(gzipfile) if ::File.exist?(gzipfile)
-      FileUtils.remove_entry_secure(expected_file)
-      FileUtils.remove_entry_secure(tmpdir)
-    end
-    let(:gzipfile) { Assist.generate_gzip('some_content') }
-    let(:expected_file) { gzipfile.gsub('.gz', '') }
-    let(:tmpdir) { Stud::Temporary.directory }
-
-    it 'decompresses a gzip file'do
-      expect { FileDependencies::Archive.ungzip(gzipfile, tmpdir) }.to_not(raise_error)
-      expect(File.exist?(expected_file))
-    end
-
-    let(:expected_file) { Assist.generate_file('some_content') }
-    it 'raises error extracting non gzip file' do
-      expect { FileDependencies::Archive.ungzip(expected_file, tmpdir) }.to(raise_error(Zlib::GzipFile::Error))
-    end
-  end
-
-  describe ".untar" do
-
-    after do
-      FileUtils.remove_entry_secure(tmpdir)
-      FileUtils.remove_entry_secure(file)
-    end
-
-    let(:file) { Assist.generate_file('some_content') }
-    let(:tarball) { Assist.generate_tarball('some/file' => 'content1', 'some/other/file' => 'content2', 'other' => 'content3') }
-    let(:tmpdir) { Stud::Temporary.directory }
-
-    it 'extracts a full tarball' do
-      entries = ['some/file', 'some/other/file', 'other']
-
-      FileDependencies::Archive.untar(tarball) do |entry|
-        ::File.join(tmpdir, entry.full_name)
-      end
-      found_files = Dir.glob(File.join(tmpdir, '**', '*')).reject { |entry| File.directory?(entry) }.sort
-      expected_files = entries.map { |k| ::File.join(tmpdir, k) }.sort
-      expect(expected_files).to(eq(found_files))
-    end
-
-    it 'extracts some files' do
-      entries = ['some/file', 'some/other/file']
-
-      FileDependencies::Archive.untar(tarball) do |entry|
-        if entries.include?(entry.full_name)
-          ::File.join(tmpdir, entry.full_name)
-        end
-      end
-      found_files = Dir.glob(File.join(tmpdir, '**', '*')).reject { |entry| File.directory?(entry) }.sort
-      expected_files = entries.map { |k| ::File.join(tmpdir, k) }.sort
-      expect(expected_files).to(eq(found_files))
-    end
-
-    it 'raises error when invalid file is provided' do
-      expect do
-        FileDependencies::Archive.untar(file) do |entry|
-          ::File.join(tmpdir, entry.full_name)
-        end
-      end.to(raise_error(Zlib::GzipFile::Error))
-    end
-  end
-
-  describe ".eval_file" do
-
-    let(:entries) { ['sometar/PaxHeaders', 'sometar/some/dir/PaxHeaders', 'sometar/some/dir/somefile', 'sometar/somefile', 'sometar/some/other/file', 'sometar/some/jars/file1.jar', 'sometar/some/jars/file2.jar', 'sometar/other/jars/file3.jar'] }
-    let(:prefix) { 'sometar' }
-
-    let(:extract1) { '.jars' }
-    let(:expect1)  { ['file1.jar', 'file2.jar', 'file3.jar'] }
-    let(:extract2) { ['/some/other/file', '/somefile', '/other/jars/file3.jar'] }
-    let(:expect2)  { ['file', 'somefile', 'file3.jar'] }
-
-    it 'returns all files based on a wildcard' do
-      filelist = entries.reject { |entry| FileDependencies::Archive.eval_file(entry, extract1, prefix) == false }.map { |entry| entry.gsub(prefix, '').split("/").last }
-      expect(filelist.sort).to(eq(expect1.sort))
-    end
-
-    it 'returns all files based on an array' do
-      filelist = entries.reject { |entry| FileDependencies::Archive.eval_file(entry, extract2, prefix) == false }.map { |entry| entry.gsub(prefix, '').split("/").last }
-      expect(filelist.sort).to(eq(expect2.sort))
-    end
-
-  end
-
-end