jr-paperclip 8.0.1 → 8.0.3

data/spec/paperclip/content_type_detector_spec.rb

@@ -1,58 +0,0 @@
-require "spec_helper"
-
-describe Paperclip::ContentTypeDetector do
-  it "returns a meaningful content type for open xml spreadsheets" do
-    file = File.new(fixture_file("empty.xlsx"))
-    assert_equal "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet",
-                 Paperclip::ContentTypeDetector.new(file.path).detect
-  end
-
-  it 'returns a more specific content type based on the filename if it matches
-      multiple content types' do
-    file = File.new(fixture_file('sample.xlsm'))
-    assert_equal 'application/vnd.ms-excel.sheet.macroenabled.12',
-                 Paperclip::ContentTypeDetector.new(file.path).detect
-  end
-
-  it "gives a sensible default when the name is empty" do
-    assert_equal "application/octet-stream", Paperclip::ContentTypeDetector.new("").detect
-  end
-
-  it "returns the empty content type when the file is empty" do
-    tempfile = Tempfile.new("empty")
-    assert_equal "inode/x-empty", Paperclip::ContentTypeDetector.new(tempfile.path).detect
-    tempfile.close
-  end
-
-  it "returns content type of file if it is an acceptable type" do
-    allow(MIME::Types).to receive(:type_for).and_return([
-      MIME::Type.new("content-type" => "application/mp4"),
-      MIME::Type.new("content-type" => "video/mp4"),
-      MIME::Type.new("content-type" => "audio/mp4"),
-    ])
-    allow_any_instance_of(Paperclip::ContentTypeDetector).to receive(:type_from_file_contents).and_return("video/mp4")
-    @filename = "my_file.mp4"
-    assert_equal "video/mp4", Paperclip::ContentTypeDetector.new(@filename).detect
-  end
-
-  it "finds the right type in the list via the file command" do
-    @filename = "#{Dir.tmpdir}/something.hahalolnotreal"
-    File.open(@filename, "w+") do |file|
-      file.puts "This is a text file."
-      file.rewind
-      assert_equal "text/plain", Paperclip::ContentTypeDetector.new(file.path).detect
-    end
-    FileUtils.rm @filename
-  end
-
-  it "returns a sensible default if something is wrong, like the file is gone" do
-    @filename = "/path/to/nothing"
-    assert_equal "application/octet-stream", Paperclip::ContentTypeDetector.new(@filename).detect
-  end
-
-  it "returns a sensible default when the file command is missing" do
-    allow(Paperclip).to receive(:run).and_raise(Terrapin::CommandLineError.new)
-    @filename = "/path/to/something"
-    assert_equal "application/octet-stream", Paperclip::ContentTypeDetector.new(@filename).detect
-  end
-end

data/spec/paperclip/file_command_content_type_detector_spec.rb

@@ -1,40 +0,0 @@
-require "spec_helper"
-
-describe Paperclip::FileCommandContentTypeDetector do
-  it "returns a content type based on the content of the file" do
-    tempfile = Tempfile.new("something")
-    tempfile.write("This is a file.")
-    tempfile.rewind
-
-    assert_equal "text/plain", Paperclip::FileCommandContentTypeDetector.new(tempfile.path).detect
-
-    tempfile.close
-  end
-
-  it "returns a sensible default when the file command is missing" do
-    allow(Paperclip).to receive(:run).and_raise(Terrapin::CommandLineError.new)
-    @filename = "/path/to/something"
-    assert_equal "application/octet-stream",
-                 Paperclip::FileCommandContentTypeDetector.new(@filename).detect
-  end
-
-  it "returns a sensible default on the odd chance that run returns nil" do
-    allow(Paperclip).to receive(:run).and_return(nil)
-    assert_equal "application/octet-stream",
-                 Paperclip::FileCommandContentTypeDetector.new("windows").detect
-  end
-
-  context "#type_from_file_command" do
-    let(:detector) { Paperclip::FileCommandContentTypeDetector.new("html") }
-
-    it "does work with the output of old versions of file" do
-      allow(Paperclip).to receive(:run).and_return("text/html charset=us-ascii")
-      expect(detector.detect).to eq("text/html")
-    end
-
-    it "does work with the output of new versions of file" do
-      allow(Paperclip).to receive(:run).and_return("text/html; charset=us-ascii")
-      expect(detector.detect).to eq("text/html")
-    end
-  end
-end

data/spec/paperclip/filename_cleaner_spec.rb

