archive-zip 0.3.0 → 0.4.0

data/spec/archive/zip/codec/store/decompress/close_spec.rb

@@ -0,0 +1,33 @@
+# encoding: UTF-8
+
+require File.dirname(__FILE__) + '/../../../../../../spec_helper'
+require File.dirname(__FILE__) + '/../fixtures/classes'
+require 'archive/zip/codec/store'
+require 'archive/support/binary_stringio'
+
+describe "Archive::Zip::Codec::Store::Decompress#close" do
+  it "closes the stream" do
+    c = Archive::Zip::Codec::Store::Decompress.new(BinaryStringIO.new)
+    c.close
+    c.closed?.should be_true
+  end
+
+  it "closes the delegate stream by default" do
+    delegate = mock('delegate')
+    delegate.should_receive(:close).and_return(nil)
+    c = Archive::Zip::Codec::Store::Decompress.new(delegate)
+    c.close
+  end
+
+  it "optionally leaves the delegate stream open" do
+    delegate = mock('delegate')
+    delegate.should_receive(:close).and_return(nil)
+    c = Archive::Zip::Codec::Store::Decompress.new(delegate)
+    c.close(true)
+
+    delegate = mock('delegate')
+    delegate.should_not_receive(:close)
+    c = Archive::Zip::Codec::Store::Decompress.new(delegate)
+    c.close(false)
+  end
+end

data/spec/archive/zip/codec/store/decompress/data_descriptor_spec.rb

@@ -0,0 +1,68 @@
+# encoding: UTF-8
+
+require File.dirname(__FILE__) + '/../../../../../../spec_helper'
+require File.dirname(__FILE__) + '/../fixtures/classes'
+require 'archive/support/zlib'
+require 'archive/zip/codec/store'
+
+describe "Archive::Zip::Codec::Store::Decompress#data_descriptor" do
+  it "is an instance of Archive::Zip::DataDescriptor" do
+    StoreSpecs.compressed_data do |cd|
+      closed_decompressor = Archive::Zip::Codec::Store::Decompress.open(
+        cd
+      ) do |decompressor|
+        decompressor.read
+        decompressor.data_descriptor.class.should ==
+          Archive::Zip::DataDescriptor
+        decompressor
+      end
+      closed_decompressor.data_descriptor.class.should ==
+        Archive::Zip::DataDescriptor
+    end
+  end
+
+  it "has a crc32 attribute containing the CRC32 checksum" do
+    crc32 = Zlib.crc32(StoreSpecs.test_data)
+    StoreSpecs.compressed_data do |cd|
+      closed_decompressor = Archive::Zip::Codec::Store::Decompress.open(
+        cd
+      ) do |decompressor|
+        decompressor.read
+        decompressor.data_descriptor.crc32.should == crc32
+        decompressor
+      end
+      closed_decompressor.data_descriptor.crc32.should == crc32
+    end
+  end
+
+  it "has a compressed_size attribute containing the size of the compressed data" do
+    compressed_size = StoreSpecs.compressed_data.size
+    StoreSpecs.compressed_data do |cd|
+      closed_decompressor = Archive::Zip::Codec::Store::Decompress.open(
+        cd
+      ) do |decompressor|
+        decompressor.read
+        decompressor.data_descriptor.compressed_size.should == compressed_size
+        decompressor
+      end
+      closed_decompressor.data_descriptor.compressed_size.should ==
+        compressed_size
+    end
+  end
+
+  it "has an uncompressed_size attribute containing the size of the input data" do
+    uncompressed_size = StoreSpecs.test_data.size
+    StoreSpecs.compressed_data do |cd|
+      closed_decompressor = Archive::Zip::Codec::Store::Decompress.open(
+        cd
+      ) do |decompressor|
+        decompressor.read
+        decompressor.data_descriptor.uncompressed_size.should ==
+          uncompressed_size
+        decompressor
+      end
+      closed_decompressor.data_descriptor.uncompressed_size.should ==
+        uncompressed_size
+    end
+  end
+end

data/spec/archive/zip/codec/store/decompress/new_spec.rb

@@ -0,0 +1,14 @@
+# encoding: UTF-8
+
+require File.dirname(__FILE__) + '/../../../../../../spec_helper'
+require File.dirname(__FILE__) + '/../fixtures/classes'
+require 'archive/zip/codec/store'
+require 'archive/support/binary_stringio'
+
+describe "Archive::Zip::Codec::Store::Decompress.new" do
+  it "returns a new instance" do
+    d = Archive::Zip::Codec::Store::Decompress.new(BinaryStringIO.new)
+    d.class.should == Archive::Zip::Codec::Store::Decompress
+    d.close
+  end
+end

