paperclip 3.5.4 → 5.3.0

data/spec/paperclip/schema_spec.rb

@@ -0,0 +1,248 @@
+require 'spec_helper'
+require 'paperclip/schema'
+require 'active_support/testing/deprecation'
+
+describe Paperclip::Schema do
+  include ActiveSupport::Testing::Deprecation
+
+  before do
+    rebuild_class
+  end
+
+  after do
+    Dummy.connection.drop_table :dummies rescue nil
+  end
+
+  context "within table definition" do
+    context "using #has_attached_file" do
+      before do
+        ActiveSupport::Deprecation.silenced = false
+      end
+      it "creates attachment columns" do
+        Dummy.connection.create_table :dummies, force: true do |t|
+          ActiveSupport::Deprecation.silence do
+            t.has_attached_file :avatar
+          end
+        end
+
+        columns = Dummy.columns.map{ |column| [column.name, column.type] }
+
+        expect(columns).to include(['avatar_file_name', :string])
+        expect(columns).to include(['avatar_content_type', :string])
+        expect(columns).to include(['avatar_file_size', :integer])
+        expect(columns).to include(['avatar_updated_at', :datetime])
+      end
+
+      it "displays deprecation warning" do
+        Dummy.connection.create_table :dummies, force: true do |t|
+          assert_deprecated do
+            t.has_attached_file :avatar
+          end
+        end
+      end
+    end
+
+    context "using #attachment" do
+      before do
+        Dummy.connection.create_table :dummies, force: true do |t|
+          t.attachment :avatar
+        end
+      end
+
+      it "creates attachment columns" do
+        columns = Dummy.columns.map{ |column| [column.name, column.type] }
+
+        expect(columns).to include(['avatar_file_name', :string])
+        expect(columns).to include(['avatar_content_type', :string])
+        expect(columns).to include(['avatar_file_size', :integer])
+        expect(columns).to include(['avatar_updated_at', :datetime])
+      end
+    end
+
+    context "using #attachment with options" do
+      before do
+        Dummy.connection.create_table :dummies, force: true do |t|
+          t.attachment :avatar, default: 1, file_name: { default: 'default' }
+        end
+      end
+
+      it "sets defaults on columns" do
+        defaults_columns = ["avatar_file_name", "avatar_content_type", "avatar_file_size"]
+        columns = Dummy.columns.select { |e| defaults_columns.include? e.name }
+
+        expect(columns).to have_column("avatar_file_name").with_default("default")
+        expect(columns).to have_column("avatar_content_type").with_default("1")
+        expect(columns).to have_column("avatar_file_size").with_default(1)
+      end
+    end
+  end
+
+  context "within schema statement" do
+    before do
+      Dummy.connection.create_table :dummies, force: true
+    end
+
+    context "migrating up" do
+      context "with single attachment" do
+        before do
+          Dummy.connection.add_attachment :dummies, :avatar
+        end
+
+        it "creates attachment columns" do
+          columns = Dummy.columns.map{ |column| [column.name, column.type] }
+
+          expect(columns).to include(['avatar_file_name', :string])
+          expect(columns).to include(['avatar_content_type', :string])
+          expect(columns).to include(['avatar_file_size', :integer])
+          expect(columns).to include(['avatar_updated_at', :datetime])
+        end
+      end
+
+      context "with single attachment and options" do
+        before do
+          Dummy.connection.add_attachment :dummies, :avatar, default: '1', file_name: { default: 'default' }
+        end
+
+        it "sets defaults on columns" do
+          defaults_columns = ["avatar_file_name", "avatar_content_type", "avatar_file_size"]
+          columns = Dummy.columns.select { |e| defaults_columns.include? e.name }
+
+          expect(columns).to have_column("avatar_file_name").with_default("default")
+          expect(columns).to have_column("avatar_content_type").with_default("1")
+          expect(columns).to have_column("avatar_file_size").with_default(1)
+        end
+      end
+
+      context "with multiple attachments" do
+        before do
+          Dummy.connection.add_attachment :dummies, :avatar, :photo
+        end
+
+        it "creates attachment columns" do
+          columns = Dummy.columns.map{ |column| [column.name, column.type] }
+
+          expect(columns).to include(['avatar_file_name', :string])
+          expect(columns).to include(['avatar_content_type', :string])
+          expect(columns).to include(['avatar_file_size', :integer])
+          expect(columns).to include(['avatar_updated_at', :datetime])
+          expect(columns).to include(['photo_file_name', :string])
+          expect(columns).to include(['photo_content_type', :string])
+          expect(columns).to include(['photo_file_size', :integer])
+          expect(columns).to include(['photo_updated_at', :datetime])
+        end
+      end
+
+      context "with multiple attachments and options" do
+        before do
+          Dummy.connection.add_attachment :dummies, :avatar, :photo, default: '1', file_name: { default: 'default' }
+        end
+
+        it "sets defaults on columns" do
+          defaults_columns = ["avatar_file_name", "avatar_content_type", "avatar_file_size", "photo_file_name", "photo_content_type", "photo_file_size"]
+          columns = Dummy.columns.select { |e| defaults_columns.include? e.name }
+
+          expect(columns).to have_column("avatar_file_name").with_default("default")
+          expect(columns).to have_column("avatar_content_type").with_default("1")
+          expect(columns).to have_column("avatar_file_size").with_default(1)
+          expect(columns).to have_column("photo_file_name").with_default("default")
+          expect(columns).to have_column("photo_content_type").with_default("1")
+          expect(columns).to have_column("photo_file_size").with_default(1)
+        end
+      end
+
+      context "with no attachment" do
+        it "raises an error" do
+          assert_raises ArgumentError do
+            Dummy.connection.add_attachment :dummies
+          end
+        end
+      end
+    end
+
+    context "migrating down" do
+      before do
+        Dummy.connection.change_table :dummies do |t|
+          t.column :avatar_file_name, :string
+          t.column :avatar_content_type, :string
+          t.column :avatar_file_size, :integer
+          t.column :avatar_updated_at, :datetime
+        end
+      end
+
+      context "using #drop_attached_file" do
+        before do
+          ActiveSupport::Deprecation.silenced = false
+        end
+        it "removes the attachment columns" do
+          ActiveSupport::Deprecation.silence do
+            Dummy.connection.drop_attached_file :dummies, :avatar
+          end
+
+          columns = Dummy.columns.map{ |column| [column.name, column.type] }
+
+          expect(columns).to_not include(['avatar_file_name', :string])
+          expect(columns).to_not include(['avatar_content_type', :string])
+          expect(columns).to_not include(['avatar_file_size', :integer])
+          expect(columns).to_not include(['avatar_updated_at', :datetime])
+        end
+
+        it "displays a deprecation warning" do
+          assert_deprecated do
+            Dummy.connection.drop_attached_file :dummies, :avatar
+          end
+        end
+      end
+
+      context "using #remove_attachment" do
+        context "with single attachment" do
+          before do
+            Dummy.connection.remove_attachment :dummies, :avatar
+          end
+
+          it "removes the attachment columns" do
+            columns = Dummy.columns.map{ |column| [column.name, column.type] }
+
+            expect(columns).to_not include(['avatar_file_name', :string])
+            expect(columns).to_not include(['avatar_content_type', :string])
+            expect(columns).to_not include(['avatar_file_size', :integer])
+            expect(columns).to_not include(['avatar_updated_at', :datetime])
+          end
+        end
+
+        context "with multiple attachments" do
+          before do
+            Dummy.connection.change_table :dummies do |t|
+              t.column :photo_file_name, :string
+              t.column :photo_content_type, :string
+              t.column :photo_file_size, :integer
+              t.column :photo_updated_at, :datetime
+            end
+
+            Dummy.connection.remove_attachment :dummies, :avatar, :photo
+          end
+
+          it "removes the attachment columns" do
+            columns = Dummy.columns.map{ |column| [column.name, column.type] }
+
+            expect(columns).to_not include(['avatar_file_name', :string])
+            expect(columns).to_not include(['avatar_content_type', :string])
+            expect(columns).to_not include(['avatar_file_size', :integer])
+            expect(columns).to_not include(['avatar_updated_at', :datetime])
+            expect(columns).to_not include(['photo_file_name', :string])
+            expect(columns).to_not include(['photo_content_type', :string])
+            expect(columns).to_not include(['photo_file_size', :integer])
+            expect(columns).to_not include(['photo_updated_at', :datetime])
+          end
+        end
+
+        context "with no attachment" do
+          it "raises an error" do
+            assert_raises ArgumentError do
+              Dummy.connection.remove_attachment :dummies
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/{test/storage/filesystem_test.rb → spec/paperclip/storage/filesystem_spec.rb}