@@ -1,13 +0,0 @@
-require "spec_helper"
-
-describe Paperclip::FilenameCleaner do
-  it "converts invalid characters to underscores" do
-    cleaner = Paperclip::FilenameCleaner.new(/[aeiou]/)
-    expect(cleaner.call("baseball")).to eq "b_s_b_ll"
-  end
-
-  it "does not convert anything if the character regex is nil" do
-    cleaner = Paperclip::FilenameCleaner.new(nil)
-    expect(cleaner.call("baseball")).to eq "baseball"
-  end
-end

data/spec/paperclip/geometry_detector_spec.rb

@@ -1,96 +0,0 @@
-require "spec_helper"
-
-describe Paperclip::GeometryDetector do
-  [:image_magick, :vips].each do |backend|
-    context "when configured to use #{backend}" do
-      let(:original_backend) { Paperclip.options[:backend] }
-
-      before do
-        Paperclip.options[:backend] = backend
-      end
-
-      after do
-        Paperclip.options[:backend] = original_backend
-      end
-
-      it "identifies an image and extract its dimensions" do
-        allow_any_instance_of(Paperclip::GeometryParser).to receive(:make).and_return(:correct)
-        file = fixture_file("5k.png")
-        factory = Paperclip::GeometryDetector.new(file)
-
-        output = factory.make
-
-        expect(output).to eq :correct
-      end
-
-      it "identifies an image and extract its dimensions and orientation" do
-        allow_any_instance_of(Paperclip::GeometryParser).to receive(:make).and_return(:correct)
-        file = fixture_file("rotated.jpg")
-        factory = Paperclip::GeometryDetector.new(file)
-
-        output = factory.make
-
-        expect(output).to eq :correct
-      end
-
-      it "avoids reading EXIF orientation if so configured" do
-        begin
-          Paperclip.options[:use_exif_orientation] = false
-          allow_any_instance_of(Paperclip::GeometryParser).to receive(:make).and_return(:correct)
-          file = fixture_file("rotated.jpg")
-          factory = Paperclip::GeometryDetector.new(file)
-
-          output = factory.make
-
-          expect(output).to eq :correct
-        ensure
-          Paperclip.options[:use_exif_orientation] = true
-        end
-      end
-
-      it "raises an exception with a message when the file is not an image" do
-        file = fixture_file("text.txt")
-        factory = Paperclip::GeometryDetector.new(file)
-
-        expect do
-          factory.make
-        end.to raise_error(Paperclip::Errors::NotIdentifiedByBackendError, "Could not identify image size")
-      end
-
-      it "uses the correct backend to identify the image" do
-        if backend == :vips
-          begin
-            require "vips"
-          rescue LoadError
-            skip "ruby-vips gem not available"
-          end
-          expect(Vips::Image).to receive(:new_from_file).and_call_original
-          expect(Paperclip).not_to receive(:run).with(include("identify"), any_args)
-        else
-          expect(Paperclip).to receive(:run).with(include("identify"), any_args).and_call_original
-        end
-
-        file = fixture_file("5k.png")
-        factory = Paperclip::GeometryDetector.new(file)
-
-        geometry = factory.make
-        expect(geometry.width).to eq(434)
-        expect(geometry.height).to eq(66)
-      end
-
-      if backend == :vips
-        it "raises CommandNotFoundError (and not NameError) when vips is missing" do
-          hide_const("Vips")
-          allow_any_instance_of(Paperclip::GeometryDetector).to receive(:require).with("vips").and_raise(LoadError)
-
-          file = fixture_file("5k.png")
-          factory = Paperclip::GeometryDetector.new(file)
-
-          expect {
-            factory.make
-          }.to raise_error(Paperclip::Errors::CommandNotFoundError, /Could not load ruby-vips/)
-        end
-      end
-    end
-  end
-end

data/spec/paperclip/geometry_parser_spec.rb