data/spec/archive/zip/codec/store/decompress/open_spec.rb

@@ -0,0 +1,27 @@
+# encoding: UTF-8
+
+require File.dirname(__FILE__) + '/../../../../../../spec_helper'
+require File.dirname(__FILE__) + '/../fixtures/classes'
+require 'archive/zip/codec/store'
+require 'archive/support/binary_stringio'
+
+describe "Archive::Zip::Codec::Store::Decompress.open" do
+  it "returns a new instance when run without a block" do
+    d = Archive::Zip::Codec::Store::Decompress.open(BinaryStringIO.new)
+    d.class.should == Archive::Zip::Codec::Store::Decompress
+    d.close
+  end
+
+  it "executes a block with a new instance as an argument" do
+    Archive::Zip::Codec::Store::Decompress.open(BinaryStringIO.new) do |decompressor|
+      decompressor.class.should == Archive::Zip::Codec::Store::Decompress
+    end
+  end
+
+  it "closes the object after executing a block" do
+    d = Archive::Zip::Codec::Store::Decompress.open(BinaryStringIO.new) do |decompressor|
+      decompressor
+    end
+    d.closed?.should.be_true
+  end
+end

data/spec/archive/zip/codec/store/decompress/read_spec.rb

@@ -0,0 +1,24 @@
+# encoding: UTF-8
+
+require File.dirname(__FILE__) + '/../../../../../../spec_helper'
+require File.dirname(__FILE__) + '/../fixtures/classes'
+require 'archive/zip/codec/store'
+
+describe "Archive::Zip::Codec::Store::Decompress#read" do
+  it "calls the read method of the delegate" do
+    delegate = mock('delegate')
+    delegate.should_receive(:read).and_return(nil)
+    delegate.should_receive(:close).and_return(nil)
+    Archive::Zip::Codec::Store::Decompress.open(delegate) do |d|
+      d.read
+    end
+  end
+
+  it "passes data through unmodified" do
+    StoreSpecs.compressed_data do |cd|
+      Archive::Zip::Codec::Store::Decompress.open(cd) do |d|
+        d.read.should == StoreSpecs.test_data
+      end
+    end
+  end
+end

data/spec/archive/zip/codec/store/decompress/rewind_spec.rb

@@ -0,0 +1,25 @@
+# encoding: UTF-8
+
+require File.dirname(__FILE__) + '/../../../../../../spec_helper'
+require File.dirname(__FILE__) + '/../fixtures/classes'
+require 'archive/zip/codec/store'
+
+describe "Archive::Zip::Codec::Store::Decompress#rewind" do
+  it "can rewind the stream when the delegate responds to rewind" do
+    StoreSpecs.compressed_data do |cd|
+      Archive::Zip::Codec::Store::Decompress.open(cd) do |d|
+        d.read(4)
+        lambda { d.rewind }.should_not raise_error
+        d.read.should == StoreSpecs.test_data
+      end
+    end
+  end
+
+  it "raises Errno::EINVAL when attempting to rewind the stream when the delegate does not respond to rewind" do
+    delegate = mock('delegate')
+    delegate.should_receive(:close).and_return(nil)
+    Archive::Zip::Codec::Store::Decompress.open(delegate) do |d|
+      lambda { d.rewind }.should raise_error(Errno::EINVAL)
+    end
+  end
+end

data/spec/archive/zip/codec/store/decompress/seek_spec.rb

