rubyzip 1.3.0 → 2.0.0

data/test/errors_test.rb

@@ -1,35 +0,0 @@
-# encoding: utf-8
-
-require 'test_helper'
-
-class ErrorsTest < MiniTest::Test
-  def test_rescue_legacy_zip_error
-    raise ::Zip::Error
-  rescue ::Zip::ZipError
-  end
-
-  def test_rescue_legacy_zip_entry_exists_error
-    raise ::Zip::EntryExistsError
-  rescue ::Zip::ZipEntryExistsError
-  end
-
-  def test_rescue_legacy_zip_destination_file_exists_error
-    raise ::Zip::DestinationFileExistsError
-  rescue ::Zip::ZipDestinationFileExistsError
-  end
-
-  def test_rescue_legacy_zip_compression_method_error
-    raise ::Zip::CompressionMethodError
-  rescue ::Zip::ZipCompressionMethodError
-  end
-
-  def test_rescue_legacy_zip_entry_name_error
-    raise ::Zip::EntryNameError
-  rescue ::Zip::ZipEntryNameError
-  end
-
-  def test_rescue_legacy_zip_internal_error
-    raise ::Zip::InternalError
-  rescue ::Zip::ZipInternalError
-  end
-end

data/test/extra_field_test.rb

@@ -1,76 +0,0 @@
-require 'test_helper'
-
-class ZipExtraFieldTest < MiniTest::Test
-  def test_new
-    extra_pure = ::Zip::ExtraField.new('')
-    extra_withstr = ::Zip::ExtraField.new('foo')
-    assert_instance_of(::Zip::ExtraField, extra_pure)
-    assert_instance_of(::Zip::ExtraField, extra_withstr)
-  end
-
-  def test_unknownfield
-    extra = ::Zip::ExtraField.new('foo')
-    assert_equal(extra['Unknown'], 'foo')
-    extra.merge('a')
-    assert_equal(extra['Unknown'], 'fooa')
-    extra.merge('barbaz')
-    assert_equal(extra.to_s, 'fooabarbaz')
-  end
-
-  def test_ntfs
-    str = "\x0A\x00 \x00\x00\x00\x00\x00\x01\x00\x18\x00\xC0\x81\x17\xE8B\xCE\xCF\x01\xC0\x81\x17\xE8B\xCE\xCF\x01\xC0\x81\x17\xE8B\xCE\xCF\x01"
-    extra = ::Zip::ExtraField.new(str)
-    assert(extra.member?('NTFS'))
-    t = ::Zip::DOSTime.at(1_410_496_497.405178)
-    assert_equal(t, extra['NTFS'].mtime)
-    assert_equal(t, extra['NTFS'].atime)
-    assert_equal(t, extra['NTFS'].ctime)
-  end
-
-  def test_merge
-    str = "UT\x5\0\x3\250$\r@Ux\0\0"
-    extra1 = ::Zip::ExtraField.new('')
-    extra2 = ::Zip::ExtraField.new(str)
-    assert(!extra1.member?('UniversalTime'))
-    assert(extra2.member?('UniversalTime'))
-    extra1.merge(str)
-    assert_equal(extra1['UniversalTime'].mtime, extra2['UniversalTime'].mtime)
-  end
-
-  def test_length
-    str = "UT\x5\0\x3\250$\r@Ux\0\0Te\0\0testit"
-    extra = ::Zip::ExtraField.new(str)
-    assert_equal(extra.local_size, extra.to_local_bin.size)
-    assert_equal(extra.c_dir_size, extra.to_c_dir_bin.size)
-    extra.merge('foo')
-    assert_equal(extra.local_size, extra.to_local_bin.size)
-    assert_equal(extra.c_dir_size, extra.to_c_dir_bin.size)
-  end
-
-  def test_to_s
-    str = "UT\x5\0\x3\250$\r@Ux\0\0Te\0\0testit"
-    extra = ::Zip::ExtraField.new(str)
-    assert_instance_of(String, extra.to_s)
-
-    s = extra.to_s
-    extra.merge('foo')
-    assert_equal(s.length + 3, extra.to_s.length)
-  end
-
-  def test_equality
-    str = "UT\x5\0\x3\250$\r@"
-    extra1 = ::Zip::ExtraField.new(str)
-    extra2 = ::Zip::ExtraField.new(str)
-    extra3 = ::Zip::ExtraField.new(str)
-    assert_equal(extra1, extra2)
-
-    extra2['UniversalTime'].mtime = ::Zip::DOSTime.now
-    assert(extra1 != extra2)
-
-    extra3.create('IUnix')
-    assert(extra1 != extra3)
-
-    extra1.create('IUnix')
-    assert_equal(extra1, extra3)
-  end
-end

data/test/file_extract_directory_test.rb