@@ -1,73 +0,0 @@
-require "spec_helper"
-
-describe Paperclip::GeometryParser do
-  it "identifies an image and extract its dimensions with no orientation" do
-    allow(Paperclip::Geometry).to receive(:new).with(
-      height: "73",
-      width: "434",
-      modifier: nil,
-      orientation: nil
-    ).and_return(:correct)
-    factory = Paperclip::GeometryParser.new("434x73")
-
-    output = factory.make
-
-    assert_equal :correct, output
-  end
-
-  it "identifies an image and extract its dimensions with an empty orientation" do
-    allow(Paperclip::Geometry).to receive(:new).with(
-      height: "73",
-      width: "434",
-      modifier: nil,
-      orientation: ""
-    ).and_return(:correct)
-    factory = Paperclip::GeometryParser.new("434x73,")
-
-    output = factory.make
-
-    assert_equal :correct, output
-  end
-
-  it "identifies an image and extract its dimensions and orientation" do
-    allow(Paperclip::Geometry).to receive(:new).with(
-      height: "200",
-      width: "300",
-      modifier: nil,
-      orientation: "6"
-    ).and_return(:correct)
-    factory = Paperclip::GeometryParser.new("300x200,6")
-
-    output = factory.make
-
-    assert_equal :correct, output
-  end
-
-  it "identifies an image and extract its dimensions and modifier" do
-    allow(Paperclip::Geometry).to receive(:new).with(
-      height: "64",
-      width: "64",
-      modifier: "#",
-      orientation: nil
-    ).and_return(:correct)
-    factory = Paperclip::GeometryParser.new("64x64#")
-
-    output = factory.make
-
-    assert_equal :correct, output
-  end
-
-  it "identifies an image and extract its dimensions, orientation, and modifier" do
-    allow(Paperclip::Geometry).to receive(:new).with(
-      height: "50",
-      width: "100",
-      modifier: ">",
-      orientation: "7"
-    ).and_return(:correct)
-    factory = Paperclip::GeometryParser.new("100x50,7>")
-
-    output = factory.make
-
-    assert_equal :correct, output
-  end
-end

data/spec/paperclip/geometry_spec.rb

