capybara-screenshot-diff 1.3.0 → 1.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6e4f48d95925d0da6d7b8afcb53b0179c0477fb034ea88e9e2dde8db02d4fb61
-  data.tar.gz: 9c6c456601df79a02b4b647f3c0cca0c0439194534c73522054c87d39926b89c
+  metadata.gz: 7ea29ec1e5ab38bc1c6eb4eafb7f01bb6fbbf2ca85ea7bbbc8f19dc1bbe9f101
+  data.tar.gz: 71150ea0adabe29276710229034975d66ba9d7029913a6af618e3a1b91d2a6bb
 SHA512:
-  metadata.gz: 54f1f31ec364031eed15cadd22a3e2305874802480efb9ca0b0b2c35a6954293b31552971751f143fc2a6056a96d6d49df31f0c16c3d2785491ed7664f3b1f7c
-  data.tar.gz: 872732c38c6f1822b68d56b94b27c5aeba37ffd1f39379a4e15f76b8ed248d62c787514daa724ed10f22cda1f56b4442dd0753889fc28b99d2fb8299be16c821
+  metadata.gz: 3019d948c07af6e81fc4b54afc98f1ca17565995e8028edc0120127db530d651dcc2cab950df0696ddcdc4964181d05776cf7fe01e616e6e06225a17ba3fe8a5
+  data.tar.gz: 863ee41be9dc9b37311a14ab9eb279230ed7f123f82adee3e644ab558f60921619106d32e0b125857684baea74deadb18c5b3ea70b5691ed2efbc0fdfddf34c4

data/.gitattributes

@@ -0,0 +1,4 @@
+*.gemspec diff=ruby
+*.rake    diff=ruby
+*.rb      diff=ruby
+*.md      diff=md

data/.github/workflows/test.yml

@@ -0,0 +1,138 @@
+name: Test
+
+on:
+  push:
+    branches: [ $default-branch ]
+  pull_request_target:
+    branches: [ $default-branch ]
+  pull_request:
+
+env:
+  BUNDLE_GEMFILE: 'gemfiles/rails61_gems.rb'
+  FERRUM_PROCESS_TIMEOUT: '15'
+  WD_CACHE_TIME: '864000' # 10 days
+
+jobs:
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: '2.7'
+          bundler-cache: true
+
+      - name: Run Standard Ruby linter
+        run: bin/standardrb --no-fix --fail-fast
+
+  test:
+    name: Functional Testing
+    runs-on: ubuntu-20.04 # In order to install libvips 8.9+ version
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: '2.7'
+          bundler-cache: true
+
+      - name: Install libvips
+        run: sudo apt install libvips libvips-dev libvips-tools
+
+      - name: Run Tests with coverage
+        run: bundle exec rake test
+        env:
+          COVERAGE: enabled
+
+      - name: Upload Screenshots
+        if: ${{ always() }}
+        uses: actions/upload-artifact@v2
+        with:
+          path: test/fixtures/app/doc/screenshots/
+
+      - name: Upload Coverage
+        uses: actions/upload-artifact@v2
+        with:
+          name: coverage
+          path: coverage
+
+  matrix:
+    name: Test Integration
+    needs: [ 'test', 'lint' ]
+    runs-on: ubuntu-20.04
+
+    strategy:
+      matrix:
+        ruby-version: [ '2.7', '2.6', '2.5', 'jruby' ]
+        gemfile:
+          - 'rails61_gems.rb'
+          - 'rails60_gems.rb'
+          - 'rails52.gemfile'
+          - 'rails51.gemfile'
+          - 'rails50.gemfile'
+          - 'rails42.gemfile'
+        include:
+          - ruby-version: '3.0'
+            gemfile: 'rails61_gems.rb'
+          - ruby-version: '3.0'
+            gemfile: 'rails60_gems.rb'
+
+
+    env:
+      BUNDLE_GEMFILE: gemfiles/${{ matrix.gemfile }}
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby-version }}
+          bundler-cache: true
+
+      - name: Install libvips
+        run: sudo apt install libvips libvips-dev libvips-tools
+
+      - name: Run tests
+        run: bundle exec rake test
+
+  matrix_screenshot_driver:
+    name: Test Integration
+    needs: [ 'test', 'lint' ]
+    runs-on: ubuntu-20.04
+
+    strategy:
+      matrix:
+        screenshot-driver: [ 'vips', 'chunky_png' ]
+        capybara-driver: [ 'selenium_headless', 'selenium_chrome_headless' ]
+        include:
+          - screenshot-driver: 'chunky_png'
+            capybara-driver: 'cuprite'
+
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: '2.7'
+          bundler-cache: true
+
+      - name: Install libvips
+        run: sudo apt install libvips libvips-dev libvips-tools
+
+      - name: Run tests
+        run: bundle exec rake test:integration
+        env:
+          SCREENSHOT_DRIVER: ${{ matrix.screenshot-driver }}
+          CAPYBARA_DRIVER: ${{ matrix.capybara-driver }}

data/.gitignore

