Skip to content

Commit c0b4065

Browse files
committedAug 10, 2018
Fix failing SauceLabs tests by updating configuration
- 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
1 parent 5273811 commit c0b4065

File tree

1 file changed

+28
-11
lines changed

1 file changed

+28
-11
lines changed
 

‎karma.conf.js

+28-11
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,21 @@ module.exports = function(config) {
5656
// customLaunchers.SL_Safari7 = createCustomLauncher('safari', 7);
5757
// customLaunchers.SL_Safari8 = createCustomLauncher('safari', 8);
5858

59-
customLaunchers.SL_Safari9 = createCustomLauncher('safari', 9);
59+
customLaunchers.SL_Safari9 = createCustomLauncher(
60+
"safari",
61+
9.0,
62+
"OS X 10.11"
63+
);
64+
customLaunchers.SL_Safari10 = createCustomLauncher(
65+
"safari",
66+
"10.1",
67+
"macOS 10.12"
68+
);
69+
customLaunchers.SL_Safari11 = createCustomLauncher(
70+
"safari",
71+
"11.1",
72+
"macOS 10.13"
73+
);
6074
}
6175

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

143159

144160
// web server port
@@ -178,15 +194,16 @@ module.exports = function(config) {
178194
webpack: {
179195
cache: true,
180196
devtool: 'inline-source-map',
181-
module: {
182-
postLoaders: [
183-
{
184-
test: /\.js$/,
185-
exclude: /(node_modules|test)/,
186-
loader: 'istanbul-instrumenter'
187-
}
188-
]
189-
},
197+
// Disable code coverage, as it's breaking CI
198+
// module: {
199+
// postLoaders: [
200+
// {
201+
// test: /\.js$/,
202+
// exclude: /(node_modules|test)/,
203+
// loader: 'istanbul-instrumenter'
204+
// }
205+
// ]
206+
// },
190207
externals: [
191208
{
192209
'./adapters/http': 'var undefined'

0 commit comments

Comments
 (0)
Please sign in to comment.