paperclip-deflater 0.1.1 → 0.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 539dd168f00b6c091360f30478cad895785a6934
-  data.tar.gz: be6cbe7df876926c11981631be5b21f181f886b9
+  metadata.gz: 0dd150f55ee5cd1546e2fabb1f79de33123dbac6
+  data.tar.gz: 115cfeed1efbb5368788909402f7aeb33180739e
 SHA512:
-  metadata.gz: 8399e7e84341b206c85a97311f254b379014765f7d2d466491a69ed2b821e1b1700aea2df8c6d986b35d9919929a368348729f203b2187ba9f42d2cbd869fc65
-  data.tar.gz: adb2cb920abebc405886b46f2bc4d500d77a206496d5f47ba111377f28387ba0567bf41d25d2ce6f8e285bfc039172cc19eb8d60872a645f50281412e50cd5b1
+  metadata.gz: 83dfc7c024c791b69b28620adfc8ef1bd05811d4061ad0a648ccf39d6f15e6c1b8ca3f86b582492db74c92b5bcec34d213e57f6132420ea32e5e504a9f3befd9
+  data.tar.gz: f52376327d04a63029eea8c72e0056ebbc405e68b8075460d2dcb5ed2810905f396f5cd1dd630d9cbafd1a98768f0b98a1a2243271e70eec2e31f88bcad4c711

data/VERSION

@@ -1 +1 @@
-0.1.1
+0.1.2

data/lib/paperclip/processors/gzip.rb

@@ -15,10 +15,12 @@ module Paperclip
         dst = create_tempfile
         begin
           gz = Zlib::GzipWriter.new(dst, level, strategy)
-          gz.write(@file.read)
-        ensure
-          gz.close
+          gz.write @file.read
+        rescue ::Exception => e
+          gz.close rescue nil
+          raise e
         end
+        gz.close
         @file.rewind
         dst.open
 

data/spec/paperclip/processors/gzip_spec.rb

@@ -28,6 +28,37 @@ describe Paperclip::Processors::Gzip do
         let(:options) { {gzip_options: {level: Zlib::BEST_COMPRESSION}} }
         include_examples "deflate"
       end
+      context "gzip failed to close" do
+        before do
+          @error = RuntimeError.new "error"
+          @writer = {}
+          allow(Zlib::GzipWriter).to receive(:new).and_return @writer
+          allow(@writer).to receive(:write).and_return nil
+        end
+
+        it "raises an error to close" do
+          expect(@writer).to receive(:close).and_raise @error
+          expect {
+            subject.make
+          }.to raise_error @error
+        end
+      end
+      context "gzip failed to write" do
+        before do
+          @error1 = RuntimeError.new "error1"
+          @error2 = RuntimeError.new "error2"
+          @writer = {}
+          allow(Zlib::GzipWriter).to receive(:new).and_return @writer
+          allow(@writer).to receive(:write).and_raise @error1
+        end
+
+        it "does not raise an error to close" do
+          expect(@writer).to receive(:close).and_raise @error2
+          expect {
+            subject.make
+          }.to raise_error @error1
+        end
+      end
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: paperclip-deflater
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.2
 platform: ruby
 authors:
 - Daisuke Taniwaki
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-11-19 00:00:00.000000000 Z
+date: 2015-06-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: paperclip
@@ -115,7 +115,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.3
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 4
 summary: Deflate Processor for Paperclip
@@ -127,3 +127,4 @@ test_files:
 - spec/paperclip/processors/plain_spec.rb
 - spec/spec_helper.rb
 - spec/support/files.rb
+has_rdoc: