Skip to content

Testing travis ci sauce labs - Ready for Review / Merge @muxator #3743

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.

Sign up for GitHub

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

Merged
muxator merged 4 commits into from
Mar 23, 2020
Merged

Testing travis ci sauce labs - Ready for Review / Merge @muxator #3743

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9558f1e
tests: configure the OS/Browsers versions on which tests will be run
JohnMcLear Mar 22, 2020
d1f3cd6
tests: enable sauce labs tests again
JohnMcLear Mar 22, 2020
ff77b50
tests: quote bash variables in sauce_tunnel.sh
muxator Mar 23, 2020
79427d1
tests: explicit tunnel version URI in sauce_tunnel.sh
JohnMcLear Mar 22, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions .travis.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,17 @@ node_js:
services:
- docker

install:
- "bin/installDeps.sh"
- "export GIT_HASH=$(git rev-parse --verify --short HEAD)"
- "npm install ep_test_line_attrib"

before_script:
- "tests/frontend/travis/sauce_tunnel.sh"

script:
- "tests/frontend/travis/runner.sh"

env:
global:
- secure: "WMGxFkOeTTlhWB+ChMucRtIqVmMbwzYdNHuHQjKCcj8HBEPdZLfCuK/kf4rG\nVLcLQiIsyllqzNhBGVHG1nyqWr0/LTm8JRqSCDDVIhpyzp9KpCJQQJG2Uwjk\n6/HIJJh/wbxsEdLNV2crYU/EiVO3A4Bq0YTHUlbhUqG3mSCr5Ec="
Expand Down
32 changes: 15 additions & 17 deletions tests/frontend/travis/remote_runner.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -67,41 +67,39 @@ var sauceTestWorker = async.queue(function (testSettings, callback) {
});
}, 5); //run 5 tests in parrallel

// Firefox
// 1) Firefox on Linux
sauceTestWorker.push({
'platform' : 'Linux'
, 'browserName' : 'firefox'
, 'version' : ''
, 'version' : 'latest'
});

// Chrome
// 2) Chrome on Linux
sauceTestWorker.push({
'platform' : 'Linux'
, 'browserName' : 'googlechrome'
, 'version' : ''
, 'version' : 'latest'
});

/*
// IE 8
// 3) Safari on OSX 10.15
sauceTestWorker.push({
'platform' : 'Windows 2003'
, 'browserName' : 'iexplore'
, 'version' : '8'
'platform' : 'OS X 10.15'
, 'browserName' : 'safari'
, 'version' : '13'
});
*/

// IE 9
// 4) IE 10 on Win 8
sauceTestWorker.push({
'platform' : 'Windows XP'
'platform' : 'Windows 8'
, 'browserName' : 'iexplore'
, 'version' : '9'
, 'version' : '10.0'
Copy link
Copy Markdown
Contributor

@muxator muxator Mar 23, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it work if we put IE 11 here? I know it took a lot of time to find these combinations.

IE 10 is practically out of support.

Copy link
Copy Markdown
Member Author

@JohnMcLear JohnMcLear Mar 23, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I intentionally chose IE10 and Edge to get opposite ends of the spectrum from MS. You could bring another blob in for IE11 should you wish. We're supposed to have 5 concurrent tests run but afaik it only does 4(saucelabs is weird) so it wouldn't slow things down at all..

Copy link
Copy Markdown
Contributor

@muxator muxator Mar 23, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fine. Let's keep IE 10.

});

// IE 10
// 5) Edge on Win 10
sauceTestWorker.push({
'platform' : 'Windows 2012'
, 'browserName' : 'iexplore'
, 'version' : '10'
'platform' : 'Windows 10'
, 'browserName' : 'microsoftedge'
, 'version' : 'latest'
});

sauceTestWorker.drain = function() {
Expand Down
4 changes: 2 additions & 2 deletions tests/frontend/travis/sauce_tunnel.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,12 +6,12 @@
# example https://saucelabs.com/downloads/sc-4.5.4-linux.tar.gz
# Supported versions are currently listed at:
# https://wiki.saucelabs.com/display/DOCS/Downloading+Sauce+Connect+Proxy
curl https://saucelabs.com/downloads/sc-latest-linux.tar.gz > /tmp/sauce.tar.gz
curl https://saucelabs.com/downloads/sc-4.5.4-linux.tar.gz > /tmp/sauce.tar.gz
tar zxf /tmp/sauce.tar.gz --directory /tmp
mv /tmp/sc-*-linux /tmp/sauce_connect

# start the sauce connector in background and make sure it doesn't output the secret key
(/tmp/sauce_connect/bin/sc --user $SAUCE_USERNAME --key $SAUCE_ACCESS_KEY --pidfile /tmp/sauce.pid --readyfile /tmp/tunnel > /dev/null )&
(/tmp/sauce_connect/bin/sc --user "${SAUCE_USERNAME}" --key "${SAUCE_ACCESS_KEY}" --pidfile /tmp/sauce.pid --readyfile /tmp/tunnel > /dev/null )&

# wait for the tunnel to build up
while [ ! -e "/tmp/tunnel" ]
Expand Down