Skip to content

Commit

Permalink
Use webdrivers gem to simplify local development
Browse files Browse the repository at this point in the history
I noticed when doing an unrelated PR (houndci#1809) that running the specs locally requires `chromedriver`.

```
Failures:

  1) Account user with Stripe Customer ID
     Failure/Error: visit root_path

     Selenium::WebDriver::Error::WebDriverError:
       Unable to find chromedriver. Please download the server from
       https://chromedriver.storage.googleapis.com/index.html and place it somewhere on your PATH.
       More info at https://github.com/SeleniumHQ/selenium/wiki/ChromeDriver.
     # ./spec/support/helpers/authentication_helper.rb:10:in `sign_in_as'
     # ./spec/features/account_spec.rb:17:in `block (2 levels) in <top (required)>'
     # ./spec/support/background_jobs.rb:4:in `block (3 levels) in <top (required)>'
     # ./spec/support/background_jobs.rb:22:in `block in run_background_jobs_immediately'
     # ./spec/support/background_jobs.rb:21:in `run_background_jobs_immediately'
     # ./spec/support/background_jobs.rb:3:in `block (2 levels) in <top (required)>'
```

This PR simplifies local development by using [webdrivers][1] to automatically download the correct
version of `chromedriver`.

[1]: https://github.com/titusfortner/webdrivers
  • Loading branch information
joehorsnell committed Aug 6, 2020
1 parent 5cdb8ec commit 83c86b8
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 1 deletion.
1 change: 1 addition & 0 deletions Gemfile
Original file line number Diff line number Diff line change
Expand Up @@ -56,4 +56,5 @@ group :test do
gem "selenium-webdriver", ">= 4.0.0.alpha4"
gem "shoulda-matchers"
gem "webmock"
gem "webdrivers", "~> 4.4"
end
5 changes: 5 additions & 0 deletions Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -348,6 +348,10 @@ GEM
thread_safe (~> 0.1)
uglifier (4.2.0)
execjs (>= 0.3.0, < 3)
webdrivers (4.4.1)
nokogiri (~> 1.6)
rubyzip (>= 1.3.0)
selenium-webdriver (>= 3.0, < 4.0)
webmock (3.8.2)
addressable (>= 2.3.6)
crack (>= 0.3.2)
Expand Down Expand Up @@ -413,6 +417,7 @@ DEPENDENCIES
split
stripe
uglifier (>= 2.7.2)
webdrivers (~> 4.4)
webmock
webpacker
webpacker-react (~> 1.0.0.beta.1)
Expand Down
7 changes: 6 additions & 1 deletion spec/spec_helper.rb
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
require "attr_extras"
require "byebug"
require "webmock/rspec"
require 'webdrivers'

ENV["REDIS_URL"] = "redis://localhost:6379/1"

Expand All @@ -14,7 +15,11 @@
config.order = "random"
config.include GitHubApiHelper
config.include StripeApiHelper
WebMock.disable_net_connect!(allow_localhost: true)
WebMock.disable_net_connect!(
allow_localhost: true,
# https://github.com/titusfortner/webdrivers/issues/4
allow: 'chromedriver.storage.googleapis.com'
)

config.define_derived_metadata do |meta|
meta[:aggregate_failures] = true
Expand Down

0 comments on commit 83c86b8

Please sign in to comment.