@@ -1,54 +0,0 @@
-require 'test_helper'
-
-class ZipFileExtractDirectoryTest < MiniTest::Test
-  include CommonZipFileFixture
-
-  TEST_OUT_NAME = 'test/data/generated/emptyOutDir'
-
-  def open_zip(&aProc)
-    assert(!aProc.nil?)
-    ::Zip::File.open(TestZipFile::TEST_ZIP4.zip_name, &aProc)
-  end
-
-  def extract_test_dir(&aProc)
-    open_zip do |zf|
-      zf.extract(TestFiles::EMPTY_TEST_DIR, TEST_OUT_NAME, &aProc)
-    end
-  end
-
-  def setup
-    super
-
-    Dir.rmdir(TEST_OUT_NAME) if File.directory? TEST_OUT_NAME
-    File.delete(TEST_OUT_NAME) if File.exist? TEST_OUT_NAME
-  end
-
-  def test_extract_directory
-    extract_test_dir
-    assert(File.directory?(TEST_OUT_NAME))
-  end
-
-  def test_extract_directory_exists_as_dir
-    Dir.mkdir TEST_OUT_NAME
-    extract_test_dir
-    assert(File.directory?(TEST_OUT_NAME))
-  end
-
-  def test_extract_directory_exists_as_file
-    File.open(TEST_OUT_NAME, 'w') { |f| f.puts 'something' }
-    assert_raises(::Zip::DestinationFileExistsError) { extract_test_dir }
-  end
-
-  def test_extract_directory_exists_as_file_overwrite
-    File.open(TEST_OUT_NAME, 'w') { |f| f.puts 'something' }
-    gotCalled = false
-    extract_test_dir do |entry, destPath|
-      gotCalled = true
-      assert_equal(TEST_OUT_NAME, destPath)
-      assert(entry.directory?)
-      true
-    end
-    assert(gotCalled)
-    assert(File.directory?(TEST_OUT_NAME))
-  end
-end

data/test/file_extract_test.rb

@@ -1,145 +0,0 @@
-require 'test_helper'
-
-class ZipFileExtractTest < MiniTest::Test
-  include CommonZipFileFixture
-  EXTRACTED_FILENAME = 'test/data/generated/extEntry'
-  ENTRY_TO_EXTRACT, *REMAINING_ENTRIES = TEST_ZIP.entry_names.reverse
-
-  def setup
-    super
-    ::File.delete(EXTRACTED_FILENAME) if ::File.exist?(EXTRACTED_FILENAME)
-  end
-
-  def teardown
-    ::Zip.reset!
-  end
-
-  def test_extract
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      zf.extract(ENTRY_TO_EXTRACT, EXTRACTED_FILENAME)
-
-      assert(File.exist?(EXTRACTED_FILENAME))
-      AssertEntry.assert_contents(EXTRACTED_FILENAME,
-                                  zf.get_input_stream(ENTRY_TO_EXTRACT) { |is| is.read })
-
-      ::File.unlink(EXTRACTED_FILENAME)
-
-      entry = zf.get_entry(ENTRY_TO_EXTRACT)
-      entry.extract(EXTRACTED_FILENAME)
-
-      assert(File.exist?(EXTRACTED_FILENAME))
-      AssertEntry.assert_contents(EXTRACTED_FILENAME,
-                                  entry.get_input_stream { |is| is.read })
-    end
-  end
-
-  def test_extract_exists
-    writtenText = 'written text'
-    ::File.open(EXTRACTED_FILENAME, 'w') { |f| f.write(writtenText) }
-
-    assert_raises(::Zip::DestinationFileExistsError) do
-      ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-        zf.extract(zf.entries.first, EXTRACTED_FILENAME)
-      end
-    end
-    File.open(EXTRACTED_FILENAME, 'r') do |f|
-      assert_equal(writtenText, f.read)
-    end
-  end
-
-  def test_extract_exists_overwrite
-    writtenText = 'written text'
-    ::File.open(EXTRACTED_FILENAME, 'w') { |f| f.write(writtenText) }
-
-    gotCalledCorrectly = false
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      zf.extract(zf.entries.first, EXTRACTED_FILENAME) do |entry, extractLoc|
-        gotCalledCorrectly = zf.entries.first == entry &&
-                             extractLoc == EXTRACTED_FILENAME
-        true
-      end
-    end
-
-    assert(gotCalledCorrectly)
-    ::File.open(EXTRACTED_FILENAME, 'r') do |f|
-      assert(writtenText != f.read)
-    end
-  end
-
-  def test_extract_non_entry
-    zf = ::Zip::File.new(TEST_ZIP.zip_name)
-    assert_raises(Errno::ENOENT) { zf.extract('nonExistingEntry', 'nonExistingEntry') }
-  ensure
-    zf.close if zf
-  end
-
-  def test_extract_non_entry_2
-    outFile = 'outfile'
-    assert_raises(Errno::ENOENT) do
-      zf = ::Zip::File.new(TEST_ZIP.zip_name)
-      nonEntry = 'hotdog-diddelidoo'
-      assert(!zf.entries.include?(nonEntry))
-      zf.extract(nonEntry, outFile)
-      zf.close
-    end
-    assert(!File.exist?(outFile))
-  end
-
-  def test_extract_incorrect_size
-    # The uncompressed size fields in the zip file cannot be trusted. This makes
-    # it harder for callers to validate the sizes of the files they are
-    # extracting, which can lead to denial of service. See also
-    # https://en.wikipedia.org/wiki/Zip_bomb
-    Dir.mktmpdir do |tmp|
-      real_zip = File.join(tmp, 'real.zip')
-      fake_zip = File.join(tmp, 'fake.zip')
-      file_name = 'a'
-      true_size = 500_000
-      fake_size = 1
-
-      ::Zip::File.open(real_zip, ::Zip::File::CREATE) do |zf|
-        zf.get_output_stream(file_name) do |os|
-          os.write 'a' * true_size
-        end
-      end
-
-      compressed_size = nil
-      ::Zip::File.open(real_zip) do |zf|
-        a_entry = zf.find_entry(file_name)
-        compressed_size = a_entry.compressed_size
-        assert_equal true_size, a_entry.size
-      end
-
-      true_size_bytes = [compressed_size, true_size, file_name.size].pack('LLS')
-      fake_size_bytes = [compressed_size, fake_size, file_name.size].pack('LLS')
-
-      data = File.binread(real_zip)
-      assert data.include?(true_size_bytes)
-      data.gsub! true_size_bytes, fake_size_bytes
-
-      File.open(fake_zip, 'wb') do |file|
-        file.write data
-      end
-
-      Dir.chdir tmp do
-        ::Zip::File.open(fake_zip) do |zf|
-          a_entry = zf.find_entry(file_name)
-          assert_equal fake_size, a_entry.size
-
-          ::Zip.validate_entry_sizes = false
-          a_entry.extract
-          assert_equal true_size, File.size(file_name)
-          FileUtils.rm file_name
-
-          ::Zip.validate_entry_sizes = true
-          error = assert_raises ::Zip::EntrySizeError do
-            a_entry.extract
-          end
-          assert_equal \
-            'Entry a should be 1B but is larger when inflated',
-            error.message
-        end
-      end
-    end
-  end
-end

