kt-paperclip 5.4.0 → 7.1.1

data/spec/paperclip/attachment_processing_spec.rb

@@ -1,78 +1,77 @@
-# encoding: utf-8
-require 'spec_helper'
+require "spec_helper"
 
-describe 'Attachment Processing' do
+describe "Attachment Processing" do
   before { rebuild_class }
 
-  context 'using validates_attachment_content_type' do
-    it 'processes attachments given a valid assignment' do
+  context "using validates_attachment_content_type" do
+    it "processes attachments given a valid assignment" do
       file = File.new(fixture_file("5k.png"))
       Dummy.validates_attachment_content_type :avatar, content_type: "image/png"
       instance = Dummy.new
       attachment = instance.avatar
-      attachment.expects(:post_process_styles)
+      expect(attachment).to receive(:post_process_styles)
 
       attachment.assign(file)
     end
 
-    it 'does not process attachments given an invalid assignment with :not' do
+    it "does not process attachments given an invalid assignment with :not" do
       file = File.new(fixture_file("5k.png"))
       Dummy.validates_attachment_content_type :avatar, not: "image/png"
       instance = Dummy.new
       attachment = instance.avatar
-      attachment.expects(:post_process_styles).never
+      expect(attachment).not_to receive(:post_process_styles)
 
       attachment.assign(file)
     end
 
-    it 'does not process attachments given an invalid assignment with :content_type' do
+    it "does not process attachments given an invalid assignment with :content_type" do
       file = File.new(fixture_file("5k.png"))
       Dummy.validates_attachment_content_type :avatar, content_type: "image/tiff"
       instance = Dummy.new
       attachment = instance.avatar
-      attachment.expects(:post_process_styles).never
+      expect(attachment).not_to receive(:post_process_styles)
 
       attachment.assign(file)
     end
 
-    it 'allows what would be an invalid assignment when validation :if clause returns false' do
+    it "allows what would be an invalid assignment when validation :if clause returns false" do
       invalid_assignment = File.new(fixture_file("5k.png"))
-      Dummy.validates_attachment_content_type :avatar, content_type: "image/tiff", if: lambda{false}
+      Dummy.validates_attachment_content_type :avatar, content_type: "image/tiff", if: lambda { false }
       instance = Dummy.new
       attachment = instance.avatar
-      attachment.expects(:post_process_styles)
+      expect(attachment).to receive(:post_process_styles)
 
       attachment.assign(invalid_assignment)
     end
   end
 
-  context 'using validates_attachment' do
-    it 'processes attachments given a valid assignment' do
+  context "using validates_attachment" do
+    it "processes attachments given a valid assignment" do
       file = File.new(fixture_file("5k.png"))
-      Dummy.validates_attachment :avatar, content_type: {content_type: "image/png"}
+      Dummy.validates_attachment :avatar, content_type: { content_type: "image/png" }
       instance = Dummy.new
       attachment = instance.avatar
-      attachment.expects(:post_process_styles)
+      expect(attachment).to receive(:post_process_styles)
 
       attachment.assign(file)
     end
 
-    it 'does not process attachments given an invalid assignment with :not' do
+    it "does not process attachments given an invalid assignment with :not" do
       file = File.new(fixture_file("5k.png"))
-      Dummy.validates_attachment :avatar, content_type: {not: "image/png"}
+      Dummy.validates_attachment :avatar, content_type: { not: "image/png" }
       instance = Dummy.new
       attachment = instance.avatar
-      attachment.expects(:post_process_styles).never
+      expect(attachment).not_to receive(:post_process_styles)
 
       attachment.assign(file)
     end
 
-    it 'does not process attachments given an invalid assignment with :content_type' do
+    it "does not process attachments given an invalid assignment with :content_type" do
       file = File.new(fixture_file("5k.png"))
-      Dummy.validates_attachment :avatar, content_type: {content_type: "image/tiff"}
+      Dummy.validates_attachment :avatar, content_type: { content_type: "image/tiff" }
       instance = Dummy.new
       attachment = instance.avatar
-      attachment.expects(:post_process_styles).never
+      expect(attachment).not_to receive(:post_process_styles)
 
       attachment.assign(file)
     end

data/spec/paperclip/attachment_registry_spec.rb

@@ -1,19 +1,19 @@
-require 'spec_helper'
+require "spec_helper"
 
-describe 'Attachment Registry' do
+describe "Attachment Registry" do
   before do
     Paperclip::AttachmentRegistry.clear
   end
 
-  context '.names_for' do
-    it 'includes attachment names for the given class' do
+  context ".names_for" do
+    it "includes attachment names for the given class" do
       foo = Class.new
       Paperclip::AttachmentRegistry.register(foo, :avatar, {})
 
       assert_equal [:avatar], Paperclip::AttachmentRegistry.names_for(foo)
     end
 
-    it 'does not include attachment names for other classes' do
+    it "does not include attachment names for other classes" do
       foo = Class.new
       bar = Class.new
       Paperclip::AttachmentRegistry.register(foo, :avatar, {})
@@ -22,13 +22,13 @@ describe 'Attachment Registry' do
       assert_equal [:lover], Paperclip::AttachmentRegistry.names_for(bar)
     end
 
-    it 'produces the empty array for a missing key' do
+    it "produces the empty array for a missing key" do
       assert_empty Paperclip::AttachmentRegistry.names_for(Class.new)
     end
   end
 
-  context '.each_definition' do
-    it 'calls the block with the class, attachment name, and options' do
+  context ".each_definition" do
+    it "calls the block with the class, attachment name, and options" do
       foo = Class.new
       expected_accumulations = [
         [foo, :avatar, { yo: "greeting" }],
@@ -47,8 +47,8 @@ describe 'Attachment Registry' do
     end
   end
 
-  context '.definitions_for' do
-    it 'produces the attachment name and options' do
+  context ".definitions_for" do
+    it "produces the attachment name and options" do
       expected_definitions = {
         avatar: { yo: "greeting" },
         greeter: { ciao: "greeting" }
@@ -70,7 +70,7 @@ describe 'Attachment Registry' do
       assert_equal expected_definitions, definitions
     end
 
-    it 'produces defintions for subclasses' do
+    it "produces defintions for subclasses" do
       expected_definitions = { avatar: { yo: "greeting" } }
       foo = Class.new
       bar = Class.new(foo)
@@ -85,7 +85,7 @@ describe 'Attachment Registry' do
       assert_equal expected_definitions, definitions
     end
 
-    it 'produces defintions for subclasses but deep merging them' do
+    it "produces defintions for subclasses but deep merging them" do
       foo_definitions = { avatar: { yo: "greeting" } }
       bar_definitions = { avatar: { ciao: "greeting" } }
       expected_definitions = {
@@ -112,7 +112,7 @@ describe 'Attachment Registry' do
       assert_equal expected_definitions, definitions
     end
 
-    it 'allows subclasses to override attachment defitions' do
+    it "allows subclasses to override attachment defitions" do
       foo_definitions = { avatar: { yo: "greeting" } }
       bar_definitions = { avatar: { yo: "hello" } }
 
@@ -141,8 +141,8 @@ describe 'Attachment Registry' do
     end
   end
 
-  context '.clear' do
-    it 'removes all of the existing attachment definitions' do
+  context ".clear" do
+    it "removes all of the existing attachment definitions" do
       foo = Class.new
       Paperclip::AttachmentRegistry.register(
         foo,