@@ -0,0 +1,57 @@
+# encoding: UTF-8
+
+require File.dirname(__FILE__) + '/../../../../../../spec_helper'
+require File.dirname(__FILE__) + '/../fixtures/classes'
+require 'archive/zip/codec/store'
+
+describe "Archive::Zip::Codec::Store::Decompress#seek" do
+  it "can seek to the beginning of the stream when the delegate responds to rewind" do
+    StoreSpecs.compressed_data do |cd|
+      Archive::Zip::Codec::Store::Decompress.open(cd) do |d|
+        d.read(4)
+        lambda { d.seek(0) }.should_not raise_error
+      end
+    end
+  end
+
+  it "raises Errno::EINVAL when attempting to seek to the beginning of the stream when the delegate does not respond to rewind" do
+    delegate = mock('delegate')
+    delegate.should_receive(:close).and_return(nil)
+    Archive::Zip::Codec::Store::Decompress.open(delegate) do |d|
+      lambda { d.seek(0) }.should raise_error(Errno::EINVAL)
+    end
+  end
+
+  it "raises Errno::EINVAL when seeking forward or backward from the current position of the stream" do
+    StoreSpecs.compressed_data do |cd|
+      Archive::Zip::Codec::Store::Decompress.open(cd) do |d|
+        # Disable read buffering to avoid some seeking optimizations implemented
+        # by IO::Like which allow seeking forward within the buffer.
+        d.fill_size = 0
+
+        d.read(4)
+        lambda { d.seek(1, IO::SEEK_CUR) }.should raise_error(Errno::EINVAL)
+        lambda { d.seek(-1, IO::SEEK_CUR) }.should raise_error(Errno::EINVAL)
+      end
+    end
+  end
+
+  it "raises Errno::EINVAL when seeking a non-zero offset relative to the beginning of the stream" do
+    StoreSpecs.compressed_data do |cd|
+      Archive::Zip::Codec::Store::Decompress.open(cd) do |d|
+        lambda { d.seek(-1, IO::SEEK_SET) }.should raise_error(Errno::EINVAL)
+        lambda { d.seek(1, IO::SEEK_SET) }.should raise_error(Errno::EINVAL)
+      end
+    end
+  end
+
+  it "raises Errno::EINVAL when seeking relative to the end of the stream" do
+    StoreSpecs.compressed_data do |cd|
+      Archive::Zip::Codec::Store::Decompress.open(cd) do |d|
+        lambda { d.seek(0, IO::SEEK_END) }.should raise_error(Errno::EINVAL)
+        lambda { d.seek(-1, IO::SEEK_END) }.should raise_error(Errno::EINVAL)
+        lambda { d.seek(1, IO::SEEK_END) }.should raise_error(Errno::EINVAL)
+      end
+    end
+  end
+end

data/spec/archive/zip/codec/store/decompress/tell_spec.rb

@@ -0,0 +1,21 @@
+# encoding: UTF-8
+
+require File.dirname(__FILE__) + '/../../../../../../spec_helper'
+require File.dirname(__FILE__) + '/../fixtures/classes'
+require 'archive/zip/codec/store'
+
+describe "Archive::Zip::Codec::Store::Decompress#tell" do
+  it "returns the current position of the stream" do
+    StoreSpecs.compressed_data do |cd|
+      Archive::Zip::Codec::Store::Decompress.open(cd) do |d|
+        d.tell.should == 0
+        d.read(4)
+        d.tell.should == 4
+        d.read
+        d.tell.should == 235
+        d.rewind
+        d.tell.should == 0
+      end
+    end
+  end
+end

data/spec/archive/zip/codec/store/fixtures/classes.rb

@@ -0,0 +1,12 @@
+# encoding: UTF-8
+
+class StoreSpecs
+  class << self
+    def test_data
+      File.open(File.join(File.dirname(__FILE__), 'raw_file.txt'), 'rb') do |f|
+        block_given? ? yield(f) : f.read
+      end
+    end
+    alias :compressed_data :test_data
+  end
+end

data/spec/archive/zip/codec/store/fixtures/raw_file.txt

@@ -0,0 +1,10 @@
+test
+This is a file with test data.
+The quick brown fox jumps over the lazy dog.
+
+Mary had a little lamb
+Whose fleece was white as snow
+And everywhere that Mary went
+The lamb was sure to go
+
+She sells sea shells down by the sea shore.

data/spec/archive/zip/codec/traditional_encryption/decrypt/close_spec.rb

