capybara-screenshot-diff 0.5.2 → 0.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 31b02395af8b464ceeb774dbf6364497e0127daa
-  data.tar.gz: 2bd199042023db6a41d51e0312fb791d2da0ece6
+  metadata.gz: 4918e7dba3e123055b2c5aae345fce7dd8a1219d
+  data.tar.gz: 662f6f68099b888e068a62ec18454ddbcdf2adf6
 SHA512:
-  metadata.gz: 116bc4c1258f770998f2451ab3abcb124bc4b6926961e4cae7edf3cb32e128d990fed362b18b1226d9c84fdfb92fb88e21bffb5691fe6faf5077884bd1d6e7e9
-  data.tar.gz: a40a2b6e2f471d9762fc0e2cfe8da22ec3a89bf5ccbb0971ef5afb43944386193a3636849281194094c2810e24905a755ede1e3ac700580057d32cc1da02912d
+  metadata.gz: 6929b38a90048524ec74c867f93a2b14160c34cae83d325963bfefd0bbd28a134e9f3471360b3649226b33e43414d60da35ccab499d4101fddac08688eed91ab
+  data.tar.gz: 3638b34d359a68c75eaaa49593396efc227383126259b95ad5af12b6e75103164bd9b7f22485cf97831f83415defcb8f5072923de6cdbfedb8b450836ff692e9

data/lib/capybara/screenshot/diff/capybara_setup.rb

@@ -97,9 +97,10 @@ module ActionDispatch
 
     teardown do
       if Capybara::Screenshot::Diff.enabled && @test_screenshots
-        @test_screenshots.each { |args| assert_image_not_changed(*args) }
+        test_screenshot_errors =
+          @test_screenshots.map { |args| assert_image_not_changed(*args) }.compact
+        fail(test_screenshot_errors.join("\n\n")) if test_screenshot_errors.any?
       end
-      fail(@test_screenshot_errors.join("\n\n")) if @test_screenshot_errors
     end
 
     def screenshot_section(name)
@@ -227,8 +228,7 @@ EOF
     def assert_image_not_changed(caller, name, file_name, committed_file_name, new_name, org_name)
       if Capybara::Screenshot::Diff::ImageCompare.compare(committed_file_name, file_name,
           Capybara::Screenshot.window_size)
-        (@test_screenshot_errors ||= []) <<
-          "Screenshot does not match for '#{name}'\n#{file_name}\n#{org_name}\n#{new_name}\nat #{caller}"
+        "Screenshot does not match for '#{name}'\n#{file_name}\n#{org_name}\n#{new_name}\nat #{caller}"
       end
     end
   end

data/lib/capybara/screenshot/diff/image_compare.rb

@@ -99,8 +99,8 @@ module Capybara
 
         def find_diff_rectangle(org_img, new_img)
           top = bottom = nil
-          left = org_img.width
-          right = -1
+          left = org_img.width - 1
+          right = 0
           org_img.height.times do |y|
             (0...left).find do |x|
               next if org_img[x, y] == new_img[x, y]
@@ -118,7 +118,7 @@ module Capybara
             end
           end
           (org_img.height - 1).step(bottom + 1, -1).find do |y|
-            ((left + 1)..(right - 1)).find do |x|
+            (left..right).find do |x|
               bottom = y if org_img[x, y] != new_img[x, y]
             end
           end

data/lib/capybara/screenshot/diff/version.rb

@@ -3,7 +3,7 @@
 module Capybara
   module Screenshot
     module Diff
-      VERSION = '0.5.2'.freeze
+      VERSION = '0.5.3'.freeze
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: capybara-screenshot-diff
 version: !ruby/object:Gem::Version
-  version: 0.5.2
+  version: 0.5.3
 platform: ruby
 authors:
 - Uwe Kubosch