seven_zip_ruby 1.2.4 → 1.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 406e705d876048808c1a43bde40472e38daeda43
-  data.tar.gz: 61fbef4bf41b1f5c0589b609e27022f4cf85e908
+  metadata.gz: 24d424d3f87e77974e436f6c044a54e5783baf95
+  data.tar.gz: f5280318feffbdc6e25222916fb1168969bd3a73
 SHA512:
-  metadata.gz: ba40d7b46b2b3488690fc76ee4dad03c4d8f889f25f87d0f8a3e53842fb391d8a82b9d1b5672347fe2419a7c986160de475269552030c606680f12de5a3152b2
-  data.tar.gz: e201cbd4e6b0d18db7d9fc8531a3e5840f40e3f823e71015f3709d4384e3bf224ac5c62ac88735d20ccb94050094a9489d754977aa1af4b927c4219118b97df3
+  metadata.gz: 2db16fba18dfa60a526bd4b03317da8a512c541e1fefb55633f785493086047ce8334b47e839a72d470e1210a51546433d1ee6e67513c5649c3d45bc7b99676b
+  data.tar.gz: ccaa6fec52258413febf2db20388945fe30a19d48839e7f8287a4498fbe323893250fdfb5c54c788aa8a9ea51fee2438974f1296a2453e287c8276b4a3bcb1f9

data/.travis.yml

@@ -1,19 +1,35 @@
 language: ruby
 rvm:
-  - "2.0.0"
-  - "2.1.1"
-  - "rbx-2"
+  - "2.0"
+  - "2.1"
+  - "2.2"
+  - "2.3.0"
+  - "2.3"
+  - "rbx-2.2.7"
   - "ruby-head"
 matrix:
   allow_failures:
-    - rvm: "rbx-2"
+    - rvm: "rbx-2.2.7"
     - rvm: "ruby-head"
+  include:
+    - rvm: 2.3.1
+      os: osx
+      osx_image: xcode7.3
 notifications:
   email: false
 cache:
   - apt
 before_install:
+  - gem update bundler
   - bundle install --path=vendor/bundle
-  - sudo apt-get install p7zip-full
+  - case $TRAVIS_OS_NAME in
+      linux)
+        sudo apt-get install p7zip-full
+        ;;
+      osx)
+        brew update && brew tap wk8/p7zip9.20 && brew install p7zip920
+        ;;
+    esac
+
 install: bundle exec rake build_local
 script: bundle exec rspec spec/seven_zip_ruby_spec.rb

data/README.md