@@ -0,0 +1,64 @@
+# encoding: UTF-8
+
+require File.dirname(__FILE__) + '/../../../../../../spec_helper'
+require File.dirname(__FILE__) + '/../fixtures/classes'
+require 'archive/zip/codec/traditional_encryption'
+require 'archive/support/binary_stringio'
+
+describe "Archive::Zip::Codec::TraditionalEncryption::Decrypt#close" do
+  it "closes the stream" do
+    d = Archive::Zip::Codec::TraditionalEncryption::Decrypt.new(
+      BinaryStringIO.new("\000" * 12),
+      TraditionalEncryptionSpecs.password,
+      TraditionalEncryptionSpecs.mtime
+    )
+    d.close
+    d.closed?.should be_true
+  end
+
+  it "closes the delegate stream by default" do
+    delegate = mock('delegate')
+    # RSpec's mocking facility supposedly supports this, but MSpec's does not as
+    # of version 1.5.10.
+    #delegate.should_receive(:read).with(an_instance_of(Fixnum)).at_least(:once).and_return { |n| "\000" * n }
+    # Use the following instead for now.
+    delegate.should_receive(:read).once.and_return("\000" * 12)
+    delegate.should_receive(:close).and_return(nil)
+    d = Archive::Zip::Codec::TraditionalEncryption::Decrypt.new(
+      delegate,
+      TraditionalEncryptionSpecs.password,
+      TraditionalEncryptionSpecs.mtime
+    )
+    d.close
+  end
+
+  it "optionally leaves the delegate stream open" do
+    delegate = mock('delegate')
+    # RSpec's mocking facility supposedly supports this, but MSpec's does not as
+    # of version 1.5.10.
+    #delegate.should_receive(:read).with(an_instance_of(Fixnum)).at_least(:once).and_return { |n| "\000" * n }
+    # Use the following instead for now.
+    delegate.should_receive(:read).once.and_return("\000" * 12)
+    delegate.should_receive(:close).and_return(nil)
+    d = Archive::Zip::Codec::TraditionalEncryption::Decrypt.new(
+      delegate,
+      TraditionalEncryptionSpecs.password,
+      TraditionalEncryptionSpecs.mtime
+    )
+    d.close(true)
+
+    delegate = mock('delegate')
+    # RSpec's mocking facility supposedly supports this, but MSpec's does not as
+    # of version 1.5.10.
+    #delegate.should_receive(:read).with(an_instance_of(Fixnum)).at_least(:once).and_return { |n| "\000" * n }
+    # Use the following instead for now.
+    delegate.should_receive(:read).once.and_return("\000" * 12)
+    delegate.should_not_receive(:close)
+    d = Archive::Zip::Codec::TraditionalEncryption::Decrypt.new(
+      delegate,
+      TraditionalEncryptionSpecs.password,
+      TraditionalEncryptionSpecs.mtime
+    )
+    d.close(false)
+  end
+end

data/spec/archive/zip/codec/traditional_encryption/decrypt/new_spec.rb

@@ -0,0 +1,18 @@
+# encoding: UTF-8
+
+require File.dirname(__FILE__) + '/../../../../../../spec_helper'
+require File.dirname(__FILE__) + '/../fixtures/classes'
+require 'archive/zip/codec/store'
+require 'archive/support/binary_stringio'
+
+describe "Archive::Zip::Codec::TraditionalEncryption::Decrypt.new" do
+  it "returns a new instance" do
+    d = Archive::Zip::Codec::TraditionalEncryption::Decrypt.new(
+      BinaryStringIO.new("\000" * 12),
+      TraditionalEncryptionSpecs.password,
+      TraditionalEncryptionSpecs.mtime
+    )
+    d.class.should == Archive::Zip::Codec::TraditionalEncryption::Decrypt
+    d.close
+  end
+end

data/spec/archive/zip/codec/traditional_encryption/decrypt/open_spec.rb

@@ -0,0 +1,39 @@
+# encoding: UTF-8
+
+require File.dirname(__FILE__) + '/../../../../../../spec_helper'
+require File.dirname(__FILE__) + '/../fixtures/classes'
+require 'archive/zip/codec/store'
+require 'archive/support/binary_stringio'
+
+describe "Archive::Zip::Codec::TraditionalEncryption::Decrypt.open" do
+  it "returns a new instance when run without a block" do
+    d = Archive::Zip::Codec::TraditionalEncryption::Decrypt.open(
+      BinaryStringIO.new("\000" * 12),
+      TraditionalEncryptionSpecs.password,
+      TraditionalEncryptionSpecs.mtime
+    )
+    d.class.should == Archive::Zip::Codec::TraditionalEncryption::Decrypt
+    d.close
+  end
+
+  it "executes a block with a new instance as an argument" do
+    Archive::Zip::Codec::TraditionalEncryption::Decrypt.open(
+      BinaryStringIO.new("\000" * 12),
+      TraditionalEncryptionSpecs.password,
+      TraditionalEncryptionSpecs.mtime
+    ) do |decryptor|
+      decryptor.class.should == Archive::Zip::Codec::TraditionalEncryption::Decrypt
+    end
+  end
+
+  it "closes the object after executing a block" do
+    d = Archive::Zip::Codec::TraditionalEncryption::Decrypt.open(
+      BinaryStringIO.new("\000" * 12),
+      TraditionalEncryptionSpecs.password,
+      TraditionalEncryptionSpecs.mtime
+    ) do |decryptor|
+      decryptor
+    end
+    d.closed?.should.be_true
+  end
+end

