Skip to content

Commit 682e41b

Browse files
authored
Add message for edgedriver download failures (#591)
1 parent 9dc1ce6 commit 682e41b

File tree

2 files changed

+16
-2
lines changed

2 files changed

+16
-2
lines changed

lib/install.js

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -301,7 +301,21 @@ async function install(_opts) {
301301
.once('end', () => {
302302
downloadStreams.delete(downloadStream);
303303
})
304-
.once('error', (err) => reject(logError('getDownloadStream', err, 'Could not download ' + downloadUrl)));
304+
.once('error', (err) => {
305+
if (err.code === 'ERR_NON_2XX_3XX_RESPONSE' && downloadUrl.includes('edge')) {
306+
reject(
307+
logError(
308+
'getDownloadStream',
309+
err,
310+
'It may be due to the specified edge driver version ' +
311+
downloadUrl.split('/')[3] +
312+
' is unavailable for current platform. Try downloading a different version of edge driver '
313+
)
314+
);
315+
} else {
316+
reject(logError('getDownloadStream', err, 'Could not download ' + downloadUrl));
317+
}
318+
});
305319
});
306320
}
307321
}

lib/log-error.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
const logError = (fnName, error, message = '') => {
2-
console.error(`Error in "${fnName}". ${message}\nSee more details below:`);
2+
console.error(`\nError in "${fnName}". ${message}\nSee more details below:`);
33
if (error) {
44
if (error.response) {
55
console.log(error.response.statusCode, error.response.url);

0 commit comments

Comments
 (0)