capybara-screenshot-diff 1.4.2 → 1.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 164b1f7468f9195f25acad64b3cceafde5efbdf12f773d23c9d6aad004fa7095
-  data.tar.gz: 858f675a45e8fa50b806fbeb0536a8c9317fe11ae882dcb7d0eaf37b1972360e
+  metadata.gz: 7337bf34cbab401846cb66e06c1d497fb5fe7ee79d30cd7588abb740d7a66fcd
+  data.tar.gz: abe35f6703d18bfa84bac0235d9897d0ce0b3f737641603e5c69a44eded4bb46
 SHA512:
-  metadata.gz: ca8c20611a29d763df3c3ed44e9978173d5c33b5acc623af0b8b82583868100c2f11a1d4ae82b590720f98949e61c45d564dca8e501de072b4e1c5cf9cdd7c3f
-  data.tar.gz: 5f0489076c391d6f61b86ce7b942e320ae13e1b7531996adfd755446605650ec235f30ecac7ae7de092e2e512778456004ac139fcfbc6a668e8be974ba07a25e
+  metadata.gz: 8bf1be3c966ddec256071897645ff15116de082842eb85d9622322e221d64f7ca86ae199a53b7f2e6ffc7e25d11e228f31646ca4ddcb876227f7c34a782752ca
+  data.tar.gz: c85c3b6d01f224cc7bee5822c0ec4a4262fe966ec071f6cbc1c025d91a727557ee2efb7485c4b400607308dfbcabe2cf21bea3778cde2e105ac4ec4f2f8b7526

data/.github/workflows/test.yml

@@ -3,6 +3,8 @@ name: Test
 on:
   push:
     branches: [ $default-branch ]
+  pull_request_target:
+    branches: [ $default-branch ]
   pull_request:
 
 env:
@@ -69,7 +71,7 @@ jobs:
 
     strategy:
       matrix:
-        ruby-version: [ '2.7', '2.6', '2.5', 'jruby' ]
+        ruby-version: [ '3.0', '2.7', '2.6', 'jruby' ]
         gemfile:
           - 'rails61_gems.rb'
           - 'rails60_gems.rb'
@@ -77,6 +79,12 @@ jobs:
           - 'rails51.gemfile'
           - 'rails50.gemfile'
           - 'rails42.gemfile'
+#        exclude:
+#          - ruby-version: '3.0'
+#            gemfile: 'rails50_gems.rb'
+#          - ruby-version: '3.0'
+#            gemfile: 'rails42_gems.rb'
+
     env:
       BUNDLE_GEMFILE: gemfiles/${{ matrix.gemfile }}
 

data/README.md

@@ -67,6 +67,15 @@ describe 'Permissions admin', :type => :feature, :js => true do
 
 end
 ```
+But it's better to include it within your *_helper.rb file so that it can used anywhere in your feature specs.
+```ruby
+# spec/feature_helper.rb
+require 'capybara/screenshot/diff'
+
+RSpec.configure do |config|
+    config.include Capybara::Screenshot::Diff::TestMethods
+end
+```
 
 ### Taking screenshots
 

data/capybara-screenshot-diff.gemspec

@@ -12,7 +12,7 @@ Gem::Specification.new do |spec|
   spec.summary = "Track your GUI changes with diff assertions"
   spec.description = "Save screen shots and track changes with graphical diff"
   spec.homepage = "https://github.com/donv/capybara-screenshot-diff"
-  spec.required_ruby_version = "~> 2.5"
+  spec.required_ruby_version = ">= 2.5.0"
   spec.license = "MIT"
   spec.metadata["allowed_push_host"] = "https://rubygems.org/"
   spec.files = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }

data/lib/capybara/screenshot/diff/drivers/chunky_png_driver.rb

@@ -209,8 +209,8 @@ module Capybara
           def find_left_right_and_top(old_img, new_img, region)
             left = region[0] || old_img.width - 1
             top = region[1]
-            bottom = region[2]
-            right = region[3] || 0
+            right = region[2] || 0
+            bottom = region[3]
             old_img.height.times do |y|
               (0...left).find do |x|
                 next if same_color?(old_img, new_img, x, y)

data/lib/capybara/screenshot/diff/drivers/vips_driver.rb

@@ -82,7 +82,7 @@ module Capybara
             @chunky_png_comparator ||= ImageCompare.new(
               @new_file_name,
               @old_file_name,
-              @options.merge(driver: :chunky_png, tolerance: nil, median_filter_window_size: nil)
+              **@options.merge(driver: :chunky_png, tolerance: nil, median_filter_window_size: nil)
             )
           end
 

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

@@ -21,13 +21,13 @@ module Capybara
         JS
 
         HIDE_CARET_SCRIPT = <<~JS
-              if (!document.getElementById('csdHideCaretStyle')) {
-                let style = document.createElement('style');
-                style.setAttribute('id', 'csdHideCaretStyle');
-                document.head.appendChild(style);
-                let styleSheet = style.sheet;
-                styleSheet.insertRule("* { caret-color: transparent !important; }", 0);
-              }
+          if (!document.getElementById('csdHideCaretStyle')) {
+            let style = document.createElement('style');
+            style.setAttribute('id', 'csdHideCaretStyle');
+            document.head.appendChild(style);
+            let styleSheet = style.sheet;
+            styleSheet.insertRule("* { caret-color: transparent !important; }", 0);
+          }
         JS
 
         def take_stable_screenshot(comparison, stability_time_limit:, wait:)

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

@@ -82,7 +82,7 @@ module Capybara
 
           # Allow nil or single or multiple areas
           if driver_options[:skip_area]
-            driver_options[:skip_area] = driver_options[:skip_area].compact.flatten&.each_cons(4)&.to_a
+            driver_options[:skip_area] = driver_options[:skip_area].compact.flatten&.each_slice(4)&.to_a
           end
 
           if @screenshot_counter

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

@@ -3,7 +3,7 @@
 module Capybara
   module Screenshot
     module Diff
-      VERSION = "1.4.2"
+      VERSION = "1.5.3"
     end
   end
 end

data/lib/capybara/screenshot/diff.rb

@@ -55,6 +55,12 @@ module Capybara
       mattr_accessor(:tolerance) { 0.001 }
 
       AVAILABLE_DRIVERS = Utils.detect_available_drivers.freeze
+      begin
+        require "minitest"
+        ASSERTION = Minitest::Assertion
+      rescue
+        ASSERTION = RuntimeError
+      end
 
       def self.included(klass)
         klass.include TestMethods
@@ -73,7 +79,7 @@ module Capybara
             test_screenshot_errors = @test_screenshots
               .map { |caller, name, compare| assert_image_not_changed(caller, name, compare) }
             test_screenshot_errors.compact!
-            fail(test_screenshot_errors.join("\n\n")) if test_screenshot_errors.any?
+            raise ASSERTION, test_screenshot_errors.join("\n\n") if test_screenshot_errors.any?
           end
         end
       end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: capybara-screenshot-diff
 version: !ruby/object:Gem::Version
-  version: 1.4.2
+  version: 1.5.3
 platform: ruby
 authors:
 - Uwe Kubosch
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2020-12-29 00:00:00.000000000 Z
+date: 2021-12-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: actionpack
@@ -115,16 +115,16 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - "~>"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: '2.5'
+      version: 2.5.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.4
+rubygems_version: 3.2.22
 signing_key:
 specification_version: 4
 summary: Track your GUI changes with diff assertions