@@ -1,33 +1,33 @@
-require './test/helper'
+require 'spec_helper'
 
-class FileSystemTest < Test::Unit::TestCase
+describe Paperclip::Storage::Filesystem do
   context "Filesystem" do
     context "normal file" do
-      setup do
-        rebuild_model :styles => { :thumbnail => "25x25#" }
+      before do
+        rebuild_model styles: { thumbnail: "25x25#" }
         @dummy = Dummy.create!
 
         @file = File.open(fixture_file('5k.png'))
         @dummy.avatar = @file
       end
 
-      teardown { @file.close }
+      after { @file.close }
 
-      should "allow file assignment" do
+      it "allows file assignment" do
         assert @dummy.save
       end
 
-      should "store the original" do
+      it "stores the original" do
         @dummy.save
         assert_file_exists(@dummy.avatar.path)
       end
 
-      should "store the thumbnail" do
+      it "stores the thumbnail" do
         @dummy.save
         assert_file_exists(@dummy.avatar.path(:thumbnail))
       end
 
-      should "be rewinded after flush_writes" do
+      it "is rewinded after flush_writes" do
         @dummy.avatar.instance_eval "def after_flush_writes; end"
 
         files = @dummy.avatar.queued_for_write.values
@@ -35,14 +35,14 @@ class FileSystemTest < Test::Unit::TestCase
         assert files.none?(&:eof?), "Expect all the files to be rewinded."
       end
 
