Merge pull request #43 from alphagov/drop-webdrivers-gem

Stop using webdrivers gem
alphagov · Mar 7, 2022 · 60036e5 · 60036e5
2 parents 53a8c72 + 03ca912
commit 60036e5
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 10 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,10 @@
   useful summary for people upgrading their application, not a replication
   of the commit log.
 
+## 3.0.0
+
+* BREAKING: Remove dependency on `webdrivers` gem. It's now expected that ChromeDriver is present on the underlying operating system. ([#43](https://github.com/alphagov/govuk_test/pull/43))
+
 ## 2.3.0
 
 * Update brakeman to fix false positive warning ([#41](https://github.com/alphagov/govuk_test/pull/41))

diff --git a/govuk_test.gemspec b/govuk_test.gemspec
@@ -26,7 +26,6 @@ Gem::Specification.new do |spec|
   spec.add_dependency "capybara"
   spec.add_dependency "puma"
   spec.add_dependency "selenium-webdriver", ">= 3.142"
-  spec.add_dependency "webdrivers", ">= 4"
 
   spec.add_development_dependency "bundler"
   spec.add_development_dependency "climate_control"

diff --git a/lib/govuk_test.rb b/lib/govuk_test.rb
@@ -4,14 +4,6 @@
 require "puma"
 require "selenium-webdriver"
 
-unless ENV["GOVUK_TEST_USE_SYSTEM_CHROMEDRIVER"]
-  require "webdrivers"
-
-  # This stop webdrivers doing a lazy check for new versions
-  # of chromedriver, which interferes with WebMock.disable_net_connect
-  Selenium::WebDriver::Chrome::Service.driver_path = Webdrivers::Chromedriver.update
-end
-
 module GovukTest
   def self.configure
     Capybara.register_driver :headless_chrome do |app|

diff --git a/lib/govuk_test/version.rb b/lib/govuk_test/version.rb
@@ -1,3 +1,3 @@
 module GovukTest
-  VERSION = "2.3.0"
+  VERSION = "3.0.0"
 end