Skip to content

Commit

Permalink
Fix Travis tests failing on distributions other than trusty (RefugeRe…
Browse files Browse the repository at this point in the history
…strooms#606)

* Revert ".travis.yml: Set "dist" to "trusty" (RefugeRestrooms#600)"

This reverts commit ac8f6ab.

Doing this to run tests on Travis to investigate why they
aren't passing.

* Explicitly require locations.rb in rspec.rb

Tentative fix for tests not passing in xenial but passing in trusty.
This might be because different distributions load files in a
different order. In xenial, `rspec.rb` might get loaded before
`locations.rb`, making `Locations` uninitialized. Explicit require
fixes this.
  • Loading branch information
stardust66 authored and tkwidmer committed Nov 1, 2019
1 parent 7381796 commit 6c96368
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 0 additions & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ env:
global:
- CC_TEST_REPORTER_ID=c18df080592f9c99ca8080a6d5e052aa5fd3964044a0fe0b71e48f8e18998dc2
language: minimal
dist: trusty
services: docker
install:
- docker-compose build
Expand Down
2 changes: 2 additions & 0 deletions spec/support/rspec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@
# spec/spec_helper.rb
#

require_relative './locations.rb'

Capybara.register_driver :poltergeist_debug do |app|
Capybara::Poltergeist::Driver.new(app,
:js_errors => false
Expand Down

0 comments on commit 6c96368

Please sign in to comment.