@@ -2,11 +2,11 @@
 /.bundle/
 /.idea
 /.yardoc
-/Gemfile.lock
 /_yardoc/
 /coverage/
 /doc/
 /gemfiles/*.lock
+/gems.locked
 /pkg/
 /spec/reports/
 /tmp/

data/.standard.yml

@@ -0,0 +1,11 @@
+fix: true               # default: false
+parallel: true          # default: false
+format: progress        # default: Standard::Formatter
+ruby_version: 2.5       # default: RUBY_VERSION
+default_ignores: false  # default: true
+
+ignore:                 # default: []
+  - 'gemfiles/**/*':
+    - Security/Eval
+  - 'gemfiles/vendor/**/*'
+  - 'vendor/**/*'

data/Dockerfile

@@ -0,0 +1,60 @@
+# Usage:
+#
+#   $ docker build . -t csd
+#   $ docker run -v $(pwd):/app -ti csd rake test
+
+ARG RUBY_VERSION=2.7.2
+
+FROM circleci/ruby:2.7.2-node-browsers
+
+RUN \
+  # Install dependencies
+  sudo apt-get update && \
+  DEBIAN_FRONTEND=noninteractive sudo apt-get install -y \
+    automake \
+    build-essential \
+    curl \
+    fftw3-dev \
+    gettext \
+    gobject-introspection \
+    gtk-doc-tools \
+    libexif-dev \
+    libfftw3-dev \
+    libgif-dev \
+    libglib2.0-dev \
+    libgsf-1-dev \
+    libgtk2.0-dev \
+    libmagickwand-dev \
+    libmatio-dev \
+    libopenexr-dev \
+    libopenslide-dev \
+    liborc-0.4-dev \
+    libpango1.0-dev \
+    libpoppler-glib-dev \
+    librsvg2-dev \
+    libtiff5-dev \
+    libwebp-dev \
+    libxml2-dev \
+    swig
+
+
+WORKDIR /app
+RUN sudo chmod a+w -R /app
+
+ADD ./bin/install-vips /app/bin/
+RUN sudo /app/bin/install-vips
+
+ADD ./lib/capybara/screenshot/diff/version.rb /app/lib/capybara/screenshot/diff/
+ADD ./capybara-screenshot-diff.gemspec /app/
+ADD ./gems.rb /app/
+
+RUN bundle install
+
+RUN \
+  # Clean up
+  sudo apt-get remove -y curl automake build-essential && \
+  sudo apt-get autoremove -y && \
+  sudo apt-get autoclean && \
+  sudo apt-get clean && \
+  sudo rm -rf /var/lib/apt/lists/* /tmp/* /var/tmp/*
+

data/README.md

@@ -27,6 +27,10 @@ Or install it yourself as:
 
     $ gem install capybara-screenshot-diff
 
+### Requirements
+
+* [for :vips driver] libvips 8.9 or later, see the [libvips install instructions](https://libvips.github.io/libvips/install.html)
+
 ## Usage
 
 ### Minitest
@@ -164,6 +168,24 @@ doc
 ```
 
 
+#### Setting `screenshot_section` and/or `screenshot_group` for all tests
+
+Setting the `screenshot_section` and/or `screenshot_group` for all tests can be
+done in the super class setup:
+
+```ruby
+class ApplicationSystemTestCase < ActionDispatch::SystemTestCase
+  setup do
+    screenshot_section class_name.underscore.sub(/(_feature|_system)?_test$/, '')
+    screenshot_group name[5..-1]
+  end
+end
+```
+
+`screenshot_section` and/or `screenshot_group` can still be overridden in each
+test.
+
+
 
 ### Multiple Capybara drivers
 
@@ -274,7 +296,7 @@ configuration options that that are relevant.
 The most likely one you'll want to modify is ...
 
 ```ruby
-Capybara::Screenshot::Diff.save_path = "other/path"
+Capybara::Screenshot.save_path = "other/path"
 ```
 
 The `save_path` option is relative to `Capybara::Screenshot.root`.
@@ -434,10 +456,70 @@ If you need to ignore multiple areas, you can supply an array of arrays:
 screenshot 'index', skip_area: [[0, 0, 64, 48], [17, 6, 27, 16]]
 ```
 
+### Available Image Processing Drivers
+
+There are several image processing supported by this gem.
+There are several options to setup active driver: `:auto`, `:chunky_png` and `:vips`.
+
+* `:auto` - will try to load `:vips` if there is gem `ruby-vips`, in other cases will load `:chunky_png`
+* `:chunky_png` and `:vips` will load correspondent driver  
+
+### Enable VIPS image processing
+
+[Vips](https://www.rubydoc.info/gems/ruby-vips/Vips/Image) driver provides a faster comparison,
+and could be enabled by adding `ruby-vips` to `Gemfile`.
+
+If need to setup explicitly Vips driver, there are several ways to do this:
+
+  * Globally: `Capybara::Screenshot::Diff.driver = :vips`  
+  * Per screenshot option: `screenshot 'index', driver: :vips`  
+
+With enabled VIPS there are new alternatives to process differences, which easier to find and support.
+For example, `shift_distance_limit` is very heavy operation. Instead better to use `median_filter_window_size`. 
+
+#### Tolerance level (vips only)
+
+You can set a “tolerance” anywhere from 0% to 100%. This is the amount of change that's allowable.
+If the screenshot has changed by more than that amount, it'll flag it as a failure.
+
+This is alternative to "Allowed difference size", only the difference that area calculates including valid pixels.
+But "tolerance" compares only different pixels.
+
+You can use the `tolerance` option to the `screenshot` method to set level:
+
+```ruby
+test 'unstable area' do
+  visit '/'
+  screenshot 'index', tolerance: 0.3
+end
+```
+
+You can also set this globally:
+
+```ruby
+Capybara::Screenshot::Diff.tolerance = 0.3
+```
+
+#### Median filter size (vips only)
+
+This is an alternative to "Allowed shift distance", but much faster.
+You can find more about this strategy on [Median Filter](https://en.wikipedia.org/wiki/Median_filter).
+Think about this like smoothing of the image, before comparison.
+
+You can use the `median_filter_window_size` option to the `screenshot` method to set level:
+
+```ruby
+test 'unstable area' do
+  visit '/'
+  screenshot 'index', median_filter_window_size: 2
+end
+```
 
 ## Development
 
-After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake test` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.
+After checking out the repo, run `bin/setup` to install dependencies.
+Then, run `rake test` to run the tests.
+You can also run `bin/console` for an interactive prompt that will allow you to experiment.
 
 To install this gem onto your local machine, run `bundle exec rake install`.
 
@@ -445,7 +527,9 @@ To release a new version, update the version number in `lib/capybara/screenshot/
 
 ## Contributing
 
-Bug reports and pull requests are welcome on GitHub at https://github.com/donv/capybara-screenshot-diff. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [Contributor Covenant](http://contributor-covenant.org) code of conduct.
+Bug reports and pull requests are welcome on GitHub at https://github.com/donv/capybara-screenshot-diff.
+This project is intended to be a safe, welcoming space for collaboration,
+and contributors are expected to adhere to the [Contributor Covenant](http://contributor-covenant.org) code of conduct.
 
 
 ## License

data/Rakefile

@@ -1,16 +1,18 @@
 # frozen_string_literal: true
 
-require 'bundler/gem_tasks'
-require 'rake/testtask'
+require "bundler/gem_tasks"
+require "rake/testtask"
 
 task default: :test
 
 Rake::TestTask.new(:test) do |t|
-  t.libs << 'test'
-  t.libs << 'lib'
-  t.test_files = FileList['test/**/*_test.rb']
+  t.libs << "test"
+  t.libs << "lib"
+  t.test_files = FileList["test/**/*_test.rb"]
 end
 
