libretic
/
etherpad-lite
5
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

remote_runner: Use Error objects to convey pass/fail

This commit is contained in:
Richard Hansen 2021-05-02 16:40:01 -04:00
parent 4ec02a9af9
commit 7f57b17b2e
1 changed files with 8 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/tests/frontend/travis/remote_runner.js
View File

@ -47,14 +47,14 @@ const sauceTestWorker = async.queue((testSettings, callback) => {
log(`Remote sauce test started! ${url}`, pfx);
// tear down the test excecution
const stopSauce = (success, timesup) => {
const stopSauce = (err) => {
clearInterval(getStatusInterval);
clearTimeout(timeout);
browser.quit(() => {
if (!success) process.exitCode = 1;
if (timesup) {
log('[red]FAILED[clear] allowed test duration exceeded', pfx);
if (err) {
log(`[red]FAILED[clear] ${err}`, pfx);
process.exitCode = 1;
}
log(`Remote sauce test finished! ${url}`, pfx);
@ -70,7 +70,7 @@ const sauceTestWorker = async.queue((testSettings, callback) => {
* https://wiki.saucelabs.com/display/DOCS/Test+Configuration+Options#TestConfigurationOptions-Timeouts
*/
const timeout = setTimeout(() => {
stopSauce(false, true);
stopSauce(new Error('allowed test duration exceeded'));
}, 870000); // travis timeout is 15 minutes, set this to a slightly lower value
// how many characters of the log have been sent to travis
@ -83,7 +83,9 @@ const sauceTestWorker = async.queue((testSettings, callback) => {
consoleText.substring(logIndex).split('\\n').forEach((line) => log(line, pfx));
logIndex = consoleText.length;
const [finished, nFailedStr] = consoleText.match(finishedRegex) || [];
if (finished) stopSauce(nFailedStr === '0');
if (finished) {
stopSauce(nFailedStr === '0' ? null : new Error(`${nFailedStr} tests failed`));
}
});
}, 5000);
});