Skip to content

Commit

Permalink
chore(ci/client/browserstack): modify concurrency and timeout settings
Browse files Browse the repository at this point in the history
In our client tests, for BrowserStack, reduce the `browserDisconnectTimeout`
and the `browserNoActivityTimeout` to a minute, instead of 5. Also, reduce the
concurrency of browsers to 2 instead of 3. This is because BS is being flaky
when handling many browsers concurrently and, when a browser has been
disconnected it will generally not come back, so we do not need to wait a whole
5 minutes; it is better to just use another try from the
`browserDisconnectTolerance` configuration pool. The overall result is a more
efficient CI run.
  • Loading branch information
Jonathan Ginsburg authored and anthony-redFox committed May 5, 2023
1 parent c3fe0de commit f0daadb
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/client/karma.conf.js
Original file line number Diff line number Diff line change
Expand Up @@ -103,9 +103,9 @@ module.exports = function (config) {
// https://github.com/karma-runner/karma-browserstack-launcher/issues/61
captureTimeout: 3e5,
browserDisconnectTolerance: 3,
browserDisconnectTimeout: 3e5,
browserDisconnectTimeout: 6e4,
browserSocketTimeout: 1.2e5,
browserNoActivityTimeout: 3e5,
browserNoActivityTimeout: 6e4,

// Auto run tests on start (when browsers are captured) and exit
// CLI --single-run --no-single-run
Expand All @@ -124,7 +124,7 @@ module.exports = function (config) {
'karma-browserstack-launcher'
],

concurrency: 3,
concurrency: 2,

forceJSONP: true,

Expand Down

0 comments on commit f0daadb

Please sign in to comment.