From 8d1d6dace36d3d274b94224d1d516c5adbc7227b Mon Sep 17 00:00:00 2001 From: Thomas von Deyen Date: Wed, 7 Sep 2022 17:47:22 +0200 Subject: [PATCH 1/2] Configure solidus_chrome_headless as capybara-screenshot driver We have a custom driver. Capybara-screenshot needs to know about it. --- lib/solidus_dev_support/rspec/feature_helper.rb | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/lib/solidus_dev_support/rspec/feature_helper.rb b/lib/solidus_dev_support/rspec/feature_helper.rb index 7f1328af..d3be3119 100644 --- a/lib/solidus_dev_support/rspec/feature_helper.rb +++ b/lib/solidus_dev_support/rspec/feature_helper.rb @@ -12,6 +12,10 @@ require 'capybara-screenshot/rspec' require 'solidus_dev_support/rspec/capybara' +Capybara::Screenshot.register_driver(:solidus_chrome_headless) do |driver, path| + driver.browser.save_screenshot(path) +end + def dev_support_assets_preload if Rails.application.respond_to?(:precompiled_assets) Rails.application.precompiled_assets From fe9ec8a1184d88290f160264b5896abc342fdef5 Mon Sep 17 00:00:00 2001 From: Thomas von Deyen Date: Wed, 7 Sep 2022 17:58:20 +0200 Subject: [PATCH 2/2] Move capybara-screenshot require into caybara config file. Have all capybara setup files in one place. --- lib/solidus_dev_support/rspec/capybara.rb | 6 ++++++ lib/solidus_dev_support/rspec/feature_helper.rb | 6 ------ 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/lib/solidus_dev_support/rspec/capybara.rb b/lib/solidus_dev_support/rspec/capybara.rb index 5d3f0d66..b4bf2dbb 100644 --- a/lib/solidus_dev_support/rspec/capybara.rb +++ b/lib/solidus_dev_support/rspec/capybara.rb @@ -21,4 +21,10 @@ end end +require 'capybara-screenshot/rspec' + +Capybara::Screenshot.register_driver(:solidus_chrome_headless) do |driver, path| + driver.browser.save_screenshot(path) +end + require 'spree/testing_support/capybara_ext' diff --git a/lib/solidus_dev_support/rspec/feature_helper.rb b/lib/solidus_dev_support/rspec/feature_helper.rb index d3be3119..c17ff943 100644 --- a/lib/solidus_dev_support/rspec/feature_helper.rb +++ b/lib/solidus_dev_support/rspec/feature_helper.rb @@ -8,14 +8,8 @@ # require 'solidus_dev_support/rspec/rails_helper' - -require 'capybara-screenshot/rspec' require 'solidus_dev_support/rspec/capybara' -Capybara::Screenshot.register_driver(:solidus_chrome_headless) do |driver, path| - driver.browser.save_screenshot(path) -end - def dev_support_assets_preload if Rails.application.respond_to?(:precompiled_assets) Rails.application.precompiled_assets