data/test/file_permissions_test.rb

@@ -1,65 +0,0 @@
-require 'test_helper'
-
-class FilePermissionsTest < MiniTest::Test
-  ZIPNAME = File.join(File.dirname(__FILE__), 'umask.zip')
-  FILENAME = File.join(File.dirname(__FILE__), 'umask.txt')
-
-  def teardown
-    ::File.unlink(ZIPNAME)
-    ::File.unlink(FILENAME)
-  end
-
-  def test_current_umask
-    create_files
-    assert_matching_permissions FILENAME, ZIPNAME
-  end
-
-  def test_umask_000
-    set_umask(0o000) do
-      create_files
-    end
-
-    assert_matching_permissions FILENAME, ZIPNAME
-  end
-
-  def test_umask_066
-    set_umask(0o066) do
-      create_files
-    end
-
-    assert_matching_permissions FILENAME, ZIPNAME
-  end
-
-  def test_umask_027
-    set_umask(0o027) do
-      create_files
-    end
-
-    assert_matching_permissions FILENAME, ZIPNAME
-  end
-
-  def assert_matching_permissions(expected_file, actual_file)
-    assert_equal(
-      ::File.stat(expected_file).mode.to_s(8).rjust(4, '0'),
-      ::File.stat(actual_file).mode.to_s(8).rjust(4, '0')
-    )
-  end
-
-  def create_files
-    ::Zip::File.open(ZIPNAME, ::Zip::File::CREATE) do |zip|
-      zip.comment = 'test'
-    end
-
-    ::File.open(FILENAME, 'w') do |file|
-      file << 'test'
-    end
-  end
-
-  # If anything goes wrong, make sure the umask is restored.
-  def set_umask(umask)
-    saved_umask = ::File.umask(umask)
-    yield
-  ensure
-    ::File.umask(saved_umask)
-  end
-end

data/test/file_split_test.rb

@@ -1,57 +0,0 @@
-require 'test_helper'
-
-class ZipFileSplitTest < MiniTest::Test
-  TEST_ZIP = TestZipFile::TEST_ZIP2.clone
-  TEST_ZIP.zip_name = 'large_zip_file.zip'
-  EXTRACTED_FILENAME = 'test/data/generated/extEntrySplit'
-  UNSPLITTED_FILENAME = 'test/data/generated/unsplitted.zip'
-  ENTRY_TO_EXTRACT = TEST_ZIP.entry_names.first
-
-  def setup
-    FileUtils.cp(TestZipFile::TEST_ZIP2.zip_name, TEST_ZIP.zip_name)
-  end
-
-  def teardown
-    File.delete(TEST_ZIP.zip_name)
-    File.delete(UNSPLITTED_FILENAME) if File.exist?(UNSPLITTED_FILENAME)
-
-    Dir["#{TEST_ZIP.zip_name}.*"].each do |zip_file_name|
-      File.delete(zip_file_name) if File.exist?(zip_file_name)
-    end
-  end
-
-  def test_split_method_respond
-    assert_respond_to ::Zip::File, :split, 'Does not have split class method'
-  end
-
-  def test_split
-    result = ::Zip::File.split(TEST_ZIP.zip_name, 65_536, false)
-
-    return if result.nil?
-    Dir["#{TEST_ZIP.zip_name}.*"].sort.each_with_index do |zip_file_name, index|
-      File.open(zip_file_name, 'rb') do |zip_file|
-        zip_file.read([::Zip::File::SPLIT_SIGNATURE].pack('V').size) if index == 0
-        File.open(UNSPLITTED_FILENAME, 'ab') do |file|
-          file << zip_file.read
-        end
-      end
-    end
-
-    ::Zip::File.open(UNSPLITTED_FILENAME) do |zf|
-      zf.extract(ENTRY_TO_EXTRACT, EXTRACTED_FILENAME)
-
-      assert(File.exist?(EXTRACTED_FILENAME))
-      AssertEntry.assert_contents(EXTRACTED_FILENAME,
-                                  zf.get_input_stream(ENTRY_TO_EXTRACT) { |is| is.read })
-
-      File.unlink(EXTRACTED_FILENAME)
-
-      entry = zf.get_entry(ENTRY_TO_EXTRACT)
-      entry.extract(EXTRACTED_FILENAME)
-
-      assert(File.exist?(EXTRACTED_FILENAME))
-      AssertEntry.assert_contents(EXTRACTED_FILENAME,
-                                  entry.get_input_stream { |is| is.read })
-    end
-  end
-end

