paperclip_upload 1.0.0 → 1.1.0

data/spec/dummy/spec/models/promotion_spec.rb

@@ -12,47 +12,74 @@ RSpec.describe Promotion, type: :model do
       expect(Promotion.new.photo).to be_a(Paperclip::Attachment)
     end
 
-    it "has upload accessors" do
-      promotion.upload = "upload"
-      expect(promotion.upload).to eq("upload")
+    it "has upload accessor" do
+      expect(promotion).to respond_to(:upload)
     end
 
     it "has upload_identifier accessor" do
-      promotion.upload_identifier = "upload_identifier"
-      expect(promotion.upload_identifier).to eq("upload_identifier")
+      expect(promotion).to respond_to(:upload_identifier)
+    end
+
+    context "with use_prefix option enabled" do
+      it "has upload accessors" do
+        expect(promotion).to respond_to(:prefixed_file_upload)
+      end
+
+      it "has upload_identifier accessor" do
+        expect(promotion).to respond_to(:prefixed_file_upload_identifier)
+      end
+
+      it "raises error trying to redefine upload_accessor", focus: true do
+        expect { Promotion.has_attached_upload(:photo) }.to raise_error(
+          "you are trying to redefine upload_identifier attribute")
+        expect { Promotion.has_attached_upload(:prefixed_file, upload: { use_prefix: true }) }.to raise_error(
+          "you are trying to redefine prefixed_file_upload_identifier attribute")
+      end
     end
 
     context "saving promotion" do
       context "with invalid upload" do
-        it "raise error passing an invalid upload_identifier" do
+        it "raises error passing an invalid upload_identifier" do
           promotion.upload_identifier = "invalid upload identifier"
           expect { promotion.save }.to raise_error(ActiveRecord::RecordNotFound)
         end
 
-        it "raise error passing an invalid upload" do
+        it "raises error passing an invalid upload" do
           promotion.upload = "invalid upload"
           expect { promotion.save }.to raise_error("invalid PaperclipUpload::Upload instance")
         end
       end
 
       context "with valid upload" do
-        let(:upload) { create(:upload) }
+        let!(:upload) { create(:upload) }
 
-        def saves_with_upload
+        def saves_with_upload(_paperclip_attr)
           expect(promotion.photo.exists?).to be_falsy
           yield(promotion)
           expect(promotion.save).to be_truthy
-          expect(promotion.photo.exists?).to be_truthy
+          expect(promotion.send(_paperclip_attr).exists?).to be_truthy
           expect { PaperclipUpload::Upload.find(upload.id) }.to(
             raise_error(ActiveRecord::RecordNotFound))
         end
 
         it "saves photo using upload instance" do
-          saves_with_upload { |promo| promotion.upload = upload }
+          saves_with_upload(:photo) { promotion.upload = upload }
         end
 
         it "saves photo using upload_identifier" do
-          saves_with_upload { |promo| promotion.upload_identifier = upload.identifier }
+          saves_with_upload(:photo) { promotion.upload_identifier = upload.identifier }
+        end
+
+        context "with use_prefix option enabled" do
+          it "saves prefixed_file using upload_identifier" do
+            saves_with_upload(:prefixed_file) do
+              promotion.prefixed_file_upload_identifier = upload.identifier
+            end
+          end
+
+          it "saves prefixed_file using upload instance" do
+            saves_with_upload(:prefixed_file) { promotion.prefixed_file_upload = upload }
+          end
         end
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: paperclip_upload
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.1.0
 platform: ruby
 authors:
 - Leandro Segovia
@@ -271,6 +271,9 @@ files:
 - spec/dummy/db/migrate/20150612173817_add_attachment_file_to_uploads.paperclip_upload.rb
 - spec/dummy/db/migrate/20150613212501_create_promotions.rb
 - spec/dummy/db/migrate/20150613212609_add_attachment_photo_to_promotions.rb
+- spec/dummy/db/migrate/20150717201744_add_attachment_prefixed_file_one_to_promotions.rb
+- spec/dummy/db/migrate/20150717201754_add_attachment_prefixed_file_two_to_promotions.rb
+- spec/dummy/db/migrate/20150717202131_add_attachment_prefixed_file_to_promotions.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/test.sqlite3
 - spec/dummy/log/development.log
@@ -280,6 +283,7 @@ files:
 - spec/dummy/public/422.html
 - spec/dummy/public/500.html
 - spec/dummy/public/favicon.ico
+- spec/dummy/public/system/promotions/photos/000/000/021/original/data
 - spec/dummy/Rakefile
 - spec/dummy/README.md
 - spec/dummy/spec/assets/bukowski.jpg
@@ -288,12 +292,30 @@ files:
 - spec/dummy/spec/models/promotion_spec.rb
 - spec/dummy/tmp/promotions/1/bukowski.jpg
 - spec/dummy/tmp/promotions/1/data
