CI: Fix screenshots on Sauce not quitting
...when fetching logs fails
diff --git a/script/screenshots.js b/script/screenshots.js
index 516c366..6db65b9 100644
--- a/script/screenshots.js
+++ b/script/screenshots.js
@@ -179,19 +179,17 @@
var logfile = baseDir + '/browser_logs/' + sessionName.replace(/ /g, '_') + '.log';
return new Promise(function(resolve, reject) {
fs.writeFile(logfile, JSON.stringify(logs, null, 2), function(err) {
- if (err) return reject(err);
- console.log(sessionName, 'writeFile');
-
- return resolve(browserDriver.quit());
+ err ? reject(err) : resolve();
});
- });
+ })
+ .then(willLog(sessionName, 'writeFile'));
}, function(err) {
- // the Edge/Internet Explorer drivers don't support logs, but the others do
+ // the Edge, IE, and Firefox-on-macOS drivers don't support logs, but the others do
console.log(sessionName, 'Error fetching logs:', JSON.stringify(err, null, 2));
- return [];
});
});
})
+ .quit()
.fail(function(err) {
console.log('ERROR:', browser.config.browserName, browser.config.platform);
console.log(JSON.stringify(err, null, 2));