feat: include file and line number on console errors

closes searls#326
klieber · Aug 24, 2022 · bcb614d · bcb614d
1 parent 123ba81
commit bcb614d
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/main/resources/jasmine-templates/RequireJsSpecRunner.htmltemplate b/src/main/resources/jasmine-templates/RequireJsSpecRunner.htmltemplate
@@ -9,13 +9,14 @@
   <script type="application/javascript">
     window.onerror = function(msg, url, line) {
       var errors = window.consoleErrors = window.consoleErrors || [];
-      errors.push(msg);
+      errors.push(msg + ' (' + url + ':' + line + ')');
     };
 
     function getConsoleErrors() {
       var errors = [];
       if (jsApiReporter && jsApiReporter.runDetails && jsApiReporter.runDetails.failedExpectations) {
-        errors = jsApiReporter.runDetails.failedExpectations.map((error) => error.message);
+        errors = jsApiReporter.runDetails.failedExpectations
+          .map((error) => error.message + ' (' + error.filename + ':' + error.lineno + ')');
       }
 
       if (errors.length === 0) {

diff --git a/src/main/resources/jasmine-templates/SpecRunner.htmltemplate b/src/main/resources/jasmine-templates/SpecRunner.htmltemplate
@@ -9,13 +9,14 @@
   <script type="application/javascript">
     window.onerror = function(msg, url, line) {
       var errors = window.consoleErrors = window.consoleErrors || [];
-      errors.push(msg);
+      errors.push(msg + ' (' + url + ':' + line + ')');
     };
 
     function getConsoleErrors() {
       var errors = [];
       if (jsApiReporter && jsApiReporter.runDetails && jsApiReporter.runDetails.failedExpectations) {
-        errors = jsApiReporter.runDetails.failedExpectations.map((error) => error.message);
+        errors = jsApiReporter.runDetails.failedExpectations
+          .map((error) => error.message + ' (' + error.filename + ':' + error.lineno + ')');
       }
 
       if (errors.length === 0) {