Merge branch 'i0rek-loop' into development

* i0rek-loop:
  Make spec text more descriptive.
  wip

lib/smartcropper.rb

@@ -110,6 +110,7 @@ def smart_crop_by_trim(requested_x, requested_y)
             bottom -= slice_height
           end
 
+          break if slice_height == step_size
           height = (bottom - top)
         end
       end

test/helper.rb

@@ -13,5 +13,12 @@
 
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 $LOAD_PATH.unshift(File.dirname(__FILE__))
+
+class Test::Unit::TestCase
+  def fixture_path
+    Pathname.new(File.join(File.expand_path(File.dirname(__FILE__)), "fixtures"))
+  end
+end
+
 require 'smartcropper'
 

test/test_croptoelie.rb

@@ -14,6 +14,7 @@ def setup
     img = SmartCropper.new(@image)
     assert_equal(img.class, SmartCropper)
   end
+
   should "create a smartcropper from an imagefile" do
     img = SmartCropper.from_file(@filename)
     assert_equal(img.class, SmartCropper)
@@ -64,6 +65,24 @@ def setup
     assert_equal(img.image, @image)
   end
 
+  ###########################################################################
+  #            Stuck in a loop when image is too small to slice             #
+  ###########################################################################
+  should "handle 1px wide images" do
+    @filename = fixture_path.join("small1.png")
+    @image    = Magick::ImageList.new(@filename).last
+    img = SmartCropper.new(@image).smart_crop_and_scale(1, 50)
+    size = [img.rows, img.columns]
+    assert_equal([50, 1], size)
+  end
+
+  should "handle 5px wide images" do
+    @filename = fixture_path.join("small2.png")
+    @image    = Magick::ImageList.new(@filename).last
+    img = SmartCropper.new(@image).smart_crop_and_scale(1, 50)
+    size = [img.rows, img.columns]
+    assert_equal([50, 1], size)
+  end
 
   ###########################################################################
   #                   Images reported to fail by issue #5                   #