mojo_magick 0.5.4 → 0.6.2

data/test/opt_builder_test.rb

@@ -1,7 +1,6 @@
-require File::join(File::dirname(__FILE__), 'test_helper')
-
-class MojoMagickOptBuilderTest < MiniTest::Unit::TestCase
+require_relative "test_helper"
 
+class MojoMagickOptBuilderTest < MiniTest::Test
   # These tests make the assumption that if we call #raw_command with the
   # correct strings, ImageMagick itself will operate correctly. We're only
   # verifying that the option builder produces the correct strings
@@ -11,36 +10,46 @@ class MojoMagickOptBuilderTest < MiniTest::Unit::TestCase
   end
 
   def test_annotate
-    @builder.annotate 'blah'
-    assert_equal  '-annotate 0 blah', @builder.to_s
+    @builder.annotate "blah"
+    assert_equal %w[-annotate 0 blah], @builder.to_a
   end
 
   def test_annotate_with_escapeable_string
-    @builder.annotate 'it\'s'
-    assert_equal '-annotate 0 "it\'s"', @builder.to_s
+    @builder.annotate "it's"
+    assert_equal %w[-annotate 0 it's], @builder.to_a
+  end
+
+  def test_annotate_with_full_args
+    @builder.annotate "5 it's"
+    assert_equal %w[-annotate 5 it's], @builder.to_a
   end
 
   def test_annotate_with_full_args
-    @builder.annotate '5 it\'s'
-    assert_equal '-annotate 5 "it\'s"', @builder.to_s
+    @builder.annotate "this thing", geometry: 3
+    assert_equal ["-annotate", "3", "this thing"], @builder.to_a
+  end
+
+  def test_annotate_with_full_array_args
+    @builder.annotate "this", "thing", geometry: 3
+    assert_equal ["-annotate", "3", "thisthing"], @builder.to_a
   end
 
   def test_option_builder_with_blocks
     # Passing in basic commands produces a string
     b = MojoMagick::OptBuilder.new
     b.image_block do
-      b.background 'red'
+      b.background "red"
     end
     b.image_block do
-      b.background 'blue'
+      b.background "blue"
     end
-    assert_equal '\( -background red \) \( -background blue \)', b.to_s
+    assert_equal ['\(', "-background", "red", '\)', '\(', "-background", "blue", '\)'], b.to_a
   end
 
-  def test_option_builder_with_hex_colors 
+  def test_option_builder_with_hex_colors
     b = MojoMagick::OptBuilder.new
-    b.background '#000000'
-    assert_equal '-background "#000000"', b.to_s
+    b.background "#000000"
+    assert_equal %w[-background #000000], b.to_a
   end
 
   def test_option_builder
@@ -48,85 +57,83 @@ class MojoMagickOptBuilderTest < MiniTest::Unit::TestCase
     b = MojoMagick::OptBuilder.new
     b.strip
     b.repage
-    assert_equal '-strip -repage', b.to_s
+    assert_equal "-strip -repage", b.to_s
 
     # Chaining commands works
     b = MojoMagick::OptBuilder.new.strip.repage
-    assert_equal '-strip -repage', b.to_s
+    assert_equal "-strip -repage", b.to_s
 
     # Bang (!) indicates the plus version of commands
     b = MojoMagick::OptBuilder.new
     b.repage
     b.repage!
-    assert_equal '-repage +repage', b.to_s
+    assert_equal "-repage +repage", b.to_s
 
     # Accepts raw data as-is
     b = MojoMagick::OptBuilder.new
     b.opt1
-    b << 'a ! b !'
+    b << "a ! b !"
     b.opt2
-    assert_equal '-opt1 a ! b ! -opt2', b.to_s
+    assert_equal "-opt1 a ! b ! -opt2", b.to_s
 
     # Treats an array of raw data as different arguments
     b = MojoMagick::OptBuilder.new
-    b << ['leave this data','alone']
-    assert_equal 'leave this data alone', b.to_s
+    b << ["leave this data", "alone"]
+    assert_equal "leave this data alone", b.to_s
 
     # String includes command arguments
     b = MojoMagick::OptBuilder.new
-    b.extent '256x256+0+0'
-    b.crop '64x64'
-    assert_equal '-extent 256x256+0+0 -crop 64x64', b.to_s
+    b.extent "256x256+0+0"
+    b.crop "64x64"
+    assert_equal %w[-extent 256x256+0+0 -crop 64x64], b.to_a
 
     # Arguments are quoted (doublequote) if appropriate
     b = MojoMagick::OptBuilder.new
-    b.comment 'white space'
-    b.comment 'w&b'
-    b.crop '6x6^'
-    assert_equal '-comment "white space" -comment "w&b" -crop "6x6^"', b.to_s
+    b.comment "white space"
+    b.comment "w&b"
+    b.crop "6x6^"
+    assert_equal ["-comment", "white space", "-comment", "w&b", "-crop", "6x6^"], b.to_a
 
     # Existing doublequotes are escaped
     b = MojoMagick::OptBuilder.new
     b.comment 'Fred "Woot" Rook'
-    assert_equal '-comment "Fred \"Woot\" Rook"', b.to_s
+    assert_equal ["-comment", "Fred \"Woot\" Rook"], b.to_a
 
     # Multi-argument commands should not be quoted together
     b = MojoMagick::OptBuilder.new
-    b.set 'comment', 'the "best" comment'
-    assert_equal '-set comment "the \"best\" comment"', b.to_s
+    b.set "comment", 'the "best" comment'
+    assert_equal ["-set", "comment", "the \"best\" comment"], b.to_a
 
     # File and files are helper methods
     b = MojoMagick::OptBuilder.new
-    b.files 'source.jpg', 'source2.jpg'
+    b.files "source.jpg", "source2.jpg"
     b.append
-    b.crop '64x64'
-    b.file 'dest%d.jpg'
-    assert_equal 'source.jpg source2.jpg -append -crop 64x64 dest%d.jpg', b.to_s
+    b.crop "64x64"
+    b.file "dest%d.jpg"
+    assert_equal %w[source.jpg source2.jpg -append -crop 64x64 dest%d.jpg], b.to_a
 
     # Files are quoted (doublequote) if appropriate
     b = MojoMagick::OptBuilder.new
-    b.file 'probably on windows.jpg'
-    assert_equal '"probably on windows.jpg"', b.to_s
+    b.file "probably on windows.jpg"
+    assert_equal ["probably on windows.jpg"], b.to_a
 
     # Blob is a shortcut for the #tempfile helper method
     b = MojoMagick::OptBuilder.new
-    b.blob 'binary data'
+    b.blob "binary data"
     filename = b.to_s
-    File.open(filename, 'rb') do |f|
-      assert_equal 'binary data', f.read
+    File.open(filename, "rb") do |f|
+      assert_equal "binary data", f.read
     end
 
-    #label for text should use 'label:"the string"' if specified
-    [[ 'mylabel', 'mylabel' ],
-     [ 'my " label', '"my \" label"' ],
-     [ 'Rock it, cuz i said so!', '"Rock it, cuz i said so!"'],
-     [ "it's like this", '"it\'s like this"'],
-     [ '#$%^&*', '"#$%^&*"']].each do |labels|
-
+    # label for text should use 'label:"the string"' if specified
+    [%w[mylabel mylabel],
+     ['my " label', '"my \" label"'],
+     ["Rock it, cuz i said so!", '"Rock it, cuz i said so!"'],
+     ["it's like this", '"it\'s like this"'],
+     ["\#$%^&*", '"#$%^&*"']].each do |labels|
       b = MojoMagick::OptBuilder.new
       b.label labels[0]
       assert_equal "label:#{labels[1]}", b.to_s
     end
-
   end
 end

data/test/parser_test.rb

@@ -1,30 +1,29 @@
-require File::join(File::dirname(__FILE__), 'test_helper')
+require_relative "test_helper"
 
-IDENTIFY_FONT_RESPONSE =<<EOF
+IDENTIFY_FONT_RESPONSE = <<~EOFONT
 
-Font: Zapf-Dingbats
-    family: Zapf Dingbats
-    style: Normal
-    stretch: Normal
-    weight: 400
-    glyphs: /System/Library/Fonts/ZapfDingbats.ttf
+  Font: Zapf-Dingbats
+      family: Zapf Dingbats
+      style: Normal
+      stretch: Normal
+      weight: 400
+      glyphs: /System/Library/Fonts/ZapfDingbats.ttf
 
-  Font: Zapfino
-    family: Zapfino
-    style: Italic
-    stretch: Normal
-    weight: 400
-    glyphs: /Library/Fonts/Zapfino.ttf
+    Font: Zapfino
+      family: Zapfino
+      style: Italic
+      stretch: Normal
+      weight: 400
+      glyphs: /Library/Fonts/Zapfino.ttf
 
 
-EOF
-
-class ParserTest < MiniTest::Unit::TestCase
+EOFONT
 
+class ParserTest < MiniTest::Test
   def test_parse_fonts
     parser = MojoMagick::Util::Parser.new
     parsed_fonts = parser.parse_fonts(IDENTIFY_FONT_RESPONSE)
     assert_equal parsed_fonts.length, 2
-    assert_equal parsed_fonts[1].style, 'Italic'
+    assert_equal parsed_fonts[1].style, "Italic"
   end
 end

data/test/test_helper.rb

@@ -1,6 +1,8 @@
-require 'test/unit'
-require File::expand_path(File::join(File::dirname(__FILE__), '..', 'init'))
-require 'fileutils'
-require 'tempfile'
-require 'rspec/expectations'
+require "simplecov"
+SimpleCov.start
 
+require "minitest/autorun"
+require File.expand_path(File.join(File.dirname(__FILE__), "..", "init"))
+require "fileutils"
+require "tempfile"
+require "rspec/expectations"

metadata

@@ -1,57 +1,113 @@
 --- !ruby/object:Gem::Specification
 name: mojo_magick
 version: !ruby/object:Gem::Version
-  version: 0.5.4
+  version: 0.6.2
 platform: ruby
 authors:
 - Steve Midgley
 - Elliot Nelson
 - Jon Rogers
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2013-12-15 00:00:00.000000000 Z
+date: 2021-01-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: rake
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: bundle-audit
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: simplecov
+  name: minitest
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec-expectations
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-performance
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: Simple Ruby stateless module interface to imagemagick.
@@ -62,8 +118,14 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .ruby-version
+- ".circleci/config.yml"
+- ".github/dependabot.yml"
+- ".gitignore"
+- ".rubocop-common.yml"
+- ".rubocop-performance.yml"
+- ".rubocop-rspec.yml"
+- ".rubocop.yml"
+- ".ruby-version"
 - Gemfile
 - Gemfile.lock
 - LICENSE.txt
@@ -73,9 +135,10 @@ files:
 - examples/composite.rb
 - init.rb
 - lib/image_magick/fonts.rb
-- lib/image_magick/resource_limits.rb
 - lib/initializers/hash.rb
 - lib/mojo_magick.rb
+- lib/mojo_magick/command_status.rb
+- lib/mojo_magick/errors.rb
 - lib/mojo_magick/font.rb
 - lib/mojo_magick/opt_builder.rb
 - lib/mojo_magick/util/parser.rb
@@ -83,13 +146,13 @@ files:
 - mojo_magick.gemspec
 - test/fixtures/5742.jpg
 - test/fixtures/not_an_image.jpg
+- test/fixtures/roll with it.jpg
 - test/fixtures/zero_byte_image.jpg
 - test/font_test.rb
 - test/fonts_test.rb
 - test/mojo_magick_test.rb
 - test/opt_builder_test.rb
 - test/parser_test.rb
-- test/resource_limits_test.rb
 - test/test_helper.rb
 homepage: http://github.com/rcode5/mojo_magick
 licenses:
@@ -109,28 +172,28 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.6.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: mojo_magick
-rubygems_version: 2.0.14
-signing_key: 
+rubygems_version: 3.0.3
+signing_key:
 specification_version: 4
-summary: mojo_magick-0.5.4
+summary: mojo_magick-0.6.2
 test_files:
 - test/fixtures/5742.jpg
 - test/fixtures/not_an_image.jpg
+- test/fixtures/roll with it.jpg
 - test/fixtures/zero_byte_image.jpg
 - test/font_test.rb
 - test/fonts_test.rb
 - test/mojo_magick_test.rb
 - test/opt_builder_test.rb
 - test/parser_test.rb
-- test/resource_limits_test.rb
 - test/test_helper.rb
+...