Fixes #4425 so tests work after Docker setup #4426
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves #4425
Description
Fixes the Docker setup files so that tests pass out of the box, by installing latest version of Chrome and also setting the DOCKER environment variable.
Tests use Cuprite which relies on a headless Chrome browser. However, you can't run Chrome's browser sandboxed in Docker containers. There's an environment variable called DOCKER when registering Cuprite (see rails_helper.rb) which tells it to run browser as non-sandboxed.
Alternative solutions
Motivation
I wanted to try out Codespaces because I have a relatively old machine and rspec locally takes about 25 minutes, rspec on Codespaces 4-core takes 10 minutes and is much faster.
Type of change
How Has This Been Tested?
bundle exec rspec
in terminal. All tests pass.bundle exec rspec
in terminal. All tests pass.