mini_magick 2.0.2 → 2.1

data/VERSION

@@ -1 +1 @@
-2.0.2
+2.1

data/lib/mini_magick.rb

@@ -141,21 +141,15 @@ module MiniMagick
     # If an unknown method is called then it is sent through the morgrify program
     # Look here to find all the commands (http://www.imagemagick.org/script/mogrify.php)
     def method_missing(symbol, *args)
-      guessed_command_name = symbol.to_s.gsub('_','-')
-
-      if MOGRIFY_COMMANDS.include?(guessed_command_name)
-        args.push(@path) # push the path onto the end
-        run_command("mogrify", "-#{guessed_command_name}", *args)
-        self
-      else
-        super(symbol, *args)
+      combine_options do |c|
+        c.method_missing(symbol, *args)
       end
     end
 
     # You can use multiple commands together using this method
     def combine_options(&block)
       c = CommandBuilder.new('mogrify')
-      block.call c
+      block.call(c)
       c << @path
       run(c)
     end
@@ -185,7 +179,7 @@ module MiniMagick
 
     # Outputs a carriage-return delimited format string for Unix and Windows
     def format_option(format)
-      windows? ? "#{format}\\n" : "#{format}\\\\n"
+      windows? ? "\"#{format}\\n\"" : "\"#{format}\\\\n\""
     end
 
     def run_command(command, *args)
@@ -207,7 +201,7 @@ module MiniMagick
         else
           # TODO: should we do something different if the command times out ...?
           # its definitely better for logging.. otherwise we dont really know
-          raise Error, "Command (#{command.inspect}) failed: #{{:status_code => sub.exitstatus, :output => sub.output}.inspect}"
+          raise Error, "Command (#{command.inspect.gsub("\\", "")}) failed: #{{:status_code => sub.exitstatus, :output => sub.output}.inspect}"
         end
       else
         sub.output
@@ -239,27 +233,33 @@ module MiniMagick
     def initialize(command, *options)
       @command = command
       @args = []
-  
-      options.each { |val| push(val) }
+      options.each { |arg| push(arg) }
     end
     
     def command
       "#{MiniMagick.processor} #{@command} #{@args.join(' ')}".strip
     end
     
-    def method_missing(symbol, *args)
+    def method_missing(symbol, *options)
       guessed_command_name = symbol.to_s.gsub('_','-')
-      if MOGRIFY_COMMANDS.include?(guessed_command_name)
-        push("-#{guessed_command_name}")
-        push(args.join(" ")) if args.any?
+      if guessed_command_name == "format"
+        raise Error, "You must call 'format' on the image object directly!"
+      elsif MOGRIFY_COMMANDS.include?(guessed_command_name)
+        add(guessed_command_name, *options)
       else
         super(symbol, *args)
       end
     end
     
-    def push(value)
-      # args can contain characters like '>' so we must escape them, but don't quote switches
-      @args << ((value !~ /^[\+\-]/) ? "\"#{value}\"" : value.to_s.strip)
+    def add(command, *options)
+      push "-#{command}"
+      if options.any?
+        push "\"#{options.join(" ")}\""
+      end
+    end
+    
+    def push(arg)
+      @args << arg.strip
     end
     alias :<< :push
 

data/test/image_test.rb

@@ -77,19 +77,6 @@ class ImageTest < Test::Unit::TestCase
     image.destroy!
   end
 
-  # def test_animation_pages
-  #   image = Image.from_file(ANIMATION_PATH)
-  #   image.format "png", 0
-  #   assert_equal "png", image[:format].downcase
-  #   image.destroy!
-  # end
-
-  # def test_animation_size
-  #   image = Image.from_file(ANIMATION_PATH)
-  #   assert_equal image[:size], 76631
-  #   image.destroy!
-  # end
-
   def test_gif_with_jpg_format
     image = Image.new(GIF_WITH_JPG_EXT)
     assert_equal "gif", image[:format].downcase
@@ -120,7 +107,7 @@ class ImageTest < Test::Unit::TestCase
     image = Image.from_file(SIMPLE_IMAGE_PATH)
     image.combine_options do |c|
       c.resize "20x30!"
-      c.blur 50
+      c.blur "50"
     end
 
     assert_equal 20, image[:width]
@@ -198,11 +185,25 @@ class ImageTest < Test::Unit::TestCase
     end
     assert `diff -s #{result.path} test/composited.jpg`.include?("identical")
   end
-
-  # def test_mini_magick_error_when_referencing_not_existing_page
-  #   image = Image.from_file(ANIMATION_PATH)
-  #   image.format('png')
-  #   assert_equal image[:format], 'PNG'
-  #   image.destroy!
-  # end
+  
+  def test_issue_8
+    image = Image.from_file(SIMPLE_IMAGE_PATH)
+    assert_nothing_raised do
+      image.combine_options do |c|
+        c.sample "50%"
+        c.rotate "-90>"
+      end
+    end
+    image.destroy!
+  end
+  
+  def test_throw_format_error
+    image = Image.from_file(SIMPLE_IMAGE_PATH)
+    assert_raise MiniMagick::Error do
+      image.combine_options do |c|
+        c.format "png"
+      end
+    end
+    image.destroy!
+  end
 end

metadata

@@ -1,13 +1,12 @@
 --- !ruby/object:Gem::Specification 
 name: mini_magick
 version: !ruby/object:Gem::Version 
-  hash: 11
+  hash: 1
   prerelease: false
   segments: 
   - 2
-  - 0
-  - 2
-  version: 2.0.2
+  - 1
+  version: "2.1"
 platform: ruby
 authors: 
 - Corey Johnson