@@ -148,6 +148,17 @@ File.open("filename.7z", "rb") do |file|
 end
 ```
 
+You can also create a self extracting archive for Windows.
+
+```ruby
+File.open("filename.exe", "rb") do |file|
+  # :gui and :console can be specified as :sfx parameter.
+  SevenZipRuby::Writer.open(file, sfx: :gui) do |szr|
+    szr.add_data "file content", "file.txt"
+  end
+end
+```
+
 ### Set compression mode
 
 7zip supports LZMA, LZMA2, PPMD, BZIP2, DEFLATE and COPY.  

data/Rakefile

@@ -9,7 +9,15 @@ task :build_platform => [ :pre_platform, :build, :post_platform ]
 
 task :pre_platform do
   FileUtils.mv("seven_zip_ruby.gemspec", "seven_zip_ruby.gemspec.bak")
-  FileUtils.cp("resources/seven_zip_ruby.gemspec.platform", "seven_zip_ruby.gemspec")
+
+  versions = Dir.glob("lib/seven_zip_ruby/*").select{ |i| File.directory?(i) }.map{ |i| i.split("/").last }.sort_by{ |i| i.split(".").map(&:to_i) }
+  min_version = versions.first + ".0"
+  max_version = versions.last.split(".").first + "." + (versions.last.split(".").last.to_i + 1).to_s + ".0"
+  gemspec = File.open("resources/seven_zip_ruby.gemspec.platform", "r", &:read)
+    .gsub("SPEC_REQUIRED_RUBY_VERSION"){ "spec.required_ruby_version = [ '>= #{min_version}', '< #{max_version}' ]" }
+  File.open("seven_zip_ruby.gemspec", "w") do |f|
+    f.write(gemspec)
+  end
 end
 
 task :post_platform do

data/ext/seven_zip_ruby/extconf.rb

@@ -59,6 +59,7 @@ def sample_cpp_source
   #  - lambda
   #  - std::function
   #  - std::array
+  #  - memset_s defined, on Darwin and BSD
   return <<'EOS'
 #include <functional>
 #include <algorithm>
@@ -67,6 +68,11 @@ def sample_cpp_source
 
 #include <ruby.h>
 
+// see the test on memset_s below, which is a purely BSD thing
+#if defined(__APPLE__) || defined(BSD)
+#include <string.h>
+#endif
+
 void test()
 {
     int array[] = { 0, 1, 2, 3, 4, 5, 6, 7, 8, 9 };
@@ -84,6 +90,11 @@ void test()
     });
 
     std::for_each(var_list.begin(), var_list.end(), [](int num){ std::cout << num << std::endl; });
+
+#if defined(__APPLE__) || defined(BSD)
+    char str[] = "imareallycoolstringright";
+    memset_s(str, sizeof str, 'b', 5);
+#endif
 }
 EOS
 end
@@ -135,8 +146,8 @@ def main
     # MinGW
     $LIBS = "-loleaut32 -static-libgcc -static-libstdc++"
 
-    cpp0x_flag = [ "", "-std=c++11", "-std=gnu++11", "-std=c++0x", "-std=gnu++0x" ].find do |opt|
-      next try_compile(sample_cpp_source, "#{opt} -x c++ ")
+    cpp0x_flag = [ "", "-std=gnu++11", "-std=c++11", "-std=gnu++0x", "-std=c++0x" ].find do |opt|
+      try_compile(sample_cpp_source, "#{opt} -x c++ ")
     end
     raise "C++11 is not supported by the compiler." unless (cpp0x_flag)
 
@@ -150,8 +161,11 @@ def main
       end
     end
 
-    cpp0x_flag = [ "", "-std=c++11", "-std=gnu++11", "-std=c++0x", "-std=gnu++0x" ].find do |opt|
-      next (try_compile(sample_cpp_source, "#{opt} -x c++ ") || try_compile(sample_cpp_source, "#{opt} "))
+    possible_cpp0x_flags = [ "", "-std=gnu++11", "-std=c++11", "-std=gnu++0x", "-std=c++0x" ].map do |opt|
+      ["#{opt} -x c++ ", "#{opt} "]
+    end.flatten
+    cpp0x_flag = possible_cpp0x_flags.find do |opt|
+      try_compile(sample_cpp_source, opt)
     end
     raise "C++11 is not supported by the compiler." unless (cpp0x_flag)
 

data/ext/seven_zip_ruby/seven_zip_archive.cpp

@@ -13,6 +13,10 @@
 
 #define INTERN(const_str) rb_intern2(const_str, sizeof(const_str) - 1)
 
+// For https://bugs.ruby-lang.org/issues/11962
+#ifndef RARRAY_CONST_PTR
+#define RARRAY_CONST_PTR(index_list) RARRAY_PTR(index_list)
+#endif
 
 ////////////////////////////////////////////////////////////////
 namespace SevenZip
@@ -505,7 +509,7 @@ VALUE ArchiveReader::extractFiles(VALUE index_list, VALUE callback_proc)
     fillEntryInfo();
 
     std::vector<UInt32> list(RARRAY_LEN(index_list));
-    std::transform(RARRAY_PTR(index_list), RARRAY_PTR(index_list) + RARRAY_LEN(index_list),
+    std::transform(RARRAY_CONST_PTR(index_list), RARRAY_CONST_PTR(index_list) + RARRAY_LEN(index_list),
                    list.begin(), [](VALUE num){ return NUM2ULONG(num); });
 
     HRESULT ret;
@@ -1220,14 +1224,13 @@ STDMETHODIMP ArchiveExtractCallback::GetStream(UInt32 index, ISequentialOutStrea
     bool ret = m_archive->runRubyAction([&](){
         rb_stream = rb_funcall(proc, INTERN("call"), 2,
                                ID2SYM(INTERN("stream")), m_archive->entryInfo(index));
+        m_archive->setProcessingStream(rb_stream, index, askExtractMode);
     });
     if (!ret){
         m_archive->clearProcessingStream();
         return E_FAIL;
     }
 
-    m_archive->setProcessingStream(rb_stream, index, askExtractMode);
-
     OutStream *stream = new OutStream(rb_stream, m_archive);
     CMyComPtr<OutStream> ptr(stream);
     *outStream = ptr.Detach();
@@ -1434,14 +1437,14 @@ STDMETHODIMP ArchiveUpdateCallback::GetStream(UInt32 index, ISequentialInStream
         }else{
             rb_stream = rb_ary_entry(ret_array, 1);
         }
+
+        m_archive->setProcessingStream(rb_stream, index);
     });
     if (!ret){
         m_archive->clearProcessingStream();
         return E_FAIL;
     }
 
-    m_archive->setProcessingStream(rb_stream, index);
-
     if (NIL_P(rb_stream) && !(filepath.empty())){
         FileInStream *stream = new FileInStream(filepath, m_archive);
         CMyComPtr<FileInStream> ptr(stream);

data/ext/seven_zip_ruby/seven_zip_archive.h

@@ -385,6 +385,7 @@ class ArchiveOpenCallback : public IArchiveOpenCallback, public ICryptoGetTextPa
   public:
     ArchiveOpenCallback(ArchiveReader *archive);
     ArchiveOpenCallback(ArchiveReader *archive, const std::string &password);
+    virtual ~ArchiveOpenCallback() {}
 
     MY_UNKNOWN_IMP2(IArchiveOpenCallback, ICryptoGetTextPassword)
 
@@ -408,6 +409,7 @@ class ArchiveExtractCallback : public IArchiveExtractCallback, public ICryptoGet
   public:
     ArchiveExtractCallback(ArchiveReader *archive);
     ArchiveExtractCallback(ArchiveReader *archive, const std::string &password);
+    virtual ~ArchiveExtractCallback() {}
 
     MY_UNKNOWN_IMP2(IArchiveExtractCallback, ICryptoGetTextPassword)
 
@@ -436,6 +438,7 @@ class ArchiveUpdateCallback : public IArchiveUpdateCallback, public ICryptoGetTe
   public:
     ArchiveUpdateCallback(ArchiveWriter *archive);
     ArchiveUpdateCallback(ArchiveWriter *archive, const std::string &password);
+    virtual ~ArchiveUpdateCallback() {}
 
     MY_UNKNOWN_IMP2(IArchiveUpdateCallback, ICryptoGetTextPassword2)
 
@@ -466,6 +469,7 @@ class InStream : public IInStream, public CMyUnknownImp
 {
   public:
     InStream(VALUE stream, ArchiveBase *archive);
+    virtual ~InStream() {}
 
     MY_UNKNOWN_IMP1(IInStream)
 
@@ -481,7 +485,7 @@ class FileInStream : public IInStream, public CMyUnknownImp
 {
   public:
     FileInStream(const std::string &filename, ArchiveBase *archive);
-    ~FileInStream();
+    virtual ~FileInStream();
 
     MY_UNKNOWN_IMP1(IInStream)
 
@@ -502,6 +506,7 @@ class OutStream : public IOutStream, public CMyUnknownImp
 {
   public:
     OutStream(VALUE stream, ArchiveBase *archive);
+    virtual ~OutStream() {}
 
     MY_UNKNOWN_IMP1(IOutStream)
 
@@ -520,6 +525,7 @@ class FileOutStream : public IOutStream, public CMyUnknownImp
 {
   public:
     FileOutStream(const std::string &filename, ArchiveBase *archive);
+    virtual ~FileOutStream() {}
 
     MY_UNKNOWN_IMP1(IOutStream)
 

data/lib/seven_zip_ruby/seven_zip_reader.rb

@@ -236,6 +236,7 @@ module SevenZipRuby
     #     szr.close
     #   end
     def open(stream, param = {})
+      param = param.clone
       param[:password] = param[:password].to_s if (param[:password])
       stream.set_encoding(Encoding::ASCII_8BIT)
       open_impl(stream, param)

data/lib/seven_zip_ruby/seven_zip_writer.rb

@@ -51,10 +51,26 @@ module SevenZipRuby
   #       szw.add_directory("test_dir")
   #     end
   #   end
+  #
+  # === Create a sfx, a self extracting archive for Windows executable binary.
+  #   File.open("filename.exe", "wb") do |file|
+  #     SevenZipRuby::SevenZipWriter.open(file, sfx: true) do |szw|
+  #       szw.add_directory("test_dir")
+  #     end
+  #   end
   class SevenZipWriter
     # Encoding used for path string in 7zip archive.
     PATH_ENCODING = Encoding::UTF_8
 
+    # Files for self extraction.
+    SFX_FILE_LIST = {
+      default: File.expand_path("../7z.sfx", __FILE__),
+      gui:     File.expand_path("../7z.sfx", __FILE__),
+      console: File.expand_path("../7zCon.sfx", __FILE__)
+    }  # :nodoc:
+
+    OPEN_PARAM_LIST = [ :password, :sfx ]  # :nodoc:
+
     @use_native_input_file_stream = true
 
     class << self
@@ -64,10 +80,12 @@ module SevenZipRuby
       #
       # ==== Args
       # +stream+ :: Output stream to write 7zip archive. <tt>stream.write</tt> is needed.
-      # +param+ :: Optional hash parameter. <tt>:password</tt> key represents password of this archive.
+      # +param+ :: Optional hash parameter.  
+      #            <tt>:password</tt> key specifies password of this archive.  
+      #            <tt>:sfx</tt> key specifies Self Extracting mode. <tt>:gui</tt> and <tt>:console</tt> can be used. <tt>true</tt> is same as <tt>:gui</tt>.
       #
       # ==== Examples
-      #   # Open archive
+      #   # Open an archive
       #   File.open("filename.7z", "wb") do |file|
       #     SevenZipRuby::SevenZipWriter.open(file) do |szw|
       #       # Create archive.
@@ -84,6 +102,14 @@ module SevenZipRuby
       #     szw.compress  # Compress must be called in this case.
       #     szw.close
       #   end
+      #
+      #   # Create a self extracting archive. <tt>:gui</tt> and <tt>:console</tt> can be used.
+      #   File.open("filename.7z", "wb") do |file|
+      #     szw = SevenZipRuby::SevenZipWriter.open(file, sfx: :gui)
+      #     # Create archive.
+      #     szw.compress  # Compress must be called in this case.
+      #     szw.close
+      #   end
       def open(stream, param = {}, &block)  # :yield: szw
         szw = self.new
         szw.open(stream, param)
@@ -104,7 +130,9 @@ module SevenZipRuby
       #
       # ==== Args
       # +filename+ :: 7zip archive filename.
-      # +param+ :: Optional hash parameter. <tt>:password</tt> key represents password of this archive.
+      # +param+ :: Optional hash parameter.  
+      #            <tt>:password</tt> key specifies password of this archive.  
+      #            <tt>:sfx</tt> key specifies Self Extracting mode. <tt>:gui</tt> and <tt>:console</tt> can be used. <tt>true</tt> is same as <tt>:gui</tt>.
       #
       # ==== Examples
       #   # Open archive
@@ -141,7 +169,9 @@ module SevenZipRuby
       # ==== Args
       # +stream+ :: Output stream to write 7zip archive. <tt>stream.write</tt> is needed.
       # +dir+ :: Directory to be added to the 7zip archive. <b><tt>dir</tt></b> must be a <b>relative path</b>.
-      # +param+ :: Optional hash parameter. <tt>:password</tt> key represents password of this archive.
+      # +param+ :: Optional hash parameter.  
+      #            <tt>:password</tt> key specifies password of this archive.  
+      #            <tt>:sfx</tt> key specifies Self Extracting mode. <tt>:gui</tt> and <tt>:console</tt> can be used. <tt>true</tt> is same as <tt>:gui</tt>.
       #
       # ==== Examples
       #   # Create 7zip archive which includes 'dir'.
@@ -149,9 +179,11 @@ module SevenZipRuby
       #     SevenZipRuby::SevenZipWriter.add_directory(file, 'dir')
       #   end
       def add_directory(stream, dir, param = {})
-        open_param = {
-          password: param.delete(:password)
-        }
+        param = param.clone
+        open_param = {}
+        OPEN_PARAM_LIST.each do |key|
+          open_param[key] = param.delete(key) if param.key?(key)
+        end
         self.open(stream, open_param) do |szw|
           szw.add_directory(dir, param)
         end
@@ -163,7 +195,9 @@ module SevenZipRuby
       # ==== Args
       # +stream+ :: Output stream to write 7zip archive. <tt>stream.write</tt> is needed.
       # +file+ :: File to be added to the 7zip archive. <b><tt>file</tt></b> must be a <b>relative path</b>.
-      # +param+ :: Optional hash parameter. <tt>:password</tt> key represents password of this archive.
+      # +param+ :: Optional hash parameter.  
+      #            <tt>:password</tt> key specifies password of this archive.  
+      #            <tt>:sfx</tt> key specifies Self Extracting mode. <tt>:gui</tt> and <tt>:console</tt> can be used. <tt>true</tt> is same as <tt>:gui</tt>.
       #
       # ==== Examples
       #   # Create 7zip archive which includes 'file.txt'.
@@ -171,9 +205,11 @@ module SevenZipRuby
       #     SevenZipRuby::SevenZipWriter.add_file(file, 'file.txt')
       #   end
       def add_file(stream, filename, param = {})
-        open_param = {
-          password: param.delete(:password)
-        }
+        param = param.clone
+        open_param = {}
+        OPEN_PARAM_LIST.each do |key|
+          open_param[key] = param.delete(key) if param.key?(key)
+        end
         self.open(stream, open_param) do |szw|
           szw.add_file(filename, param)
         end
@@ -186,7 +222,9 @@ module SevenZipRuby
     #
     # ==== Args
     # +stream+ :: Output stream to write 7zip archive. <tt>stream.write</tt> is needed.
-    # +param+ :: Optional hash parameter. <tt>:password</tt> key represents password of this archive.
+    # +param+ :: Optional hash parameter.  
+    #            <tt>:password</tt> key specifies password of this archive.  
+    #            <tt>:sfx</tt> key specifies Self Extracting mode. <tt>:gui</tt> and <tt>:console</tt> can be used. <tt>true</tt> is same as <tt>:gui</tt>.
     #
     # ==== Examples
     #   File.open("filename.7z", "wb") do |file|
@@ -197,16 +235,25 @@ module SevenZipRuby
     #     szw.close
     #   end
     def open(stream, param = {})
+      param = param.clone
+      param[:password] = param[:password].to_s if (param[:password])
       stream.set_encoding(Encoding::ASCII_8BIT)
+
+      add_sfx(stream, param[:sfx]) if param[:sfx]
+
       open_impl(stream, param)
       return self
     end
 
     # Open 7zip archive file to create.
     #
+    # <tt>close</tt> method must be called later.
+    #
     # ==== Args
     # +filename+ :: 7zip archive filename.
-    # +param+ :: Optional hash parameter. <tt>:password</tt> key represents password of this archive.
+    # +param+ :: Optional hash parameter.  
+    #            <tt>:password</tt> key specifies password of this archive.  
+    #            <tt>:sfx</tt> key specifies Self Extracting mode. <tt>:gui</tt> and <tt>:console</tt> can be used. <tt>true</tt> is same as <tt>:gui</tt>.
     #
     # ==== Examples
     #   szw = SevenZipRuby::SevenZipWriter.new
@@ -425,6 +472,17 @@ module SevenZipRuby
     end
     private :compress_proc
 
+    def add_sfx(stream, sfx)
+      sfx = :default if sfx == true
+      sfx_file = SFX_FILE_LIST[sfx]
+      raise ArgumentError.new("invalid option: :sfx") unless sfx_file
+
+      File.open(sfx_file, "rb") do |input|
+        IO.copy_stream(input, stream)
+      end
+    end
+    private :add_sfx
+
     COMPRESS_GUARD = Mutex.new  # :nodoc:
     def synchronize  # :nodoc:
       if (COMPRESS_GUARD)

data/lib/seven_zip_ruby/version.rb

@@ -1,3 +1,3 @@
 module SevenZipRuby
-  VERSION = "1.2.4"
+  VERSION = "1.2.5"
 end

data/spec/seven_zip_ruby_spec.rb

@@ -206,7 +206,7 @@ describe SevenZipRuby do
       example "invalid index" do
         File.open(SevenZipRubySpecHelper::SEVEN_ZIP_FILE, "rb") do |file|
           SevenZipRuby::SevenZipReader.open(file) do |szr|
-            expect{ szr.extract_data(nil) }.to raise_error (ArgumentError)
+            expect{ szr.extract_data(nil) }.to raise_error(ArgumentError)
           end
         end
       end
@@ -214,18 +214,22 @@ describe SevenZipRuby do
       example "invalid index for entry" do
         File.open(SevenZipRubySpecHelper::SEVEN_ZIP_FILE, "rb") do |file|
           SevenZipRuby::SevenZipReader.open(file) do |szr|
-            expect{ szr.entry("a") }.to raise_error
+            expect{ szr.entry("a") }.to raise_error(TypeError)
           end
         end
       end
 
       example "invalid password" do
         File.open(SevenZipRubySpecHelper::SEVEN_ZIP_PASSWORD_FILE, "rb") do |file|
-          expect{ SevenZipRuby::Reader.open(file){ |szr| szr.extract_data(1) } }.to raise_error
+          expect{ SevenZipRuby::Reader.open(file){ |szr| szr.extract_data(1) } }.to raise_error(StandardError)
         end
 
         File.open(SevenZipRubySpecHelper::SEVEN_ZIP_PASSWORD_FILE, "rb") do |file|
-          expect{ SevenZipRuby::Reader.open(file, password: "a"){ |szr| szr.extract_data(1) } }.to raise_error
+          expect{ SevenZipRuby::Reader.open(file, password: "a"){ |szr| szr.extract_data(1) } }.to raise_error(SevenZipRuby::InvalidArchive)
+        end
+
+        File.open(SevenZipRubySpecHelper::SEVEN_ZIP_PASSWORD_FILE, "rb") do |file|
+          expect{ SevenZipRuby::Reader.open(file, password: :InvalidType){ |szr| szr.extract_data(1) } }.to raise_error(SevenZipRuby::InvalidArchive)
         end
       end
 
@@ -299,8 +303,8 @@ describe SevenZipRuby do
       end
 
       example "clone and dup cannot be called." do
-        expect{ SevenZipRuby::SevenZipReader.new.clone }.to raise_error
-        expect{ SevenZipRuby::SevenZipReader.new.dup }.to raise_error
+        expect{ SevenZipRuby::SevenZipReader.new.clone }.to raise_error(NoMethodError)
+        expect{ SevenZipRuby::SevenZipReader.new.dup }.to raise_error(NoMethodError)
       end
 
     end
@@ -369,6 +373,70 @@ describe SevenZipRuby do
       end
     end
 
+    example "set password" do
+      sample_data = "Sample Data"
+      sample_password = "sample password"
+
+      output = StringIO.new("")
+      SevenZipRuby::SevenZipWriter.open(output, { password: sample_password }) do |szw|
+        szw.add_data(sample_data, "hoge.txt")
+      end
+
+      output.rewind
+      SevenZipRuby::SevenZipReader.open(output, { password: sample_password }) do |szr|
+        expect(szr.extract_data(0)).to eq sample_data
+      end
+
+      output.rewind
+      expect{
+        SevenZipRuby::SevenZipReader.open(output, { password: "invalid password" }) do |szr|
+          szr.extract_data(0)
+        end
+      }.to raise_error(SevenZipRuby::InvalidArchive)
+
+
+      output = StringIO.new("")
+      SevenZipRuby::SevenZipWriter.open(output, { password: sample_password.to_sym }) do |szw|
+        szw.add_data(sample_data, "hoge.txt")
+      end
+
+      output.rewind
+      SevenZipRuby::SevenZipReader.open(output, { password: sample_password }) do |szr|
+        expect(szr.extract_data(0)).to eq sample_data
+      end
+    end
+
+    example "create a sfx archive" do
+      time = Time.now
+
+      output = StringIO.new("")
+      SevenZipRuby::SevenZipWriter.open(output) do |szw|
+        szw.add_data("hogehoge", "hoge.txt", ctime: time, atime: time, mtime: time)
+      end
+
+      output1 = StringIO.new("")
+      SevenZipRuby::SevenZipWriter.open(output1, sfx: true) do |szw|
+        szw.add_data("hogehoge", "hoge.txt", ctime: time, atime: time, mtime: time)
+      end
+
+      output2 = StringIO.new("")
+      SevenZipRuby::SevenZipWriter.open(output2, sfx: :gui) do |szw|
+        szw.add_data("hogehoge", "hoge.txt", ctime: time, atime: time, mtime: time)
+      end
+
+      output3 = StringIO.new("")
+      SevenZipRuby::SevenZipWriter.open(output3, sfx: :console) do |szw|
+        szw.add_data("hogehoge", "hoge.txt", ctime: time, atime: time, mtime: time)
+      end
+
+      gui = File.open(SevenZipRuby::SevenZipWriter::SFX_FILE_LIST[:gui], "rb", &:read)
+      console = File.open(SevenZipRuby::SevenZipWriter::SFX_FILE_LIST[:console], "rb", &:read)
+
+      expect(output1.string).to eq gui + output.string
+      expect(output2.string).to eq gui + output.string
+      expect(output3.string).to eq console + output.string
+    end
+
     [ true, false ].each do |use_native_input_file_stream|
       example "add_directory: use_native_input_file_stream=#{use_native_input_file_stream}" do
         SevenZipRuby::SevenZipWriter.use_native_input_file_stream = use_native_input_file_stream
@@ -562,11 +630,11 @@ describe SevenZipRuby do
       end
 
       example "invalid method" do
-        expect{ SevenZipRuby::SevenZipWriter.open(StringIO.new("")).method = "Unknown" }.to raise_error
+        expect{ SevenZipRuby::SevenZipWriter.open(StringIO.new("")).method = "Unknown" }.to raise_error(ArgumentError)
       end
 
       example "invalid level" do
-        expect{ SevenZipRuby::SevenZipWriter.open(StringIO.new("")).level = 2 }.to raise_error
+        expect{ SevenZipRuby::SevenZipWriter.open(StringIO.new("")).level = 2 }.to raise_error(ArgumentError)
       end
 
       example "add_data/mkdir/compress/close before open" do
@@ -592,8 +660,8 @@ describe SevenZipRuby do
       end
 
       example "clone and dup cannot be called." do
-        expect{ SevenZipRuby::SevenZipWriter.new.clone }.to raise_error
-        expect{ SevenZipRuby::SevenZipWriter.new.dup }.to raise_error
+        expect{ SevenZipRuby::SevenZipWriter.new.clone }.to raise_error(NoMethodError)
+        expect{ SevenZipRuby::SevenZipWriter.new.dup }.to raise_error(NoMethodError)
       end
 
       if (false && RUBY_ENGINE == "ruby")

metadata

@@ -1,55 +1,55 @@
 --- !ruby/object:Gem::Specification
 name: seven_zip_ruby
 version: !ruby/object:Gem::Version
-  version: 1.2.4
+  version: 1.2.5
 platform: ruby
 authors:
 - Masamitsu MURASE
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-09-22 00:00:00.000000000 Z
+date: 2016-11-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '1.3'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: SevenZipRuby (seven_zip_ruby) is a ruby gem library to read and write
@@ -61,8 +61,8 @@ extensions:
 - ext/seven_zip_ruby/extconf.rb
 extra_rdoc_files: []
 files:
-- .gitignore
-- .travis.yml
+- ".gitignore"
+- ".travis.yml"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -780,7 +780,9 @@ files:
 - ext/seven_zip_ruby/win32/mutex.h
 - lib/seven_zip_ruby.rb
 - lib/seven_zip_ruby/7z.dll
+- lib/seven_zip_ruby/7z.sfx
 - lib/seven_zip_ruby/7z64.dll
+- lib/seven_zip_ruby/7zCon.sfx
 - lib/seven_zip_ruby/archive_info.rb
 - lib/seven_zip_ruby/entry_info.rb
 - lib/seven_zip_ruby/exception.rb
@@ -801,17 +803,17 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: 2.0.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.14
+rubygems_version: 2.6.8
 signing_key: 
 specification_version: 4
 summary: This is a ruby gem library to read and write 7zip files.