paperclip 3.5.2 → 5.2.1

data/test/io_adapters/data_uri_adapter_test.rb

@@ -1,67 +0,0 @@
-require './test/helper'
-
-class DataUriAdapterTest < Test::Unit::TestCase
-  context "a new instance" do
-    setup do
-      @contents = "data:image/png;base64,#{original_base64_content}"
-      @subject = Paperclip.io_adapters.for(@contents)
-    end
-
-    should "return a file name" do
-      assert_equal "base64.txt", @subject.original_filename
-    end
-
-    should "return a content type" do
-      assert_equal "image/png", @subject.content_type
-    end
-
-    should "return the size of the data" do
-      assert_equal 4456, @subject.size
-    end
-
-    should "generate a correct MD5 hash of the contents" do
-      assert_equal(
-        Digest::MD5.hexdigest(Base64.decode64(original_base64_content)),
-        @subject.fingerprint
-      )
-    end
-
-    should "generate correct fingerprint after read" do
-      fingerprint = Digest::MD5.hexdigest(@subject.read)
-      assert_equal fingerprint, @subject.fingerprint
-    end
-
-    should "generate same fingerprint" do
-      assert_equal @subject.fingerprint, @subject.fingerprint
-    end
-
-    should 'accept a content_type' do
-      @subject.content_type = 'image/png'
-      assert_equal 'image/png', @subject.content_type
-    end
-
-    should 'accept an original_filename' do
-      @subject.original_filename = 'image.png'
-      assert_equal 'image.png', @subject.original_filename
-    end
-
-    should "not generate filenames that include restricted characters" do
-      @subject.original_filename = 'image:restricted.png'
-      assert_equal 'image_restricted.png', @subject.original_filename
-    end
-
-    should "not generate paths that include restricted characters" do
-      @subject.original_filename = 'image:restricted.png'
-      assert_no_match /:/, @subject.path
-    end
-
-  end
-
-  def original_base64_content
-    Base64.encode64(original_file_contents)
-  end
-
-  def original_file_contents
-    @original_file_contents ||= File.read(fixture_file('5k.png'))
-  end
-end

data/test/io_adapters/empty_string_adapter_test.rb

@@ -1,17 +0,0 @@
-require './test/helper'
-
-class EmptyStringAdapterTest < Test::Unit::TestCase
-  context 'a new instance' do
-    setup do
-      @subject = Paperclip.io_adapters.for('')
-    end
-
-    should "return false for a call to nil?" do
-      assert !@subject.nil?
-    end
-
-    should 'return false for a call to assignment?' do
-      assert !@subject.assignment?
-    end
-  end
-end

data/test/io_adapters/file_adapter_test.rb

@@ -1,119 +0,0 @@
-require './test/helper'
-
-class FileAdapterTest < Test::Unit::TestCase
-  context "a new instance" do
-    context "with normal file" do
-      setup do
-        @file = File.new(fixture_file("5k.png"))
-        @file.binmode
-        @subject = Paperclip.io_adapters.for(@file)
-      end
-
-      teardown { @file.close }
-
-      should "get the right filename" do
-        assert_equal "5k.png", @subject.original_filename
-      end
-
-      should "force binmode on tempfile" do
-        assert @subject.instance_variable_get("@tempfile").binmode?
-      end
-
-      should "get the content type" do
-        assert_equal "image/png", @subject.content_type
-      end
-
-      should "return content type as a string" do
-        assert_kind_of String, @subject.content_type
-      end
-
-      should "get the file's size" do
-        assert_equal 4456, @subject.size
-      end
-
-      should "return false for a call to nil?" do
-        assert ! @subject.nil?
-      end
-
-      should "generate a MD5 hash of the contents" do
-        expected = Digest::MD5.file(@file.path).to_s
-        assert_equal expected, @subject.fingerprint
-      end
-
-      should "read the contents of the file" do
-        expected = @file.read
-        assert expected.length > 0
-        assert_equal expected, @subject.read
-      end
-
-      context "file with multiple possible content type" do
-        setup do
-          MIME::Types.stubs(:type_for).returns([MIME::Type.new('image/x-png'), MIME::Type.new('image/png')])
-        end
-
-        should "prefer officially registered mime type" do
-          assert_equal "image/png", @subject.content_type
-        end
-
-        should "return content type as a string" do
-          assert_kind_of String, @subject.content_type
-        end
-      end
-
-      context "file with multiple possible x-types but no official type" do
-        setup do
-          MIME::Types.stubs(:type_for).returns([MIME::Type.new('image/x-mp4'), MIME::Type.new('image/x-video')])
-          @subject = Paperclip.io_adapters.for(@file)
-        end
-
-        should "return the first" do
-          assert_equal "image/x-mp4", @subject.content_type
-        end
-      end
-
-      context "file with content type derived from file command on *nix" do
-        setup do
-          MIME::Types.stubs(:type_for).returns([])
-          Paperclip.stubs(:run).returns("application/vnd.ms-office\n")
-          @subject = Paperclip.io_adapters.for(@file)
-        end
-
-        should "return content type without newline character" do
-          assert_equal "application/vnd.ms-office", @subject.content_type
-        end
-      end
-    end
-
-    context "filename with restricted characters" do
-      setup do
-        file_contents = File.new(fixture_file("animated.gif"))
-        @file = StringIO.new(file_contents.read)
-        @file.stubs(:original_filename).returns('image:restricted.gif')
-        @subject = Paperclip.io_adapters.for(@file)
-      end
-
-      teardown { @file.close }
-
-      should "not generate filenames that include restricted characters" do
-        assert_equal 'image_restricted.gif', @subject.original_filename
-      end
-
-      should "not generate paths that include restricted characters" do
-        assert_no_match /:/, @subject.path
-      end
-    end
-
-    context "empty file" do
-      setup do
-        @file = Tempfile.new("file_adapter_test")
-        @subject = Paperclip.io_adapters.for(@file)
-      end
-
-      teardown { @file.close }
-
-      should "provide correct mime-type" do
-        assert_match %r{.*/x-empty}, @subject.content_type
-      end
-    end
-  end
-end