data/spec/archive/zip/codec/traditional_encryption/decrypt/read_spec.rb

@@ -0,0 +1,126 @@
+# encoding: UTF-8
+
+require File.dirname(__FILE__) + '/../../../../../../spec_helper'
+require File.dirname(__FILE__) + '/../fixtures/classes'
+require 'archive/zip/codec/traditional_encryption'
+
+describe "Archive::Zip::Codec::TraditionalEncryption::Decrypt#read" do
+  it "calls the read method of the delegate" do
+    delegate = mock('delegate')
+    # RSpec's mocking facility supposedly supports this, but MSpec's does not as
+    # of version 1.5.10.
+    #delegate.should_receive(:read).with(an_instance_of(Fixnum)).at_least(:once).and_return { |n| "\000" * n }
+    # Use the following instead for now.
+    delegate.should_receive(:read).twice.and_return("\000" * 12, nil)
+    delegate.should_receive(:close).and_return(nil)
+    Archive::Zip::Codec::TraditionalEncryption::Decrypt.open(
+      delegate,
+      TraditionalEncryptionSpecs.password,
+      TraditionalEncryptionSpecs.mtime
+    ) do |d|
+      d.read
+    end
+  end
+
+  it "decrypts data read from the delegate" do
+    TraditionalEncryptionSpecs.encrypted_data do |ed|
+      Archive::Zip::Codec::TraditionalEncryption::Decrypt.open(
+        ed,
+        TraditionalEncryptionSpecs.password,
+        TraditionalEncryptionSpecs.mtime
+      ) do |d|
+        d.read.should == TraditionalEncryptionSpecs.test_data
+      end
+    end
+  end
+
+  it "decrypts data read from a delegate that only returns 1 byte at a time" do
+    TraditionalEncryptionSpecs.encrypted_data do |ed|
+      # Override ed.read to raise Errno::EAGAIN every other time it's called.
+      class << ed
+        alias :read_orig :read
+        def read(length = nil, buffer = nil)
+          read_orig(1, buffer)
+        end
+      end
+
+      Archive::Zip::Codec::TraditionalEncryption::Decrypt.open(
+        ed,
+        TraditionalEncryptionSpecs.password,
+        TraditionalEncryptionSpecs.mtime
+      ) do |d|
+        buffer = ''
+        begin
+          buffer << d.read
+        rescue Errno::EAGAIN
+          retry
+        end
+        buffer.should == TraditionalEncryptionSpecs.test_data
+      end
+    end
+  end
+
+  it "decrypts data read from a delegate that raises Errno::EAGAIN" do
+    TraditionalEncryptionSpecs.encrypted_data do |ed|
+      # Override ed.read to raise Errno::EAGAIN every other time it's called.
+      class << ed
+        alias :read_orig :read
+        def read(length = nil, buffer = nil)
+          if @error_raised then
+            @error_raised = false
+            read_orig(length, buffer)
+          else
+            @error_raised = true
+            raise Errno::EAGAIN
+          end
+        end
+      end
+
+      Archive::Zip::Codec::TraditionalEncryption::Decrypt.open(
+        ed,
+        TraditionalEncryptionSpecs.password,
+        TraditionalEncryptionSpecs.mtime
+      ) do |d|
+        buffer = ''
+        begin
+          buffer << d.read
+        rescue Errno::EAGAIN
+          retry
+        end
+        buffer.should == TraditionalEncryptionSpecs.test_data
+      end
+    end
+  end
+
+  it "decrypts data read from a delegate that raises Errno::EINTR" do
+    TraditionalEncryptionSpecs.encrypted_data do |ed|
+      # Override ed.read to raise Errno::EINTR every other time it's called.
+      class << ed
+        alias :read_orig :read
+        def read(length = nil, buffer = nil)
+          if @error_raised then
+            @error_raised = false
+            read_orig(length, buffer)
+          else
+            @error_raised = true
+            raise Errno::EINTR
+          end
+        end
+      end
+
+      Archive::Zip::Codec::TraditionalEncryption::Decrypt.open(
+        ed,
+        TraditionalEncryptionSpecs.password,
+        TraditionalEncryptionSpecs.mtime
+      ) do |d|
+        buffer = ''
+        begin
+          buffer << d.read
+        rescue Errno::EINTR
+          retry
+        end
+        buffer.should == TraditionalEncryptionSpecs.test_data
+      end
+    end
+  end
+end