Skip to content

Commit

Permalink
Fix failing SauceLabs tests by updating configuration
Browse files Browse the repository at this point in the history
- Remove code coverage from SauceLabs CI, as it causes tests to hang.
- Update Safari browser tests to handle Safari 9-11 with proper OS
settings
  • Loading branch information
emilyemorehouse committed May 31, 2019
1 parent acabfbd commit adff2aa
Showing 1 changed file with 28 additions and 11 deletions.
39 changes: 28 additions & 11 deletions karma.conf.js
Expand Up @@ -56,7 +56,21 @@ module.exports = function(config) {
// customLaunchers.SL_Safari7 = createCustomLauncher('safari', 7);
// customLaunchers.SL_Safari8 = createCustomLauncher('safari', 8);

customLaunchers.SL_Safari9 = createCustomLauncher('safari', 9);
customLaunchers.SL_Safari9 = createCustomLauncher(
"safari",
9.0,
"OS X 10.11"
);
customLaunchers.SL_Safari10 = createCustomLauncher(
"safari",
"10.1",
"macOS 10.12"
);
customLaunchers.SL_Safari11 = createCustomLauncher(
"safari",
"11.1",
"macOS 10.13"
);
}

// Opera
Expand Down Expand Up @@ -141,7 +155,9 @@ module.exports = function(config) {
// test results reporter to use
// possible values: 'dots', 'progress'
// available reporters: https://npmjs.org/browse/keyword/karma-reporter
reporters: ['dots', 'coverage', 'saucelabs'],
// Disable code coverage, as it's breaking CI:
// reporters: ['dots', 'coverage', 'saucelabs'],
reporters: ['dots', 'saucelabs'],


// web server port
Expand Down Expand Up @@ -181,15 +197,16 @@ module.exports = function(config) {
webpack: {
cache: true,
devtool: 'inline-source-map',
module: {
postLoaders: [
{
test: /\.js$/,
exclude: /(node_modules|test)/,
loader: 'istanbul-instrumenter'
}
]
},
// Disable code coverage, as it's breaking CI
// module: {
// postLoaders: [
// {
// test: /\.js$/,
// exclude: /(node_modules|test)/,
// loader: 'istanbul-instrumenter'
// }
// ]
// },
externals: [
{
'./adapters/http': 'var undefined'
Expand Down

0 comments on commit adff2aa

Please sign in to comment.