-require 'rubocop/rake_task'
-RuboCop::RakeTask.new
-task test: %i[rubocop:auto_correct]
+Rake::TestTask.new("test:integration") do |t|
+  t.libs << "test"
+  t.libs << "lib"
+  t.test_files = FileList["test/integration/**/*_test.rb"]
+end

data/bin/console

@@ -1,8 +1,8 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 
-require 'bundler/setup'
-require 'capybara/screenshot/diff'
+require "bundler/setup"
+require "capybara/screenshot/diff"
 
 # You can add fixtures and/or initialization code here to make experimenting
 # with your gem easier. You can also use a different console, if you like.
@@ -11,5 +11,5 @@ require 'capybara/screenshot/diff'
 # require "pry"
 # Pry.start
 
-require 'irb'
+require "irb"
 IRB.start

data/bin/install-vips

@@ -0,0 +1,11 @@
+#!/bin/bash 
+
+set -e
+
+version=${VIPS_VERSION:-8.10.0}
+
+wget "https://github.com/libvips/libvips/releases/download/v$version/vips-$version.tar.gz"
+tar xf "vips-$version.tar.gz"
+cd "vips-$version"
+CXXFLAGS=-D_GLIBCXX_USE_CXX11_ABI=0 ./configure --enable-debug=no --without-python "$*"
+make && make install && ldconfig

data/bin/standardrb

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+# frozen_string_literal: true
+
+#
+# This file was generated by Bundler.
+#
+# The application 'standardrb' is installed as part of a gem, and
+# this file is here to facilitate running it.
+#
+
+require "pathname"
+ENV["BUNDLE_GEMFILE"] ||= File.expand_path("../../gems.rb",
+  Pathname.new(__FILE__).realpath)
+
+bundle_binstub = File.expand_path("../bundle", __FILE__)
+
+if File.file?(bundle_binstub)
+  if /This file was generated by Bundler/.match?(File.read(bundle_binstub, 300))
+    load(bundle_binstub)
+  else
+    abort("Your `bin/bundle` was not generated by Bundler, so this binstub cannot run.
+Replace `bin/bundle` by running `bundle binstubs bundler --force`, then run this command again.")
+  end
+end
+
+require "rubygems"
+require "bundler/setup"
+
+load Gem.bin_path("standard", "standardrb")