Fix sourcemapped stacktrace output to use the original column number

Financial-Times · Jun 15, 2020 · 4a400a0 · 4a400a0
1 parent b64671d
commit 4a400a0
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/lib/plugins/listr-karma.js b/lib/plugins/listr-karma.js
@@ -37,9 +37,11 @@ module.exports = function () {
 							errors.push(colors.red(error.join('\n')));
 
 							if (isCI) {
-								const error = value.log.map(log => formatError(log, '\t')).join('\n').split('\n');
-								const message = error.filter(log => log.trim().length).shift().trim();
-								const [file, line, column] = error.pop().trim().replace('at ', '').split(':');
+								const error = value.log.map(log => formatError(log, '\t')).join('\n').split('\n').filter(log => log.trim().length);
+								const message = error.shift().trim();
+								// eslint-disable-next-line prefer-const
+								let [file, line, column] = error.pop().trim().replace('at ', '').split(':');
+								column = Number.parseInt(column, 10);
 								const fullTestDescription = value.suite.map((suite) => suite + ' > ').join('') + value.description;
 
 								const errorMessage = fullTestDescription + '\n' + browser.name + ' errored with: ' + message;