data/test/file_test.rb

@@ -1,666 +0,0 @@
-require 'test_helper'
-
-class ZipFileTest < MiniTest::Test
-  include CommonZipFileFixture
-  include ZipEntryData
-
-  OK_DELETE_FILE = 'test/data/generated/okToDelete.txt'
-  OK_DELETE_MOVED_FILE = 'test/data/generated/okToDeleteMoved.txt'
-
-  def teardown
-    ::Zip.write_zip64_support = false
-  end
-
-  def test_create_from_scratch_to_buffer
-    comment = 'a short comment'
-
-    buffer = ::Zip::File.add_buffer do |zf|
-      zf.get_output_stream('myFile') { |os| os.write 'myFile contains just this' }
-      zf.mkdir('dir1')
-      zf.comment = comment
-    end
-
-    ::File.open(EMPTY_FILENAME, 'wb') { |file| file.write buffer.string }
-
-    zfRead = ::Zip::File.new(EMPTY_FILENAME)
-    assert_equal(comment, zfRead.comment)
-    assert_equal(2, zfRead.entries.length)
-  end
-
-  def test_create_from_scratch
-    comment = 'a short comment'
-
-    zf = ::Zip::File.new(EMPTY_FILENAME, ::Zip::File::CREATE)
-    zf.get_output_stream('myFile') { |os| os.write 'myFile contains just this' }
-    zf.mkdir('dir1')
-    zf.comment = comment
-    zf.close
-
-    zfRead = ::Zip::File.new(EMPTY_FILENAME)
-    assert_equal(comment, zfRead.comment)
-    assert_equal(2, zfRead.entries.length)
-  end
-
-  def test_create_from_scratch_with_old_create_parameter
-    comment = 'a short comment'
-
-    zf = ::Zip::File.new(EMPTY_FILENAME, 1)
-    zf.get_output_stream('myFile') { |os| os.write 'myFile contains just this' }
-    zf.mkdir('dir1')
-    zf.comment = comment
-    zf.close
-
-    zfRead = ::Zip::File.new(EMPTY_FILENAME)
-    assert_equal(comment, zfRead.comment)
-    assert_equal(2, zfRead.entries.length)
-  end
-
-  def test_get_input_stream_stored_with_gpflag_bit3
-    ::Zip::File.open('test/data/gpbit3stored.zip') do |zf|
-      assert_equal("foo\n", zf.read("foo.txt"))
-    end
-  end
-
-  def test_get_output_stream
-    entryCount = nil
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      entryCount = zf.size
-      zf.get_output_stream('newEntry.txt') do |os|
-        os.write 'Putting stuff in newEntry.txt'
-      end
-      assert_equal(entryCount + 1, zf.size)
-      assert_equal('Putting stuff in newEntry.txt', zf.read('newEntry.txt'))
-
-      zf.get_output_stream(zf.get_entry('test/data/generated/empty.txt')) do |os|
-        os.write 'Putting stuff in data/generated/empty.txt'
-      end
-      assert_equal(entryCount + 1, zf.size)
-      assert_equal('Putting stuff in data/generated/empty.txt', zf.read('test/data/generated/empty.txt'))
-
-      custom_entry_args = [TEST_COMMENT, TEST_EXTRA, TEST_COMPRESSED_SIZE, TEST_CRC, ::Zip::Entry::STORED, TEST_SIZE, TEST_TIME]
-      zf.get_output_stream('entry_with_custom_args.txt', nil, *custom_entry_args) do |os|
-        os.write 'Some data'
-      end
-      assert_equal(entryCount + 2, zf.size)
-      entry = zf.get_entry('entry_with_custom_args.txt')
-      assert_equal(custom_entry_args[0], entry.comment)
-      assert_equal(custom_entry_args[2], entry.compressed_size)
-      assert_equal(custom_entry_args[3], entry.crc)
-      assert_equal(custom_entry_args[4], entry.compression_method)
-      assert_equal(custom_entry_args[5], entry.size)
-      assert_equal(custom_entry_args[6], entry.time)
-
-      zf.get_output_stream('entry.bin') do |os|
-        os.write(::File.open('test/data/generated/5entry.zip', 'rb').read)
-      end
-    end
-
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      assert_equal(entryCount + 3, zf.size)
-      assert_equal('Putting stuff in newEntry.txt', zf.read('newEntry.txt'))
-      assert_equal('Putting stuff in data/generated/empty.txt', zf.read('test/data/generated/empty.txt'))
-      assert_equal(File.open('test/data/generated/5entry.zip', 'rb').read, zf.read('entry.bin'))
-    end
-  end
-
-  def test_open_buffer_with_string
-    string = File.read('test/data/rubycode.zip')
-    ::Zip::File.open_buffer string do |zf|
-      assert zf.entries.map { |e| e.name }.include?('zippedruby1.rb')
-    end
-  end
-
-  def test_open_buffer_with_stringio
-    string_io = StringIO.new File.read('test/data/rubycode.zip')
-    ::Zip::File.open_buffer string_io do |zf|
-      assert zf.entries.map { |e| e.name }.include?('zippedruby1.rb')
-    end
-  end
-
-  def test_close_buffer_with_stringio
-    string_io = StringIO.new File.read('test/data/rubycode.zip')
-    zf = ::Zip::File.open_buffer string_io
-    assert_nil zf.close
-  end
-
-  def test_open_buffer_no_op_does_not_change_file
-    Dir.mktmpdir do |tmp|
-      test_zip = File.join(tmp, 'test.zip')
-      FileUtils.cp 'test/data/rubycode.zip', test_zip
-
-      # Note: this may change the file if it is opened with r+b instead of rb.
-      # The 'extra fields' in this particular zip file get reordered.
-      File.open(test_zip, 'rb') do |file|
-        Zip::File.open_buffer(file) do |zf|
-          nil # do nothing
-        end
-      end
-
-      assert_equal \
-        File.binread('test/data/rubycode.zip'),
-        File.binread(test_zip)
-    end
-  end
-
-  def test_open_buffer_close_does_not_change_file
-    Dir.mktmpdir do |tmp|
-      test_zip = File.join(tmp, 'test.zip')
-      FileUtils.cp 'test/data/rubycode.zip', test_zip
-
-      File.open(test_zip, 'rb') do |file|
-        zf = Zip::File.open_buffer(file)
-        refute zf.commit_required?
-        assert_nil zf.close
-      end
-
-      assert_equal \
-        File.binread('test/data/rubycode.zip'),
-        File.binread(test_zip)
-    end
-  end
-
-  def test_open_buffer_with_io_and_block
-    File.open('test/data/rubycode.zip') do |io|
-      io.set_encoding(Encoding::BINARY) # not strictly required but can be set
-      Zip::File.open_buffer(io) do |zip_io|
-        # left empty on purpose
-      end
-    end
-  end
-
-  def test_open_buffer_without_block
-    string_io = StringIO.new File.read('test/data/rubycode.zip')
-    zf = ::Zip::File.open_buffer string_io
-    assert zf.entries.map { |e| e.name }.include?('zippedruby1.rb')
-  end
-
-  def test_cleans_up_tempfiles_after_close
-    zf = ::Zip::File.new(EMPTY_FILENAME, ::Zip::File::CREATE)
-    zf.get_output_stream('myFile') do |os|
-      @tempfile_path = os.path
-      os.write 'myFile contains just this'
-    end
-
-    assert_equal(true, File.exist?(@tempfile_path))
-
-    zf.close
-
-    assert_equal(false, File.exist?(@tempfile_path))
-  end
-
-  def test_add
-    srcFile = 'test/data/file2.txt'
-    entryName = 'newEntryName.rb'
-    assert(::File.exist?(srcFile))
-    zf = ::Zip::File.new(EMPTY_FILENAME, ::Zip::File::CREATE)
-    zf.add(entryName, srcFile)
-    zf.close
-
-    zfRead = ::Zip::File.new(EMPTY_FILENAME)
-    assert_equal('', zfRead.comment)
-    assert_equal(1, zfRead.entries.length)
-    assert_equal(entryName, zfRead.entries.first.name)
-    AssertEntry.assert_contents(srcFile,
-                                zfRead.get_input_stream(entryName) { |zis| zis.read })
-  end
-
-  def test_add_stored
-    srcFile = 'test/data/file2.txt'
-    entryName = 'newEntryName.rb'
-    assert(::File.exist?(srcFile))
-    zf = ::Zip::File.new(EMPTY_FILENAME, ::Zip::File::CREATE)
-    zf.add_stored(entryName, srcFile)
-    zf.close
-
-    zfRead = ::Zip::File.new(EMPTY_FILENAME)
-    entry = zfRead.entries.first
-    assert_equal('', zfRead.comment)
-    assert_equal(1, zfRead.entries.length)
-    assert_equal(entryName, entry.name)
-    assert_equal(File.size(srcFile), entry.size)
-    assert_equal(entry.size, entry.compressed_size)
-    assert_equal(::Zip::Entry::STORED, entry.compression_method)
-    AssertEntry.assert_contents(srcFile,
-                                zfRead.get_input_stream(entryName) { |zis| zis.read })
-  end
-
-  def test_recover_permissions_after_add_files_to_archive
-    srcZip = TEST_ZIP.zip_name
-    ::File.chmod(0o664, srcZip)
-    srcFile = 'test/data/file2.txt'
-    entryName = 'newEntryName.rb'
-    assert_equal(::File.stat(srcZip).mode, 0o100664)
-    assert(::File.exist?(srcZip))
-    zf = ::Zip::File.new(srcZip, ::Zip::File::CREATE)
-    zf.add(entryName, srcFile)
-    zf.close
-    assert_equal(::File.stat(srcZip).mode, 0o100664)
-  end
-
-  def test_add_existing_entry_name
-    assert_raises(::Zip::EntryExistsError) do
-      ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-        zf.add(zf.entries.first.name, 'test/data/file2.txt')
-      end
-    end
-  end
-
-  def test_add_existing_entry_name_replace
-    gotCalled = false
-    replacedEntry = nil
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      replacedEntry = zf.entries.first.name
-      zf.add(replacedEntry, 'test/data/file2.txt') { gotCalled = true; true }
-    end
-    assert(gotCalled)
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      assert_contains(zf, replacedEntry, 'test/data/file2.txt')
-    end
-  end
-
-  def test_add_directory
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      zf.add(TestFiles::EMPTY_TEST_DIR, TestFiles::EMPTY_TEST_DIR)
-    end
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      dirEntry = zf.entries.detect { |e| e.name == TestFiles::EMPTY_TEST_DIR + '/' }
-      assert(dirEntry.directory?)
-    end
-  end
-
-  def test_remove
-    entryToRemove, *remainingEntries = TEST_ZIP.entry_names
-
-    FileUtils.cp(TestZipFile::TEST_ZIP2.zip_name, TEST_ZIP.zip_name)
-
-    zf = ::Zip::File.new(TEST_ZIP.zip_name)
-    assert(zf.entries.map { |e| e.name }.include?(entryToRemove))
-    zf.remove(entryToRemove)
-    assert(!zf.entries.map { |e| e.name }.include?(entryToRemove))
-    assert_equal(zf.entries.map { |x| x.name }.sort, remainingEntries.sort)
-    zf.close
-
-    zfRead = ::Zip::File.new(TEST_ZIP.zip_name)
-    assert(!zfRead.entries.map { |e| e.name }.include?(entryToRemove))
-    assert_equal(zfRead.entries.map { |x| x.name }.sort, remainingEntries.sort)
-    zfRead.close
-  end
-
-  def test_rename
-    entryToRename, * = TEST_ZIP.entry_names
-
-    zf = ::Zip::File.new(TEST_ZIP.zip_name)
-    assert(zf.entries.map { |e| e.name }.include?(entryToRename))
-
-    contents = zf.read(entryToRename)
-    newName = 'changed entry name'
-    assert(!zf.entries.map { |e| e.name }.include?(newName))
-
-    zf.rename(entryToRename, newName)
-    assert(zf.entries.map { |e| e.name }.include?(newName))
-
-    assert_equal(contents, zf.read(newName))
-
-    zf.close
-
-    zfRead = ::Zip::File.new(TEST_ZIP.zip_name)
-    assert(zfRead.entries.map { |e| e.name }.include?(newName))
-    assert_equal(contents, zfRead.read(newName))
-    zfRead.close
-  end
-
-  def test_rename_with_each
-    zf_name = 'test_rename_zip.zip'
-    ::File.unlink(zf_name) if ::File.exist?(zf_name)
-    arr = []
-    arr_renamed = []
-    ::Zip::File.open(zf_name, ::Zip::File::CREATE) do |zf|
-      zf.mkdir('test')
-      arr << 'test/'
-      arr_renamed << 'Ztest/'
-      %w[a b c d].each do |f|
-        zf.get_output_stream("test/#{f}") { |file| file.puts 'aaaa' }
-        arr << "test/#{f}"
-        arr_renamed << "Ztest/#{f}"
-      end
-    end
-    zf = ::Zip::File.open(zf_name)
-    assert_equal(zf.entries.map(&:name), arr)
-    zf.close
-    Zip::File.open(zf_name, 'wb') do |z|
-      z.each do |f|
-        z.rename(f, "Z#{f.name}")
-      end
-    end
-    zf = ::Zip::File.open(zf_name)
-    assert_equal(zf.entries.map(&:name), arr_renamed)
-    zf.close
-    ::File.unlink(zf_name) if ::File.exist?(zf_name)
-  end
-
-  def test_rename_to_existing_entry
-    oldEntries = nil
-    ::Zip::File.open(TEST_ZIP.zip_name) { |zf| oldEntries = zf.entries }
-
-    assert_raises(::Zip::EntryExistsError) do
-      ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-        zf.rename(zf.entries[0], zf.entries[1].name)
-      end
-    end
-
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      assert_equal(oldEntries.sort.map { |e| e.name }, zf.entries.sort.map { |e| e.name })
-    end
-  end
-
-  def test_rename_to_existing_entry_overwrite
-    oldEntries = nil
-    ::Zip::File.open(TEST_ZIP.zip_name) { |zf| oldEntries = zf.entries }
-
-    gotCalled = false
-    renamedEntryName = nil
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      renamedEntryName = zf.entries[0].name
-      zf.rename(zf.entries[0], zf.entries[1].name) { gotCalled = true; true }
-    end
-
-    assert(gotCalled)
-    oldEntries.delete_if { |e| e.name == renamedEntryName }
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      assert_equal(oldEntries.sort.map { |e| e.name },
-                   zf.entries.sort.map { |e| e.name })
-    end
-  end
-
-  def test_rename_non_entry
-    nonEntry = 'bogusEntry'
-    target_entry = 'target_entryName'
-    zf = ::Zip::File.new(TEST_ZIP.zip_name)
-    assert(!zf.entries.include?(nonEntry))
-    assert_raises(Errno::ENOENT) { zf.rename(nonEntry, target_entry) }
-    zf.commit
-    assert(!zf.entries.include?(target_entry))
-  ensure
-    zf.close
-  end
-
-  def test_rename_entry_to_existing_entry
-    entry1, entry2, * = TEST_ZIP.entry_names
-    zf = ::Zip::File.new(TEST_ZIP.zip_name)
-    assert_raises(::Zip::EntryExistsError) { zf.rename(entry1, entry2) }
-  ensure
-    zf.close
-  end
-
-  def test_replace
-    entryToReplace = TEST_ZIP.entry_names[2]
-    newEntrySrcFilename = 'test/data/file2.txt'
-    zf = ::Zip::File.new(TEST_ZIP.zip_name)
-    zf.replace(entryToReplace, newEntrySrcFilename)
-
-    zf.close
-    zfRead = ::Zip::File.new(TEST_ZIP.zip_name)
-    AssertEntry.assert_contents(newEntrySrcFilename,
-                                zfRead.get_input_stream(entryToReplace) { |is| is.read })
-    AssertEntry.assert_contents(TEST_ZIP.entry_names[0],
-                                zfRead.get_input_stream(TEST_ZIP.entry_names[0]) { |is| is.read })
-    AssertEntry.assert_contents(TEST_ZIP.entry_names[1],
-                                zfRead.get_input_stream(TEST_ZIP.entry_names[1]) { |is| is.read })
-    AssertEntry.assert_contents(TEST_ZIP.entry_names[3],
-                                zfRead.get_input_stream(TEST_ZIP.entry_names[3]) { |is| is.read })
-    zfRead.close
-  end
-
-  def test_replace_non_entry
-    entryToReplace = 'nonExistingEntryname'
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      assert_raises(Errno::ENOENT) { zf.replace(entryToReplace, 'test/data/file2.txt') }
-    end
-  end
-
-  def test_commit
-    newName = 'renamedFirst'
-    zf = ::Zip::File.new(TEST_ZIP.zip_name)
-    oldName = zf.entries.first
-    zf.rename(oldName, newName)
-    zf.commit
-
-    zfRead = ::Zip::File.new(TEST_ZIP.zip_name)
-    assert(zfRead.entries.detect { |e| e.name == newName } != nil)
-    assert(zfRead.entries.detect { |e| e.name == oldName }.nil?)
-    zfRead.close
-
-    zf.close
-    res = system("unzip -tqq #{TEST_ZIP.zip_name}")
-    assert_equal(res, true)
-  end
-
-  def test_double_commit(filename = 'test/data/generated/double_commit_test.zip')
-    ::FileUtils.touch('test/data/generated/test_double_commit1.txt')
-    ::FileUtils.touch('test/data/generated/test_double_commit2.txt')
-    zf = ::Zip::File.open(filename, ::Zip::File::CREATE)
-    zf.add('test1.txt', 'test/data/generated/test_double_commit1.txt')
-    zf.commit
-    zf.add('test2.txt', 'test/data/generated/test_double_commit2.txt')
-    zf.commit
-    zf.close
-    zf2 = ::Zip::File.open(filename)
-    assert(zf2.entries.detect { |e| e.name == 'test1.txt' } != nil)
-    assert(zf2.entries.detect { |e| e.name == 'test2.txt' } != nil)
-    res = system("unzip -tqq #{filename}")
-    assert_equal(res, true)
-  end
-
-  def test_double_commit_zip64
-    ::Zip.write_zip64_support = true
-    test_double_commit('test/data/generated/double_commit_test64.zip')
-  end
-
-  def test_write_buffer
-    newName = 'renamedFirst'
-    zf = ::Zip::File.new(TEST_ZIP.zip_name)
-    oldName = zf.entries.first
-    zf.rename(oldName, newName)
-    io = ::StringIO.new('')
-    buffer = zf.write_buffer(io)
-    File.open(TEST_ZIP.zip_name, 'wb') { |f| f.write buffer.string }
-    zfRead = ::Zip::File.new(TEST_ZIP.zip_name)
-    assert(zfRead.entries.detect { |e| e.name == newName } != nil)
-    assert(zfRead.entries.detect { |e| e.name == oldName }.nil?)
-    zfRead.close
-
-    zf.close
-  end
-
-  # This test tests that after commit, you
-  # can delete the file you used to add the entry to the zip file
-  # with
-  def test_commit_use_zip_entry
-    FileUtils.cp(TestFiles::RANDOM_ASCII_FILE1, OK_DELETE_FILE)
-    zf = ::Zip::File.open(TEST_ZIP.zip_name)
-    zf.add('okToDelete.txt', OK_DELETE_FILE)
-    assert_contains(zf, 'okToDelete.txt')
-    zf.commit
-    File.rename(OK_DELETE_FILE, OK_DELETE_MOVED_FILE)
-    assert_contains(zf, 'okToDelete.txt', OK_DELETE_MOVED_FILE)
-  end
-
-  #  def test_close
-  #    zf = ZipFile.new(TEST_ZIP.zip_name)
-  #    zf.close
-  #    assert_raises(IOError) {
-  #      zf.extract(TEST_ZIP.entry_names.first, "hullubullu")
-  #    }
-  #  end
-
-  def test_compound1
-    renamedName = 'renamedName'
-    filename_to_remove = ''
-    begin
-      zf = ::Zip::File.new(TEST_ZIP.zip_name)
-      originalEntries = zf.entries.dup
-
-      assert_not_contains(zf, TestFiles::RANDOM_ASCII_FILE1)
-      zf.add(TestFiles::RANDOM_ASCII_FILE1,
-             TestFiles::RANDOM_ASCII_FILE1)
-      assert_contains(zf, TestFiles::RANDOM_ASCII_FILE1)
-
-      entry_to_rename = zf.entries.find { |entry| entry.name.match('longAscii') }
-      zf.rename(entry_to_rename, renamedName)
-      assert_contains(zf, renamedName)
-
-      TestFiles::BINARY_TEST_FILES.each do |filename|
-        zf.add(filename, filename)
-        assert_contains(zf, filename)
-      end
-
-      assert_contains(zf, originalEntries.last.to_s)
-      filename_to_remove = originalEntries.map(&:to_s).find { |name| name.match('longBinary') }
-      zf.remove(filename_to_remove)
-      assert_not_contains(zf, filename_to_remove)
-    ensure
-      zf.close
-    end
-    begin
-      zfRead = ::Zip::File.new(TEST_ZIP.zip_name)
-      assert_contains(zfRead, TestFiles::RANDOM_ASCII_FILE1)
-      assert_contains(zfRead, renamedName)
-      TestFiles::BINARY_TEST_FILES.each do |filename|
-        assert_contains(zfRead, filename)
-      end
-      assert_not_contains(zfRead, filename_to_remove)
-    ensure
-      zfRead.close
-    end
-  end
-
-  def test_compound2
-    begin
-      zf = ::Zip::File.new(TEST_ZIP.zip_name)
-      originalEntries = zf.entries.dup
-
-      originalEntries.each do |entry|
-        zf.remove(entry)
-        assert_not_contains(zf, entry)
-      end
-      assert(zf.entries.empty?)
-
-      TestFiles::ASCII_TEST_FILES.each do |filename|
-        zf.add(filename, filename)
-        assert_contains(zf, filename)
-      end
-      assert_equal(zf.entries.sort.map { |e| e.name }, TestFiles::ASCII_TEST_FILES)
-
-      zf.rename(TestFiles::ASCII_TEST_FILES[0], 'newName')
-      assert_not_contains(zf, TestFiles::ASCII_TEST_FILES[0])
-      assert_contains(zf, 'newName')
-    ensure
-      zf.close
-    end
-    begin
-      zfRead = ::Zip::File.new(TEST_ZIP.zip_name)
-      asciiTestFiles = TestFiles::ASCII_TEST_FILES.dup
-      asciiTestFiles.shift
-      asciiTestFiles.each do |filename|
-        assert_contains(zf, filename)
-      end
-
-      assert_contains(zf, 'newName')
-    ensure
-      zfRead.close
-    end
-  end
-
-  def test_change_comment
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      zf.comment = 'my changed comment'
-    end
-    zfRead = ::Zip::File.open(TEST_ZIP.zip_name)
-    assert_equal('my changed comment', zfRead.comment)
-  end
-
-  def test_preserve_file_order
-    entryNames = nil
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      entryNames = zf.entries.map { |e| e.to_s }
-      zf.get_output_stream('a.txt') { |os| os.write 'this is a.txt' }
-      zf.get_output_stream('z.txt') { |os| os.write 'this is z.txt' }
-      zf.get_output_stream('k.txt') { |os| os.write 'this is k.txt' }
-      entryNames << 'a.txt' << 'z.txt' << 'k.txt'
-    end
-
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      assert_equal(entryNames, zf.entries.map { |e| e.to_s })
-      entries = zf.entries.sort_by { |e| e.name }.reverse
-      entries.each do |e|
-        zf.remove e
-        zf.get_output_stream(e) { |os| os.write 'foo' }
-      end
-      entryNames = entries.map { |e| e.to_s }
-    end
-    ::Zip::File.open(TEST_ZIP.zip_name) do |zf|
-      assert_equal(entryNames, zf.entries.map { |e| e.to_s })
-    end
-  end
-
-  def test_streaming
-    fname = ::File.join(::File.expand_path(::File.dirname(__FILE__)), '../README.md')
-    zname = 'test/data/generated/README.zip'
-    Zip::File.open(zname, Zip::File::CREATE) do |zipfile|
-      zipfile.get_output_stream(File.basename(fname)) do |f|
-        f.puts File.read(fname)
-      end
-    end
-
-    data = nil
-    File.open(zname, 'rb') do |f|
-      Zip::File.open_buffer(f) do |zipfile|
-        zipfile.each do |entry|
-          next unless entry.name =~ /README.md/
-          data = zipfile.read(entry)
-        end
-      end
-    end
-    assert data
-    assert data =~ /Simonov/
-  end
-
-  def test_nonexistant_zip
-    assert_raises(::Zip::Error) do
-      ::Zip::File.open('fake.zip')
-    end
-  end
-
-  def test_empty_zip
-    assert_raises(::Zip::Error) do
-      ::Zip::File.open(TestFiles::NULL_FILE)
-    end
-  end
-
-  def test_odd_extra_field
-    entry_count = 0
-    File.open 'test/data/oddExtraField.zip', 'rb' do |zip_io|
-      Zip::File.open_buffer zip_io.read do |zip|
-        zip.each do |_zip_entry|
-          entry_count += 1
-        end
-      end
-    end
-    assert_equal 13, entry_count
-  end
-
-  def test_open_xls_does_not_raise_type_error
-    ::Zip::File.open('test/data/test.xls')
-  end
-
-  private
-
-  def assert_contains(zf, entryName, filename = entryName)
-    assert(zf.entries.detect { |e| e.name == entryName } != nil, "entry #{entryName} not in #{zf.entries.join(', ')} in zip file #{zf}")
-    assert_entry_contents(zf, entryName, filename) if File.exist?(filename)
-  end
-
-  def assert_not_contains(zf, entryName)
-    assert(zf.entries.detect { |e| e.name == entryName }.nil?, "entry #{entryName} in #{zf.entries.join(', ')} in zip file #{zf}")
-  end
-end