Skip to content

Commit 05cfa51

Browse files
committed
Comppletely register new driver
Do not try to inherit from the selenium web driver. It does not contain all the args we need.
1 parent 54f9147 commit 05cfa51

File tree

1 file changed

+10
-12
lines changed

1 file changed

+10
-12
lines changed

lib/solidus_dev_support/rspec/capybara.rb

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -13,19 +13,17 @@
1313
Capybara.default_max_wait_time = 10
1414
Capybara.server = :puma, { Silent: true } # A fix for rspec/rspec-rails#1897
1515

16-
Capybara.drivers[:selenium_chrome_headless].tap do |original_driver|
17-
Capybara.register_driver :solidus_chrome_headless do |app|
18-
options = Selenium::WebDriver::Chrome::Options.new.tap do |opts|
19-
opts.add_argument("--headless=new")
20-
opts.add_argument("--disable-gpu")
21-
opts.add_argument("--no-sandbox")
22-
opts.add_argument("--window-size=#{CAPYBARA_WINDOW_SIZE}")
23-
opts.add_argument("--disable-search-engine-choice-screen")
24-
opts.add_argument("--disable-backgrounding-occluded-windows")
25-
opts.browser_version = CAPYBARA_JAVASCRIPT_DRIVER_VERSION
26-
end
27-
original_driver.call(app, options: options)
16+
Capybara.register_driver :solidus_chrome_headless do |app|
17+
browser_options = Selenium::WebDriver::Chrome::Options.new.tap do |opts|
18+
opts.add_argument("--headless=new")
19+
opts.add_argument("--disable-gpu")
20+
opts.add_argument("--no-sandbox")
21+
opts.add_argument("--window-size=#{CAPYBARA_WINDOW_SIZE}")
22+
opts.add_argument("--disable-search-engine-choice-screen")
23+
opts.add_argument("--disable-backgrounding-occluded-windows")
24+
opts.browser_version = CAPYBARA_JAVASCRIPT_DRIVER_VERSION
2825
end
26+
Capybara::Selenium::Driver.new(app, browser: :chrome, options: browser_options)
2927
end
3028

3129
require 'capybara-screenshot/rspec'

0 commit comments

Comments
 (0)