data/test/io_adapters/http_url_proxy_adapter_test.rb

@@ -1,102 +0,0 @@
-require './test/helper'
-
-class HttpUrlProxyAdapterTest < Test::Unit::TestCase
-  context "a new instance" do
-    setup do
-      @open_return = StringIO.new("xxx")
-      @open_return.stubs(:content_type).returns("image/png")
-      Paperclip::HttpUrlProxyAdapter.any_instance.stubs(:download_content).returns(@open_return)
-      @url = "http://thoughtbot.com/images/thoughtbot-logo.png"
-      @subject = Paperclip.io_adapters.for(@url)
-    end
-
-    teardown do
-      @subject.close
-    end
-
-    should "return a file name" do
-      assert_equal "thoughtbot-logo.png", @subject.original_filename
-    end
-
-    should 'close open handle after reading' do
-      assert_equal true, @open_return.closed?
-    end
-
-    should "return a content type" do
-      assert_equal "image/png", @subject.content_type
-    end
-
-    should "return the size of the data" do
-      assert_equal @open_return.size, @subject.size
-    end
-
-    should "generate an MD5 hash of the contents" do
-      assert_equal Digest::MD5.hexdigest("xxx"), @subject.fingerprint
-    end
-
-    should "generate correct fingerprint after read" do
-      fingerprint = Digest::MD5.hexdigest(@subject.read)
-      assert_equal fingerprint, @subject.fingerprint
-    end
-
-    should "generate same fingerprint" do
-      assert_equal @subject.fingerprint, @subject.fingerprint
-    end
-
-    should "return the data contained in the StringIO" do
-      assert_equal "xxx", @subject.read
-    end
-
-    should 'accept a content_type' do
-      @subject.content_type = 'image/png'
-      assert_equal 'image/png', @subject.content_type
-    end
-
-    should 'accept an original_filename' do
-      @subject.original_filename = 'image.png'
-      assert_equal 'image.png', @subject.original_filename
-    end
-  end
-
-  context "a url with query params" do
-    setup do
-      Paperclip::HttpUrlProxyAdapter.any_instance.stubs(:download_content).returns(StringIO.new("x"))
-      @url = "https://github.com/thoughtbot/paperclip?file=test"
-      @subject = Paperclip.io_adapters.for(@url)
-    end
-
-    teardown do
-      @subject.close
-    end
-
-    should "return a file name" do
-      assert_equal "paperclip", @subject.original_filename
-    end
-  end
-
-  context "a url with restricted characters in the filename" do
-    setup do
-      Paperclip::HttpUrlProxyAdapter.any_instance.stubs(:download_content).returns(StringIO.new("x"))
-      @url = "https://github.com/thoughtbot/paper:clip.jpg"
-      @subject = Paperclip.io_adapters.for(@url)
-    end
-
-    teardown do
-      begin
-        @subject.close
-      rescue Exception => e
-        binding.pry
-        true
-      end
-    end
-
-    should "not generate filenames that include restricted characters" do
-      assert_equal "paper_clip.jpg", @subject.original_filename
-    end
-
-    should "not generate paths that include restricted characters" do
-      assert_no_match /:/, @subject.path
-    end
-  end
-
-end

data/test/io_adapters/identity_adapter_test.rb

@@ -1,8 +0,0 @@
-require './test/helper'
-
-class IdentityAdapterTest < Test::Unit::TestCase
-  should "respond to #new by returning the argument" do
-    adapter = Paperclip::IdentityAdapter.new
-    assert_equal :target, adapter.new(:target)
-  end
-end

data/test/io_adapters/uri_adapter_test.rb

