paperclip 4.1.1 → 4.2.0

data/test/matchers/validate_attachment_content_type_matcher_test.rb

@@ -1,111 +0,0 @@
-require './test/helper'
-
-class ValidateAttachmentContentTypeMatcherTest < Test::Unit::TestCase
-  context "validate_attachment_content_type" do
-    setup do
-      reset_table("dummies") do |d|
-        d.string :title
-        d.string :avatar_file_name
-        d.string :avatar_content_type
-      end
-      reset_class "Dummy"
-      Dummy.do_not_validate_attachment_file_type :avatar
-      Dummy.has_attached_file :avatar
-      @matcher     = self.class.validate_attachment_content_type(:avatar).
-                       allowing(%w(image/png image/jpeg)).
-                       rejecting(%w(audio/mp3 application/octet-stream))
-    end
-
-    context "given a class with no validation" do
-      should_reject_dummy_class
-    end
-
-    context "given a class with a validation that doesn't match" do
-      setup do
-        Dummy.validates_attachment_content_type :avatar, :content_type => %r{audio/.*}
-      end
-
-      should_reject_dummy_class
-    end
-
-    context "given a class with a matching validation" do
-      setup do
-        Dummy.validates_attachment_content_type :avatar, :content_type => %r{image/.*}
-      end
-
-      should_accept_dummy_class
-    end
-
-    context "given a class with other validations but matching types" do
-      setup do
-        Dummy.validates_presence_of :title
-        Dummy.validates_attachment_content_type :avatar, :content_type => %r{image/.*}
-      end
-
-      should_accept_dummy_class
-    end
-
-    context "given a class that matches and a matcher that only specifies 'allowing'" do
-      setup do
-        Dummy.validates_attachment_content_type :avatar, :content_type => %r{image/.*}
-        @matcher     = self.class.validate_attachment_content_type(:avatar).
-          allowing(%w(image/png image/jpeg))
-      end
-
-      should_accept_dummy_class
-    end
-
-    context "given a class that does not match and a matcher that only specifies 'allowing'" do
-      setup do
-        Dummy.validates_attachment_content_type :avatar, :content_type => %r{audio/.*}
-        @matcher     = self.class.validate_attachment_content_type(:avatar).
-          allowing(%w(image/png image/jpeg))
-      end
-
-      should_reject_dummy_class
-    end
-
-    context "given a class that matches and a matcher that only specifies 'rejecting'" do
-      setup do
-        Dummy.validates_attachment_content_type :avatar, :content_type => %r{image/.*}
-        @matcher     = self.class.validate_attachment_content_type(:avatar).
-          rejecting(%w(audio/mp3 application/octet-stream))
-      end
-
-      should_accept_dummy_class
-    end
-
-    context "given a class that does not match and a matcher that only specifies 'rejecting'" do
-      setup do
-        Dummy.validates_attachment_content_type :avatar, :content_type => %r{audio/.*}
-        @matcher     = self.class.validate_attachment_content_type(:avatar).
-          rejecting(%w(audio/mp3 application/octet-stream))
-      end
-
-      should_reject_dummy_class
-    end
-
-    context "using an :if to control the validation" do
-      setup do
-        Dummy.class_eval do
-          validates_attachment_content_type :avatar, :content_type => %r{image/*} , :if => :go
-          attr_accessor :go
-        end
-        @matcher = self.class.validate_attachment_content_type(:avatar).
-                        allowing(%w(image/png image/jpeg)).
-                        rejecting(%w(audio/mp3 application/octet-stream))
-        @dummy = Dummy.new
-      end
-
-      should "run the validation if the control is true" do
-        @dummy.go = true
-        assert_accepts @matcher, @dummy
-      end
-
-      should "not run the validation if the control is false" do
-        @dummy.go = false
-        assert_rejects @matcher, @dummy
-      end
-    end
-  end
-end

data/test/matchers/validate_attachment_presence_matcher_test.rb

@@ -1,70 +0,0 @@
-require './test/helper'
-
-class ValidateAttachmentPresenceMatcherTest < Test::Unit::TestCase
-  context "validate_attachment_presence" do
-    setup do
-      reset_table("dummies") do |d|
-        d.string :avatar_file_name
-      end
-      reset_class "Dummy"
-      Dummy.has_attached_file :avatar
-      Dummy.do_not_validate_attachment_file_type :avatar
-      @matcher     = self.class.validate_attachment_presence(:avatar)
-    end
-
-    context "given a class with no validation" do
-      should_reject_dummy_class
-    end
-
-    context "given a class with a matching validation" do
-      setup do
-        Dummy.validates_attachment_presence :avatar
-      end
-
-      should_accept_dummy_class
-    end
-
-    context "given an instance with other attachment validations" do
-      setup do
-        reset_table("dummies") do |d|
-          d.string :avatar_file_name
-          d.string :avatar_content_type
-        end
-
-        Dummy.class_eval do
-          validates_attachment_presence :avatar
-          validates_attachment_content_type :avatar, :content_type => 'image/gif'
-        end
-
-        @dummy = Dummy.new
-        @matcher = self.class.validate_attachment_presence(:avatar)
-      end
-
-      should "it should validate properly" do
-        @dummy.avatar = File.new fixture_file('5k.png')
-        assert_accepts @matcher, @dummy
-      end
-    end
-
-    context "using an :if to control the validation" do
-      setup do
-        Dummy.class_eval do
-          validates_attachment_presence :avatar, :if => :go
-          attr_accessor :go
-        end
-        @dummy = Dummy.new
-        @dummy.avatar = nil
-      end
-
-      should "run the validation if the control is true" do
-        @dummy.go = true
-        assert_accepts @matcher, @dummy
-      end
-
-      should "not run the validation if the control is false" do
-        @dummy.go = false
-        assert_rejects @matcher, @dummy
-      end
-    end
-  end
-end

data/test/matchers/validate_attachment_size_matcher_test.rb

@@ -1,87 +0,0 @@
-require './test/helper'
-
-class ValidateAttachmentSizeMatcherTest < Test::Unit::TestCase
-  context "validate_attachment_size" do
-    setup do
-      reset_table("dummies") do |d|
-        d.string :avatar_file_name
-        d.integer :avatar_file_size
-      end
-      reset_class "Dummy"
-      Dummy.do_not_validate_attachment_file_type :avatar
-      Dummy.has_attached_file :avatar
-    end
-
-    context "of limited size" do
-      setup{ @matcher = self.class.validate_attachment_size(:avatar).in(256..1024) }
-
-      context "given a class with no validation" do
-        should_reject_dummy_class
-      end
-
-      context "given a class with a validation that's too high" do
-        setup { Dummy.validates_attachment_size :avatar, :in => 256..2048 }
-        should_reject_dummy_class
-      end
-
-      context "given a class with a validation that's too low" do
-        setup { Dummy.validates_attachment_size :avatar, :in => 0..1024 }
-        should_reject_dummy_class
-      end
-
-      context "given a class with a validation that matches" do
-        setup { Dummy.validates_attachment_size :avatar, :in => 256..1024 }
-        should_accept_dummy_class
-      end
-    end
-
-    context "allowing anything" do
-      setup{ @matcher = self.class.validate_attachment_size(:avatar) }
-
-      context "given a class with an upper limit" do
-        setup { Dummy.validates_attachment_size :avatar, :less_than => 1 }
-        should_accept_dummy_class
-      end
-
-      context "given a class with a lower limit" do
-        setup { Dummy.validates_attachment_size :avatar, :greater_than => 1 }
-        should_accept_dummy_class
-      end
-    end
-
-    context "using an :if to control the validation" do
-      setup do
-        Dummy.class_eval do
-          validates_attachment_size :avatar, :greater_than => 1024, :if => :go
-          attr_accessor :go
-        end
-        @dummy = Dummy.new
-        @matcher = self.class.validate_attachment_size(:avatar).greater_than(1024)
-      end
-
-      should "run the validation if the control is true" do
-        @dummy.go = true
-        assert_accepts @matcher, @dummy
-      end
-
-      should "not run the validation if the control is false" do
-        @dummy.go = false
-        assert_rejects @matcher, @dummy
-      end
-    end
-
-    context "post processing" do
-      setup do
-        Dummy.validates_attachment_size :avatar, :greater_than => 1024
-
-        @dummy = Dummy.new
-        @matcher = self.class.validate_attachment_size(:avatar).greater_than(1024)
-      end
-
-      should "be skipped" do
-        @dummy.avatar.expects(:post_process).never
-        assert_accepts @matcher, @dummy
-      end
-    end
-  end
-end

data/test/meta_class_test.rb

@@ -1,32 +0,0 @@
-require './test/helper'
-
-class MetaClassTest < Test::Unit::TestCase
-  context "A meta-class of dummy" do
-    setup do
-      rebuild_model
-      @file = File.new(fixture_file("5k.png"), 'rb')
-    end
-
-    teardown { @file.close }
-
-    should "be able to use Paperclip like a normal class" do
-      reset_class("Dummy")
-      @dummy = Dummy.new
-
-      assert_nothing_raised do
-        rebuild_meta_class_of(@dummy)
-      end
-    end
-
-    should "work like any other instance" do
-      reset_class("Dummy")
-      @dummy = Dummy.new
-      rebuild_meta_class_of(@dummy)
-
-      assert_nothing_raised do
-        @dummy.avatar = @file
-      end
-      assert @dummy.save
-    end
-  end
-end

data/test/paperclip_missing_attachment_styles_test.rb

@@ -1,90 +0,0 @@
-require './test/helper'
-
-class PaperclipMissingAttachmentStylesTest < Test::Unit::TestCase
-  def setup
-    Paperclip::AttachmentRegistry.clear
-  end
-
-  context "Paperclip" do
-    setup do
-      Paperclip::AttachmentRegistry.clear
-    end
-
-    teardown do
-      File.unlink(Paperclip.registered_attachments_styles_path) rescue nil
-    end
-
-    should "enable to get and set path to registered styles file" do
-      assert_equal ROOT.join('tmp/public/system/paperclip_attachments.yml').to_s, Paperclip.registered_attachments_styles_path
-      Paperclip.registered_attachments_styles_path = '/tmp/config/paperclip_attachments.yml'
-      assert_equal '/tmp/config/paperclip_attachments.yml', Paperclip.registered_attachments_styles_path
-      Paperclip.registered_attachments_styles_path = nil
-      assert_equal ROOT.join('tmp/public/system/paperclip_attachments.yml').to_s, Paperclip.registered_attachments_styles_path
-    end
-
-    should "be able to get current attachment styles" do
-      assert_equal Hash.new, Paperclip.send(:current_attachments_styles)
-      rebuild_model :styles => {:croppable => '600x600>', :big => '1000x1000>'}
-      expected_hash = { :Dummy => {:avatar => [:big, :croppable]}}
-      assert_equal expected_hash, Paperclip.send(:current_attachments_styles)
-    end
-
-    should "be able to save current attachment styles for further comparison" do
-      rebuild_model :styles => {:croppable => '600x600>', :big => '1000x1000>'}
-      Paperclip.save_current_attachments_styles!
-      expected_hash = { :Dummy => {:avatar => [:big, :croppable]}}
-      assert_equal expected_hash, YAML.load_file(Paperclip.registered_attachments_styles_path)
-    end
-
-    should "be able to read registered attachment styles from file" do
-      rebuild_model :styles => {:croppable => '600x600>', :big => '1000x1000>'}
-      Paperclip.save_current_attachments_styles!
-      expected_hash = { :Dummy => {:avatar => [:big, :croppable]}}
-      assert_equal expected_hash, Paperclip.send(:get_registered_attachments_styles)
-    end
-
-    should "be able to calculate differences between registered styles and current styles" do
-      rebuild_model :styles => {:croppable => '600x600>', :big => '1000x1000>'}
-      Paperclip.save_current_attachments_styles!
-      rebuild_model :styles => {:thumb => 'x100', :export => 'x400>', :croppable => '600x600>', :big => '1000x1000>'}
-      expected_hash = { :Dummy => {:avatar => [:export, :thumb]} }
-      assert_equal expected_hash, Paperclip.missing_attachments_styles
-
-      ActiveRecord::Base.connection.create_table :books, :force => true
-      class ::Book < ActiveRecord::Base
-        has_attached_file :cover, :styles => {:small => 'x100', :large => '1000x1000>'}
-        has_attached_file :sample, :styles => {:thumb => 'x100'}
-      end
-
-      expected_hash = {
-        :Dummy => {:avatar => [:export, :thumb]},
-        :Book => {:sample => [:thumb], :cover => [:large, :small]}
-      }
-      assert_equal expected_hash, Paperclip.missing_attachments_styles
-      Paperclip.save_current_attachments_styles!
-      assert_equal Hash.new, Paperclip.missing_attachments_styles
-    end
-
-    should "be able to calculate differences when a new attachment is added to a model" do
-      rebuild_model :styles => {:croppable => '600x600>', :big => '1000x1000>'}
-      Paperclip.save_current_attachments_styles!
-
-      class ::Dummy
-        has_attached_file :photo, :styles => {:small => 'x100', :large => '1000x1000>'}
-      end
-
-      expected_hash = {
-        :Dummy => {:photo => [:large, :small]}
-      }
-      assert_equal expected_hash, Paperclip.missing_attachments_styles
-      Paperclip.save_current_attachments_styles!
-      assert_equal Hash.new, Paperclip.missing_attachments_styles
-    end
-
-    # It's impossible to build styles hash without loading from database whole bunch of records
-    should "skip lambda-styles" do
-      rebuild_model :styles => lambda{ |attachment| attachment.instance.other == 'a' ? {:thumb => "50x50#"} : {:large => "400x400"} }
-      assert_equal Hash.new, Paperclip.send(:current_attachments_styles)
-    end
-  end
-end

data/test/schema_test.rb

@@ -1,206 +0,0 @@
-require './test/helper'
-require 'paperclip/schema'
-require 'active_support/testing/deprecation'
-
-class SchemaTest < Test::Unit::TestCase
-  include ActiveSupport::Testing::Deprecation
-
-  def setup
-    rebuild_class
-  end
-
-  def teardown
-    Dummy.connection.drop_table :dummies rescue nil
-  end
-
-  context "within table definition" do
-    context "using #has_attached_file" do
-      setup do
-        ActiveSupport::Deprecation.silenced = false
-      end
-      should "create attachment columns" do
-        Dummy.connection.create_table :dummies, :force => true do |t|
-          ActiveSupport::Deprecation.silence do
-            t.has_attached_file :avatar
-          end
-        end
-        rebuild_class
-
-        columns = Dummy.columns.map{ |column| [column.name, column.type] }
-
-        assert_includes columns, ['avatar_file_name', :string]
-        assert_includes columns, ['avatar_content_type', :string]
-        assert_includes columns, ['avatar_file_size', :integer]
-        assert_includes columns, ['avatar_updated_at', :datetime]
-      end
-
-      should "display 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
-      setup do
-        Dummy.connection.create_table :dummies, :force => true do |t|
-          t.attachment :avatar
-        end
-        rebuild_class
-      end
-
-      should "create attachment columns" do
-        columns = Dummy.columns.map{ |column| [column.name, column.type] }
-
-        assert_includes columns, ['avatar_file_name', :string]
-        assert_includes columns, ['avatar_content_type', :string]
-        assert_includes columns, ['avatar_file_size', :integer]
-        assert_includes columns, ['avatar_updated_at', :datetime]
-      end
-    end
-  end
-
-  context "within schema statement" do
-    setup do
-      Dummy.connection.create_table :dummies, :force => true
-    end
-
-    context "migrating up" do
-      context "with single attachment" do
-        setup do
-          Dummy.connection.add_attachment :dummies, :avatar
-          rebuild_class
-        end
-
-        should "create attachment columns" do
-          columns = Dummy.columns.map{ |column| [column.name, column.type] }
-
-          assert_includes columns, ['avatar_file_name', :string]
-          assert_includes columns, ['avatar_content_type', :string]
-          assert_includes columns, ['avatar_file_size', :integer]
-          assert_includes columns, ['avatar_updated_at', :datetime]
-        end
-      end
-
-      context "with multiple attachments" do
-        setup do
-          Dummy.connection.add_attachment :dummies, :avatar, :photo
-          rebuild_class
-        end
-
-        should "create attachment columns" do
-          columns = Dummy.columns.map{ |column| [column.name, column.type] }
-
-          assert_includes columns, ['avatar_file_name', :string]
-          assert_includes columns, ['avatar_content_type', :string]
-          assert_includes columns, ['avatar_file_size', :integer]
-          assert_includes columns, ['avatar_updated_at', :datetime]
-          assert_includes columns, ['photo_file_name', :string]
-          assert_includes columns, ['photo_content_type', :string]
-          assert_includes columns, ['photo_file_size', :integer]
-          assert_includes columns, ['photo_updated_at', :datetime]
-        end
-      end
-
-      context "with no attachment" do
-        should "raise an error" do
-          assert_raise ArgumentError do
-            Dummy.connection.add_attachment :dummies
-          rebuild_class
-          end
-        end
-      end
-    end
-
-    context "migrating down" do
-      setup 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
-        setup do
-          ActiveSupport::Deprecation.silenced = false
-        end
-        should "remove the attachment columns" do
-          ActiveSupport::Deprecation.silence do
-            Dummy.connection.drop_attached_file :dummies, :avatar
-          end
-          rebuild_class
-
-          columns = Dummy.columns.map{ |column| [column.name, column.type] }
-
-          assert_not_includes columns, ['avatar_file_name', :string]
-          assert_not_includes columns, ['avatar_content_type', :string]
-          assert_not_includes columns, ['avatar_file_size', :integer]
-          assert_not_includes columns, ['avatar_updated_at', :datetime]
-        end
-
-        should "display 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
-          setup do
-            Dummy.connection.remove_attachment :dummies, :avatar
-            rebuild_class
-          end
-
-          should "remove the attachment columns" do
-            columns = Dummy.columns.map{ |column| [column.name, column.type] }
-
-            assert_not_includes columns, ['avatar_file_name', :string]
-            assert_not_includes columns, ['avatar_content_type', :string]
-            assert_not_includes columns, ['avatar_file_size', :integer]
-            assert_not_includes columns, ['avatar_updated_at', :datetime]
-          end
-        end
-
-        context "with multiple attachments" do
-          setup 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
-            rebuild_class
-          end
-
-          should "remove the attachment columns" do
-            columns = Dummy.columns.map{ |column| [column.name, column.type] }
-
-            assert_not_includes columns, ['avatar_file_name', :string]
-            assert_not_includes columns, ['avatar_content_type', :string]
-            assert_not_includes columns, ['avatar_file_size', :integer]
-            assert_not_includes columns, ['avatar_updated_at', :datetime]
-            assert_not_includes columns, ['photo_file_name', :string]
-            assert_not_includes columns, ['photo_content_type', :string]
-            assert_not_includes columns, ['photo_file_size', :integer]
-            assert_not_includes columns, ['photo_updated_at', :datetime]
-          end
-        end
-
-        context "with no attachment" do
-          should "raise an error" do
-            assert_raise ArgumentError do
-              Dummy.connection.remove_attachment :dummies
-            end
-          end
-        end
-      end
-    end
-  end
-end