@@ -1,270 +0,0 @@
-require "spec_helper"
-
-describe Paperclip::Geometry do
-  context "Paperclip::Geometry" do
-    it "correctly reports its given dimensions" do
-      assert @geo = Paperclip::Geometry.new(1024, 768)
-      assert_equal 1024, @geo.width
-      assert_equal 768, @geo.height
-    end
-
-    it "sets height to 0 if height dimension is missing" do
-      assert @geo = Paperclip::Geometry.new(1024)
-      assert_equal 1024, @geo.width
-      assert_equal 0, @geo.height
-    end
-
-    it "sets width to 0 if width dimension is missing" do
-      assert @geo = Paperclip::Geometry.new(nil, 768)
-      assert_equal 0, @geo.width
-      assert_equal 768, @geo.height
-    end
-
-    it "is generated from a WxH-formatted string" do
-      assert @geo = Paperclip::Geometry.parse("800x600")
-      assert_equal 800, @geo.width
-      assert_equal 600, @geo.height
-    end
-
-    it "is generated from a xH-formatted string" do
-      assert @geo = Paperclip::Geometry.parse("x600")
-      assert_equal 0, @geo.width
-      assert_equal 600, @geo.height
-    end
-
-    it "is generated from a Wx-formatted string" do
-      assert @geo = Paperclip::Geometry.parse("800x")
-      assert_equal 800, @geo.width
-      assert_equal 0, @geo.height
-    end
-
-    it "is generated from a W-formatted string" do
-      assert @geo = Paperclip::Geometry.parse("800")
-      assert_equal 800, @geo.width
-      assert_equal 0, @geo.height
-    end
-
-    it "ensures the modifier is nil if not present" do
-      assert @geo = Paperclip::Geometry.parse("123x456")
-      assert_nil @geo.modifier
-    end
-
-    it "recognizes an EXIF orientation and not rotate with auto_orient if not necessary" do
-      geo = Paperclip::Geometry.new(width: 1024, height: 768, orientation: 1)
-      assert geo
-      assert_equal 1024, geo.width
-      assert_equal 768, geo.height
-
-      geo.auto_orient
-
-      assert_equal 1024, geo.width
-      assert_equal 768, geo.height
-    end
-
-    it "recognizes an EXIF orientation and rotate with auto_orient if necessary" do
-      geo = Paperclip::Geometry.new(width: 1024, height: 768, orientation: 6)
-      assert geo
-      assert_equal 1024, geo.width
-      assert_equal 768, geo.height
-
-      geo.auto_orient
-
-      assert_equal 768, geo.width
-      assert_equal 1024, geo.height
-    end
-
-    it "treats x and X the same in geometries" do
-      @lower = Paperclip::Geometry.parse("123x456")
-      @upper = Paperclip::Geometry.parse("123X456")
-      assert_equal 123, @lower.width
-      assert_equal 123, @upper.width
-      assert_equal 456, @lower.height
-      assert_equal 456, @upper.height
-    end
-
-    [">", "<", "#", "@", "@>", ">@", "%", "^", "!", nil].each do |mod|
-      it "ensures the modifier #{description} is preserved" do
-        assert @geo = Paperclip::Geometry.parse("123x456#{mod}")
-        assert_equal mod, @geo.modifier
-        assert_equal "123x456#{mod}", @geo.to_s
-      end
-
-      it "ensures the modifier #{description} is preserved with no height" do
-        assert @geo = Paperclip::Geometry.parse("123x#{mod}")
-        assert_equal mod, @geo.modifier
-        assert_equal "123#{mod}", @geo.to_s
-      end
-    end
-
-    it "makes sure the modifier gets passed during transformation_to" do
-      assert @src = Paperclip::Geometry.parse("123x456")
-      assert @dst = Paperclip::Geometry.parse("123x456>")
-      assert_equal ["123x456>", nil], @src.transformation_to(@dst)
-    end
-
-    it "generates correct ImageMagick formatting string for W-formatted string" do
-      assert @geo = Paperclip::Geometry.parse("800")
-      assert_equal "800", @geo.to_s
-    end
-
-    it "generates correct ImageMagick formatting string for Wx-formatted string" do
-      assert @geo = Paperclip::Geometry.parse("800x")
-      assert_equal "800", @geo.to_s
-    end
-
-    it "generates correct ImageMagick formatting string for xH-formatted string" do
-      assert @geo = Paperclip::Geometry.parse("x600")
-      assert_equal "x600", @geo.to_s
-    end
-
-    it "generates correct ImageMagick formatting string for WxH-formatted string" do
-      assert @geo = Paperclip::Geometry.parse("800x600")
-      assert_equal "800x600", @geo.to_s
-    end
-
-    it "is generated from a file" do
-      file = fixture_file("5k.png")
-      file = File.new(file, "rb")
-      assert_nothing_raised { @geo = Paperclip::Geometry.from_file(file) }
-      assert_equal 66, @geo.height
-      assert_equal 434, @geo.width
-    end
-
-    it "is generated from a file path" do
-      file = fixture_file("5k.png")
-      assert_nothing_raised { @geo = Paperclip::Geometry.from_file(file) }
-      assert_equal 66, @geo.height
-      assert_equal 434, @geo.width
-    end
-
-    it "calculates an EXIF-rotated image dimensions from a path" do
-      file = fixture_file("rotated.jpg")
-      assert_nothing_raised { @geo = Paperclip::Geometry.from_file(file) }
-      @geo.auto_orient
-      assert_equal 300, @geo.height
-      assert_equal 200, @geo.width
-    end
-
-    it "does not generate from a bad file" do
-      file = "/home/This File Does Not Exist.omg"
-      expect do
-        @geo = Paperclip::Geometry.from_file(file)
-      end.to raise_error(Paperclip::Errors::NotIdentifiedByBackendError,
-                         "Could not identify image size")
-    end
-
-    it "does not generate from a blank filename" do
-      file = ""
-      expect do
-        @geo = Paperclip::Geometry.from_file(file)
-      end.to raise_error(Paperclip::Errors::NotIdentifiedByBackendError,
-                         "Cannot find the geometry of a file with a blank name")
-    end
-
-    it "does not generate from a nil file" do
-      file = nil
-      expect do
-        @geo = Paperclip::Geometry.from_file(file)
-      end.to raise_error(Paperclip::Errors::NotIdentifiedByBackendError,
-                         "Cannot find the geometry of a file with a blank name")
-    end
-
-    it "does not generate from a file with no path" do
-      file = double("file", path: "")
-      allow(file).to receive(:respond_to?).with(:path).and_return(true)
-      expect do
-        @geo = Paperclip::Geometry.from_file(file)
-      end.to raise_error(Paperclip::Errors::NotIdentifiedByBackendError,
-                         "Cannot find the geometry of a file with a blank name")
-    end
-
-    it "lets us know when a command isn't found versus a processing error when using imagemagick" do
-      old_path = ENV["PATH"]
-      old_backend = Paperclip.options[:backend]
-      begin
-        Paperclip.options[:backend] = :image_magick
-        ENV["PATH"] = ""
-        assert_raises(Paperclip::Errors::CommandNotFoundError) do
-          file = fixture_file("5k.png")
-          @geo = Paperclip::Geometry.from_file(file)
-        end
-      ensure
-        ENV["PATH"] = old_path
-        Paperclip.options[:backend] = old_backend
-      end
-    end
-
-    [["vertical",   900,  1440, true,  false, false, 1440, 900, 0.625],
-     ["horizontal", 1024, 768,  false, true,  false, 1024, 768, 1.3333],
-     ["square",     100,  100,  false, false, true,  100,  100, 1]].each do |args|
-      context "performing calculations on a #{args[0]} viewport" do
-        before do
-          @geo = Paperclip::Geometry.new(args[1], args[2])
-        end
-
-        it "is #{args[3] ? '' : 'not'} vertical" do
-          assert_equal args[3], @geo.vertical?
-        end
-
-        it "is #{args[4] ? '' : 'not'} horizontal" do
-          assert_equal args[4], @geo.horizontal?
-        end
-
-        it "is #{args[5] ? '' : 'not'} square" do
-          assert_equal args[5], @geo.square?
-        end
-
-        it "reports that #{args[6]} is the larger dimension" do
-          assert_equal args[6], @geo.larger
-        end
-
-        it "reports that #{args[7]} is the smaller dimension" do
-          assert_equal args[7], @geo.smaller
-        end
-
-        it "has an aspect ratio of #{args[8]}" do
-          expect(@geo.aspect).to be_within(0.0001).of(args[8])
-        end
-      end
-    end
-
-    [[[1000, 100], [64, 64],  "x64", "64x64+288+0"],
-     [[100, 1000], [50, 950], "x950", "50x950+22+0"],
-     [[100, 1000], [50, 25],  "50x", "50x25+0+237"]]. each do |args|
-      context "of #{args[0].inspect} and given a Geometry #{args[1].inspect} and sent transform_to" do
-        before do
-          @geo = Paperclip::Geometry.new(*args[0])
-          @dst = Paperclip::Geometry.new(*args[1])
-          @scale, @crop = @geo.transformation_to @dst, true
-        end
-
-        it "is able to return the correct scaling transformation geometry #{args[2]}" do
-          assert_equal args[2], @scale
-        end
-
-        it "is able to return the correct crop transformation geometry #{args[3]}" do
-          assert_equal args[3], @crop
-        end
-      end
-    end
-
-    [["256x256", { "150x150!" => [150, 150], "150x150#" => [150, 150], "150x150>" => [150, 150], "150x150<" => [256, 256], "150x150" => [150, 150] }],
-     ["256x256", { "512x512!" => [512, 512], "512x512#" => [512, 512], "512x512>" => [256, 256], "512x512<" => [512, 512], "512x512" => [512, 512] }],
-     ["600x400", { "512x512!" => [512, 512], "512x512#" => [512, 512], "512x512>" => [512, 341], "512x512<" => [600, 400], "512x512" => [512, 341] }]].each do |original_size, options|
-      options.each_pair do |size, dimensions|
-        context "#{original_size} resize_to #{size}" do
-          before do
-            @source = Paperclip::Geometry.parse original_size
-            @new_geometry = @source.resize_to size
-          end
-          it "has #{dimensions.first} width" do
-            assert_equal dimensions.first, @new_geometry.width
-          end
-          it "has #{dimensions.last} height" do
-            assert_equal dimensions.last, @new_geometry.height
-          end
-        end
-      end
-    end
-  end
-end

