saviour 0.4.8 → 0.4.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 709681c9a25bc91c834cdf0d2ad1027fbd64a458
-  data.tar.gz: 21ba25beeb98a5aa4ff061f96143896ec7006d5a
+  metadata.gz: 52d3e1c76ad21adf351834a7d0dd101582e9a835
+  data.tar.gz: 2ac25be01722c1432e8f75408be8a448b0505761
 SHA512:
-  metadata.gz: 4e2f3746a2455e442ae5920f65244eb8ba596ca6a77864e30d4bb24a388ee14fb2bf17c806112071f7c80310bcaee3fb05077484fb1bd97fd83ca6d2a2b95f0d
-  data.tar.gz: db8b60ee46c478b21faa237ed688c5a53a8526e022df2962cd8ad162948322ae3aa6c6850af5876d7c5fe4dcb01fe5f442559137a84e8f713bd5927a901b72fa
+  metadata.gz: ace50b4dfc8bfc04ab889fcbf92955d0815eb9f48fa97900ca3f9227d80deabba9f8e74e79f1ee4258f288058268d34faeab0f0563e3ece363c868e53d637195
+  data.tar.gz: b7b9b71f3150a8142a84ddcf9979ba3be6cea056692d42f580e2496db2525103368ceb158e9fadba150292103573998e82488ba0ca29af654efabfaed3273da9

data/lib/saviour/uploader/processors_runner.rb

@@ -55,7 +55,7 @@ module Saviour
           self.filename = result[1]
 
         else
-          file.rewind
+          file.reopen(file.path)
 
           result = run_method_or_block(method_or_block, opts, file)
 

data/lib/saviour/version.rb

@@ -1,3 +1,3 @@
 module Saviour
-  VERSION = "0.4.8"
+  VERSION = "0.4.9"
 end

data/spec/feature/reopens_file_at_every_process_spec.rb

@@ -0,0 +1,49 @@
+require 'spec_helper'
+
+describe "file information" do
+  before { allow(Saviour::Config).to receive(:storage).and_return(Saviour::LocalStorage.new(local_prefix: @tmpdir, public_url_prefix: "http://domain.com")) }
+
+  let(:uploader) {
+    Class.new(Saviour::BaseUploader) do
+      store_dir { "/store/dir/#{model.id}" }
+      process_with_file do |file, name|
+        model.result1 = file.stat.size
+
+        path = file.path
+
+        # Assign new contents on the same path, forcing a new inode
+        ::File.delete(path)
+        ::File.write(path, "SOME NEW DATA, LONGER")
+
+        [file, name]
+      end
+
+      process_with_file do |file, name|
+        model.result2 = file.stat.size # This must be the size of the new contents
+
+        [file, name]
+      end
+    end
+  }
+
+  let(:klass) {
+    klass = Class.new(Test) {
+      attr_accessor :result1, :result2
+      include Saviour::Model
+    }
+    klass.attach_file :file, uploader
+    klass
+  }
+
+  it "is renewed at every process_with_file, clearing stale data on the file instance" do
+    f = Tempfile.new("test")
+    f.write("original") # 8 bytes
+    f.flush
+
+    a = klass.create! file: f
+    expect(a.result1).to eq 8
+    expect(a.result2).to eq 21
+
+    f.close!
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: saviour
 version: !ruby/object:Gem::Version
-  version: 0.4.8
+  version: 0.4.9
 platform: ruby
 authors:
 - Roger Campos
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-01-13 00:00:00.000000000 Z
+date: 2018-01-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -195,6 +195,7 @@ files:
 - spec/feature/persisted_path_spec.rb
 - spec/feature/processors_api_spec.rb
 - spec/feature/reload_model_spec.rb
+- spec/feature/reopens_file_at_every_process_spec.rb
 - spec/feature/rewind_source_before_read_spec.rb
 - spec/feature/transactional_behavior_spec.rb
 - spec/feature/uploader_declaration_spec.rb