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
Update Karma and detect available browsers when testing #652
Conversation
In Safari 11.1.2, the default for `credentials` is still "omit".
This was only useful while debugging cache issues.
test/karma.config.js
Outdated
preferHeadless: true, | ||
usePhantomJS: false, | ||
postDetection: availableBrowsers => | ||
availableBrowsers.map(browser => (browser.startsWith('Chrom') ? `${browser}NoSandbox` : browser)) |
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.
is never mind I see it also catches Chrom
intentional here?Chromium
This change causes a bizarre CI failure:
Which would suggest that the version of Chrome used was 62, not 68. This branch uses the same karma-chrome-launcher version as master, so I can't guess right now why would the result be different. |
It seems like it's outdated: https://travis-ci.org/github/fetch/builds/424048996
No description provided.