Upgrade minimagick 5 (#132)

* Add gemfiles for multi-gem version testing

* Permit mini_magic v5 in gemspec

* Support valid image check for minimagic5

* Use a shim to simplify support for minimagick 4 and 5

Author: lukeasrodgers

.gitignore

@@ -1,3 +1,4 @@
 Gemfile.lock
+gemfiles/*.lock
 pkg/
 .DS_Store

gemfiles/mini_magick4.gemfile

@@ -0,0 +1,7 @@
+source 'https://rubygems.org'
+
+gemspec :path => '../'
+
+gem 'pry'
+
+gem 'mini_magick', '~> 4.9'

gemfiles/mini_magick5.gemfile

@@ -0,0 +1,7 @@
+source 'https://rubygems.org'
+
+gemspec :path => '../'
+
+gem 'pry'
+
+gem 'mini_magick', '~> 5.0'

image_processing.gemspec

@@ -19,7 +19,7 @@ Gem::Specification.new do |spec|
   spec.metadata = { "changelog_uri" => spec.homepage + "/blob/master/CHANGELOG.md",
                     "rubygems_mfa_required" => "true" }
 
-  spec.add_dependency "mini_magick", ">= 4.9.5", "< 5"
+  spec.add_dependency "mini_magick", ">= 4.9.5", "< 6"
   spec.add_dependency "ruby-vips", ">= 2.0.17", "< 3"
 
   spec.add_development_dependency "rake"

lib/image_processing/mini_magick.rb

@@ -5,9 +5,17 @@ module ImageProcessing
   module MiniMagick
     extend Chainable
 
+    def self.convert_shim(&block)
+      if ::MiniMagick.respond_to?(:convert)
+        ::MiniMagick.convert(&block)
+      else
+        ::MiniMagick::Tool::Convert.new(&block)
+      end
+    end
+
     # Returns whether the given image file is processable.
     def self.valid_image?(file)
-      ::MiniMagick::Tool::Convert.new do |convert|
+      convert_shim do |convert|
         convert << file.path
         convert << "null:"
       end
@@ -30,7 +38,7 @@ def self.load_image(path_or_magick, loader: nil, page: nil, geometry: nil, auto_
           magick = path_or_magick
         else
           source_path = path_or_magick
-          magick = ::MiniMagick::Tool::Convert.new
+          magick = ::ImageProcessing::MiniMagick.convert_shim
 
           Utils.apply_options(magick, **options)
 

test/mini_magick_test.rb

@@ -14,7 +14,7 @@
   it "applies imagemagick operations" do
     actual = ImageProcessing::MiniMagick.flip.call(@portrait)
     expected = Tempfile.new(["result", ".jpg"], binmode: true).tap do |tempfile|
-      MiniMagick::Tool::Convert.new do |cmd|
+      ImageProcessing::MiniMagick.convert_shim do |cmd|
         cmd << @portrait.path
         cmd.flip
         cmd << tempfile.path
@@ -37,7 +37,7 @@
   it "applies macro operations" do
     actual = ImageProcessing::MiniMagick.resize_to_limit(400, 400).call(@portrait)
     expected = Tempfile.new(["result", ".jpg"], binmode: true).tap do |tempfile|
-      MiniMagick::Tool::Convert.new do |cmd|
+      ImageProcessing::MiniMagick.convert_shim do |cmd|
         cmd << @portrait.path
         cmd.resize("400x400")
         cmd << tempfile.path
@@ -55,7 +55,7 @@
 
   it "accepts page" do
     tiff = Tempfile.new(["file", ".tiff"])
-    MiniMagick::Tool::Convert.new do |convert|
+    ImageProcessing::MiniMagick.convert_shim do |convert|
       convert.merge! [@portrait.path, @portrait.path, @portrait.path]
       convert << tiff.path
     end
@@ -70,7 +70,7 @@
 
   it "disallows split layers by default" do
     tiff = Tempfile.new(["file", ".tiff"])
-    MiniMagick::Tool::Convert.new do |convert|
+    ImageProcessing::MiniMagick.convert_shim do |convert|
       convert.merge! [@portrait.path, @portrait.path, @portrait.path]
       convert << tiff.path
     end
@@ -168,7 +168,7 @@
   end
 
   it "accepts magick object as source" do
-    magick = MiniMagick::Tool::Convert.new
+    magick = ImageProcessing::MiniMagick.convert_shim
     magick << fixture_image("rotated.jpg").path
     result = ImageProcessing::MiniMagick.source(magick).call
     assert_dimensions [600, 800], result
@@ -568,7 +568,7 @@
     it "appends CLI arguments" do
       actual = ImageProcessing::MiniMagick.append("-resize", "400x400").call(@portrait)
       expected = Tempfile.new(["result", ".jpg"], binmode: true).tap do |tempfile|
-        MiniMagick::Tool::Convert.new do |cmd|
+        ImageProcessing::MiniMagick.convert_shim do |cmd|
           cmd << @portrait.path
           cmd.resize("400x400")
           cmd << tempfile.path