-- spec/dummy/tmp/uploads/1/bukowski.jpg
+- spec/dummy/tmp/promotions/11/bukowski.jpg
+- spec/dummy/tmp/promotions/13/bukowski.jpg
+- spec/dummy/tmp/promotions/15/bukowski.jpg
+- spec/dummy/tmp/promotions/17/bukowski.jpg
+- spec/dummy/tmp/promotions/19/bukowski.jpg
+- spec/dummy/tmp/promotions/21/bukowski.jpg
+- spec/dummy/tmp/promotions/21/data
+- spec/dummy/tmp/promotions/3/bukowski.jpg
+- spec/dummy/tmp/promotions/5/bukowski.jpg
+- spec/dummy/tmp/promotions/7/bukowski.jpg
+- spec/dummy/tmp/promotions/9/bukowski.jpg
+- spec/dummy/tmp/uploads/10/bukowski.jpg
+- spec/dummy/tmp/uploads/12/bukowski.jpg
+- spec/dummy/tmp/uploads/14/bukowski.jpg
+- spec/dummy/tmp/uploads/18/bukowski.jpg
 - spec/dummy/tmp/uploads/2/bukowski.jpg
 - spec/dummy/tmp/uploads/2/Marilyn_Manson_imagen_003.jpg
-- spec/dummy/tmp/uploads/3/bukowski.jpg
+- spec/dummy/tmp/uploads/20/bukowski.jpg
+- spec/dummy/tmp/uploads/21/bukowski.jpg
 - spec/dummy/tmp/uploads/3/Marilyn_Manson_imagen_005.jpg
+- spec/dummy/tmp/uploads/4/bukowski.jpg
 - spec/dummy/tmp/uploads/4/Marilyn_Manson_imagen_005.jpg
+- spec/dummy/tmp/uploads/6/bukowski.jpg
+- spec/dummy/tmp/uploads/8/bukowski.jpg
 - spec/factories/paperclip_upload/uploads.rb
 - spec/factories/promotions.rb
 - spec/rails_helper.rb
@@ -359,6 +381,9 @@ test_files:
 - spec/dummy/db/migrate/20150612173817_add_attachment_file_to_uploads.paperclip_upload.rb
 - spec/dummy/db/migrate/20150613212501_create_promotions.rb
 - spec/dummy/db/migrate/20150613212609_add_attachment_photo_to_promotions.rb
+- spec/dummy/db/migrate/20150717201744_add_attachment_prefixed_file_one_to_promotions.rb
+- spec/dummy/db/migrate/20150717201754_add_attachment_prefixed_file_two_to_promotions.rb
+- spec/dummy/db/migrate/20150717202131_add_attachment_prefixed_file_to_promotions.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/test.sqlite3
 - spec/dummy/log/development.log
@@ -368,6 +393,7 @@ test_files:
 - spec/dummy/public/422.html
 - spec/dummy/public/500.html
 - spec/dummy/public/favicon.ico
+- spec/dummy/public/system/promotions/photos/000/000/021/original/data
 - spec/dummy/Rakefile
 - spec/dummy/README.md
 - spec/dummy/spec/assets/bukowski.jpg
@@ -376,12 +402,30 @@ test_files:
 - spec/dummy/spec/models/promotion_spec.rb
 - spec/dummy/tmp/promotions/1/bukowski.jpg
 - spec/dummy/tmp/promotions/1/data
-- spec/dummy/tmp/uploads/1/bukowski.jpg
+- spec/dummy/tmp/promotions/11/bukowski.jpg
+- spec/dummy/tmp/promotions/13/bukowski.jpg
+- spec/dummy/tmp/promotions/15/bukowski.jpg
+- spec/dummy/tmp/promotions/17/bukowski.jpg
+- spec/dummy/tmp/promotions/19/bukowski.jpg
+- spec/dummy/tmp/promotions/21/bukowski.jpg
+- spec/dummy/tmp/promotions/21/data
+- spec/dummy/tmp/promotions/3/bukowski.jpg
+- spec/dummy/tmp/promotions/5/bukowski.jpg
+- spec/dummy/tmp/promotions/7/bukowski.jpg
+- spec/dummy/tmp/promotions/9/bukowski.jpg
+- spec/dummy/tmp/uploads/10/bukowski.jpg
+- spec/dummy/tmp/uploads/12/bukowski.jpg
+- spec/dummy/tmp/uploads/14/bukowski.jpg
+- spec/dummy/tmp/uploads/18/bukowski.jpg
 - spec/dummy/tmp/uploads/2/bukowski.jpg
 - spec/dummy/tmp/uploads/2/Marilyn_Manson_imagen_003.jpg
-- spec/dummy/tmp/uploads/3/bukowski.jpg
+- spec/dummy/tmp/uploads/20/bukowski.jpg
+- spec/dummy/tmp/uploads/21/bukowski.jpg
 - spec/dummy/tmp/uploads/3/Marilyn_Manson_imagen_005.jpg
+- spec/dummy/tmp/uploads/4/bukowski.jpg
 - spec/dummy/tmp/uploads/4/Marilyn_Manson_imagen_005.jpg
+- spec/dummy/tmp/uploads/6/bukowski.jpg
+- spec/dummy/tmp/uploads/8/bukowski.jpg
 - spec/factories/paperclip_upload/uploads.rb
 - spec/factories/promotions.rb
 - spec/rails_helper.rb