Skip to content

Commit

Permalink
chore: switch saucelabs to websocket protocol (angular#2399)
Browse files Browse the repository at this point in the history
* Since saucelabs already supports Websockets now and it's faster we should use Websockets here.
  Reference: https://saucelabs.com/blog/sauce-connect-gets-a-speed-boost-websocket-support
  • Loading branch information
devversion authored and jelbourn committed Dec 28, 2016
1 parent 3230c33 commit dccbe41
Showing 1 changed file with 0 additions and 5 deletions.
5 changes: 0 additions & 5 deletions test/karma.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -103,11 +103,6 @@ export function config(config) {
if (platform == 'saucelabs') {
config.sauceLabs.build = buildId;
config.sauceLabs.tunnelIdentifier = process.env.TRAVIS_JOB_NUMBER;

// TODO(mlaval): remove once SauceLabs supports websockets.
// This speeds up the capturing a bit, as browsers don't even try to use websocket.
console.log('>>>> setting socket.io transport to polling <<<<');
config.transports = ['polling'];
} else if (platform == 'browserstack') {
config.browserStack.build = buildId;
config.browserStack.tunnelIdentifier = process.env.TRAVIS_JOB_NUMBER;
Expand Down

0 comments on commit dccbe41

Please sign in to comment.