carrierwave_backgrounder 1.0.1 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f6e8ff06c13487a9d2cc4b62f2384e664f5071890b2ab3f84c8c4af51fe1d3aa
-  data.tar.gz: 2bc907bba296cf9c7a9007ca4a345911def1c88a3e1d6646c1a3dbd998598e3b
+  metadata.gz: 9db3ebd17e2fba8538ee90f3f451f6d37738a9d833bc5ac9f6f8123a3f7a214a
+  data.tar.gz: 3ef48ab1f8ac80b9ea3e374fd4806b769927baec00cc8c6d7a19b9340cebe906
 SHA512:
-  metadata.gz: 1129e8d192664e21f53a20b5ad3de4801c363bdaab8bb15e4b009b2f20f178631c1b731587686e2c70ea42723f57e6cea3de16a1154cc57d81e394194b58a6b5
-  data.tar.gz: add77ebea2c899b22c6ed8d8047b83286372adc360a7bd46b394f653d9e28d9dc24d9367067e5573765af99bbc41e467ee1b0c9f30aed009b20148a8df601248
+  metadata.gz: 67965fc212f3fa08aa5936815b63a63215550780e2150759d6dc8d87683ca7be2d5614f8d7ba9b93a22e4c1537e6db9fcffb938649708e2e42508bb868835062
+  data.tar.gz: 20c1a3d50441f061701e8ada2af4b2b26706794496f71fa707ff865338358c834ede9de39369e2448b39c149a9cfb6f8528f8a7daa126c3f51a4e7bc1767fe81

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+## 1.0.2
+
+### bugfixes
+  * [#317] Fixed an issue where #process_in_backfground would copy files but not create versions [Qqwy]
 ## 1.0.1
 
 ### bugfixes

data/lib/backgrounder/version.rb

@@ -1,5 +1,5 @@
 module CarrierWave
   module Backgrounder
-    VERSION = "1.0.1"
+    VERSION = "1.0.2"
   end
 end

data/lib/backgrounder/workers/process_asset_mixin.rb

@@ -11,6 +11,7 @@ module CarrierWave
 
       def perform(*args)
         record = super(*args)
+        record.send(:"process_#{column}_upload=", true)
         asset = record.send(:"#{column}")
 
         return unless record && asset_present?(asset)

data/spec/backgrounder/workers/process_asset_spec.rb

@@ -24,6 +24,7 @@ RSpec.describe CarrierWave::Workers::ProcessAsset do
       allow(user).to receive(:find).with('22').and_return(user).once
       allow(user).to receive(:image).thrice.and_return(image)
 
+      expect(user).to receive(:process_image_upload=).with(true).once
       expect(image).to receive(:recreate_versions!).once.and_return(true)
       expect(image).to receive(:file).and_return(image)
     end
@@ -51,6 +52,7 @@ RSpec.describe CarrierWave::Workers::ProcessAsset do
       allow(admin).to receive(:avatar).twice.and_return(avatar)
 
       expect(admin).to receive(:respond_to?).with(:avatar_processing).once.and_return(false)
+      expect(admin).to receive(:process_avatar_upload=).with(true).once
       expect(avatar).to receive(:recreate_versions!).once.and_return(true)
       expect(avatar).to receive(:file).and_return(avatar)
 

data/spec/integrations/process_in_background_multi_upload_spec.rb

@@ -32,7 +32,9 @@ RSpec.describe '::process_in_background multi-upload support', clear_images: tru
 
     it 'creates the versions' do
       version_paths = AvatarUploader.versions.keys.map { |key| admin.images.map { |i| i.send(key).current_path } }.flatten
-      version_paths.each { |path| expect(File.exist? path).to be(true) }
+      version_paths.each { |path| expect(File.exist?(path)).to be(true) }
+      file_sizes = version_paths.map { |path| File.size(path) }
+      expect(file_sizes.uniq.count).to be(4)
     end
 
     it 'removes the files tmp directory' do
@@ -78,4 +80,4 @@ RSpec.describe '::process_in_background multi-upload support', clear_images: tru
       expect(Sidekiq::Queues["carrierwave"].size).to be(0)
     end
   end
-end
+end

data/spec/integrations/process_in_background_spec.rb

@@ -32,7 +32,9 @@ RSpec.describe '::process_in_background', clear_images: true do
 
     it 'creates the versions' do
       version_paths = admin.avatar.versions.keys.map { |key| admin.avatar.send(key).current_path }
-      version_paths.each { |path| expect(File.exist? path).to be(true) }
+      version_paths.each { |path| expect(File.exist?(path)).to be(true) }
+      file_sizes = version_paths.map { |path| File.size(path) }
+      expect(file_sizes.uniq.count).to be(4)
     end
 
     it 'removes the files tmp directory' do
@@ -72,4 +74,4 @@ RSpec.describe '::process_in_background', clear_images: true do
       expect(admin.avatar.file.nil?).to be(true)
     end
   end
-end
+end

data/spec/integrations/store_in_background_multi_upload_spec.rb

@@ -33,7 +33,9 @@ RSpec.describe '::store_in_background multi-upload support', clear_images: true
 
     it 'creates the versions' do
       version_paths = AvatarUploader.versions.keys.map { |key| user.images.map { |i| i.send(key).current_path } }.flatten
-      version_paths.each { |path| expect(File.exist? path).to be(true) }
+      version_paths.each { |path| expect(File.exist?(path)).to be(true) }
+      file_sizes = version_paths.map { |path| File.size(path) }
+      expect(file_sizes.uniq.count).to be(4)
     end
 
     it 'sets the <column>_tmp to nil' do
@@ -84,4 +86,4 @@ RSpec.describe '::store_in_background multi-upload support', clear_images: true
     end
 
   end
-end
+end

data/spec/integrations/store_in_background_spec.rb

@@ -33,7 +33,9 @@ RSpec.describe '::store_in_background', clear_images: true do
 
     it 'creates the versions' do
       version_paths = user.avatar.versions.keys.map { |key| user.avatar.send(key).current_path }
-      version_paths.each { |path| expect(File.exist? path).to be(true) }
+      version_paths.each { |path| expect(File.exist?(path)).to be(true) }
+      file_sizes = version_paths.map { |path| File.size(path) }
+      expect(file_sizes.uniq.count).to be(4)
     end
 
     it 'sets the <column>_tmp to nil' do
@@ -83,4 +85,4 @@ RSpec.describe '::store_in_background', clear_images: true do
       expect(Sidekiq::Queues["carrierwave"].size).to be(0)
     end
   end
-end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: carrierwave_backgrounder
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - Larry Sprock
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-09-15 00:00:00.000000000 Z
+date: 2023-10-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: carrierwave