-      should "be removed after after_flush_writes" do
+      it "is removed after after_flush_writes" do
         paths = @dummy.avatar.queued_for_write.values.map(&:path)
         @dummy.save
-        assert paths.none?{ |path| File.exists?(path) },
+        assert paths.none?{ |path| File.exist?(path) },
           "Expect all the files to be deleted."
       end
 
-      should 'copy the file to a known location with copy_to_local_file' do
+      it 'copies the file to a known location with copy_to_local_file' do
         tempfile = Tempfile.new("known_location")
         @dummy.avatar.copy_to_local_file(:original, tempfile.path)
         tempfile.rewind
@@ -52,8 +52,8 @@ class FileSystemTest < Test::Unit::TestCase
     end
 
     context "with file that has space in file name" do
-      setup do
-        rebuild_model :styles => { :thumbnail => "25x25#" }
+      before do
+        rebuild_model styles: { thumbnail: "25x25#" }
         @dummy = Dummy.create!
 
         @file = File.open(fixture_file('spaced file.png'))
@@ -61,17 +61,17 @@ class FileSystemTest < Test::Unit::TestCase
         @dummy.save
       end
 
-      teardown { @file.close }
+      after { @file.close }
 
-      should "store the file" do
+      it "stores the file" do
         assert_file_exists(@dummy.avatar.path)
       end
 
-      should "return a replaced version for path" do
+      it "returns a replaced version for path" do
         assert_match /.+\/spaced_file\.png/, @dummy.avatar.path
       end
 
-      should "return a replaced version for url" do
+      it "returns a replaced version for url" do
         assert_match /.+\/spaced_file\.png/, @dummy.avatar.url
       end
     end