New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
testem: Use --no-sandbox
on TravisCI
#7566
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’m 👍 on the change generally speaking (and think we should get a point release out with it), but had one question RE: the specific environment variable used...
@@ -11,11 +11,14 @@ module.exports = { | |||
Chrome: { | |||
mode: 'ci', | |||
args: [ | |||
// --no-sandbox is needed when running Chrome inside a container | |||
process.env.TRAVIS ? '--no-sandbox' : null, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we use process.env.CI
instead? As I understand it a number of CI providers have this same basic issues (e.g. GitLab CI), and the CI environment flag is pretty universal...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure... GitLab CI can and often does use Docker containers, but you can also run it on real hosts, AppVeyor is likely VM-based, so there are also some CI systems that do not run in containers. The condition is easy enough to adjust if needed 🤔
Nice, thanks for looking into this. I figured the sandbox was desirable for some reason? haha oops |
`ember-qunit` is now generating a test of its own which we did not expect before. Updating the expected number of tests fixes our CI runs.
FYI I had to include two more commits to make CI pass on the |
Related to use of Headless Chrome, see ember-cli/ember-cli#7566.
Related to use of Headless Chrome, see ember-cli/ember-cli#7566.
Similar to ember-cli/ember-cli-chai#45 this updates the
testem.js
file of theapp
blueprint to run Chrome with--no-sandbox
when used on TravisCI to avoid travis-ci/travis-ci#8836/cc @backspace