@@ -1,102 +0,0 @@
-require './test/helper'
-
-class UriProxyTest < Test::Unit::TestCase
-  context "a new instance" do
-    setup do
-      @open_return = StringIO.new("xxx")
-      @open_return.stubs(:content_type).returns("image/png")
-      Paperclip::UriAdapter.any_instance.stubs(:download_content).returns(@open_return)
-      @uri = URI.parse("http://thoughtbot.com/images/thoughtbot-logo.png")
-      @subject = Paperclip.io_adapters.for(@uri)
-    end
-
-    should "return a file name" do
-      assert_equal "thoughtbot-logo.png", @subject.original_filename
-    end
-
-    should 'close open handle after reading' do
-      assert_equal true, @open_return.closed?
-    end
-
-    should "return a content type" do
-      assert_equal "image/png", @subject.content_type
-    end
-
-    should "return the size of the data" do
-      assert_equal @open_return.size, @subject.size
-    end
-
-    should "generate an MD5 hash of the contents" do
-      assert_equal Digest::MD5.hexdigest("xxx"), @subject.fingerprint
-    end
-
-    should "generate correct fingerprint after read" do
-      fingerprint = Digest::MD5.hexdigest(@subject.read)
-      assert_equal fingerprint, @subject.fingerprint
-    end
-
-    should "generate same fingerprint" do
-      assert_equal @subject.fingerprint, @subject.fingerprint
-    end
-
-    should "return the data contained in the StringIO" do
-      assert_equal "xxx", @subject.read
-    end
-
-    should 'accept a content_type' do
-      @subject.content_type = 'image/png'
-      assert_equal 'image/png', @subject.content_type
-    end
-
-    should 'accept an orgiginal_filename' do
-      @subject.original_filename = 'image.png'
-      assert_equal 'image.png', @subject.original_filename
-    end
-
-  end
-
-  context "a directory index url" do
-    setup do
-      Paperclip::UriAdapter.any_instance.stubs(:download_content).returns(StringIO.new("xxx"))
-      @uri = URI.parse("http://thoughtbot.com")
-      @subject = Paperclip.io_adapters.for(@uri)
-    end
-
-    should "return a file name" do
-      assert_equal "index.html", @subject.original_filename
-    end
-
-    should "return a content type" do
-      assert_equal "text/html", @subject.content_type
-    end
-  end
-
-  context "a url with query params" do
-    setup do
-      Paperclip::UriAdapter.any_instance.stubs(:download_content).returns(StringIO.new("xxx"))
-      @uri = URI.parse("https://github.com/thoughtbot/paperclip?file=test")
-      @subject = Paperclip.io_adapters.for(@uri)
-    end
-
-    should "return a file name" do
-      assert_equal "paperclip", @subject.original_filename
-    end
-  end
-
-  context "a url with restricted characters in the filename" do
-    setup do
-      Paperclip::UriAdapter.any_instance.stubs(:download_content).returns(StringIO.new("xxx"))
-      @uri = URI.parse("https://github.com/thoughtbot/paper:clip.jpg")
-      @subject = Paperclip.io_adapters.for(@uri)
-    end
-
-    should "not generate filenames that include restricted characters" do
-      assert_equal "paper_clip.jpg", @subject.original_filename
-    end
-
-    should "not generate paths that include restricted characters" do
-      assert_no_match /:/, @subject.path
-    end
-  end
-
-end

data/test/matchers/have_attached_file_matcher_test.rb

@@ -1,24 +0,0 @@
-require './test/helper'
-
-class HaveAttachedFileMatcherTest < Test::Unit::TestCase
-  context "have_attached_file" do
-    setup do
-      @dummy_class = reset_class "Dummy"
-      reset_table "dummies"
-      @matcher     = self.class.have_attached_file(:avatar)
-    end
-
-    context "given a class with no attachment" do
-      should_reject_dummy_class
-    end
-
-    context "given a class with an attachment" do
-      setup do
-        modify_table("dummies"){|d| d.string :avatar_file_name }
-        @dummy_class.has_attached_file :avatar
-      end
-
-      should_accept_dummy_class
-    end
-  end
-end

data/test/matchers/validate_attachment_content_type_matcher_test.rb

@@ -1,110 +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
-      @dummy_class = reset_class "Dummy"
-      @dummy_class.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_class.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_class.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_class.validates_presence_of :title
-        @dummy_class.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_class.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_class.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_class.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_class.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.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_class.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,69 +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
-      @dummy_class = reset_class "Dummy"
-      @dummy_class.has_attached_file :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_class.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.class_eval do
-          validates_attachment_presence :avatar
-          validates_attachment_content_type :avatar, :content_type => 'image/gif'
-        end
-
-        @dummy = @dummy_class.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.class_eval do
-          validates_attachment_presence :avatar, :if => :go
-          attr_accessor :go
-        end
-        @dummy = @dummy_class.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