data/spec/paperclip/glue_spec.rb

@@ -1,63 +0,0 @@
-# require "spec_helper"
-
-describe Paperclip::Glue do
-  describe "when ActiveRecord does not exist" do
-    before do
-      ActiveRecordSaved = ActiveRecord
-      Object.send :remove_const, "ActiveRecord"
-    end
-
-    after do
-      ActiveRecord = ActiveRecordSaved
-      Object.send :remove_const, "ActiveRecordSaved"
-    end
-
-    it "does not fail" do
-      NonActiveRecordModel = Class.new
-      NonActiveRecordModel.include Paperclip::Glue
-      Object.send :remove_const, "NonActiveRecordModel"
-    end
-  end
-
-  describe "when ActiveRecord does exist" do
-    before do
-      if Object.const_defined?("ActiveRecord")
-        @defined_active_record = false
-      else
-        ActiveRecord = :defined
-        @defined_active_record = true
-      end
-    end
-
-    after do
-      Object.send :remove_const, "ActiveRecord" if @defined_active_record
-    end
-
-    it "does not fail" do
-      NonActiveRecordModel = Class.new
-      NonActiveRecordModel.include Paperclip::Glue
-      Object.send :remove_const, "NonActiveRecordModel"
-    end
-  end
-
-  describe "when included" do
-    it "does not mutate I18n.load_path more than once" do
-      before_load_path = I18n.load_path
-      I18n.load_path = []
-
-      # expect twice because the load_path is reset after creating the classes
-      expect(I18n.config).to receive(:load_path=).and_call_original.twice
-
-      FirstModel = Class.new
-      FirstModel.include Paperclip::Glue
-
-      SecondModel = Class.new
-      SecondModel.include Paperclip::Glue
-
-      ThirdModel = Class.new
-      ThirdModel.include Paperclip::Glue
-
-      I18n.load_path = before_load_path
-    end
-  end
-end