Always print some diagnostics to stderr on error

Even if event.error is defined, the other props includes valuable information
sinonjs · Sep 24, 2018 · 88891f5 · 88891f5
1 parent b5537a5
commit 88891f5
Showing 1 changed file with 14 additions and 7 deletions.
diff --git a/test/webworker/webworker-support-assessment.js b/test/webworker/webworker-support-assessment.js
@@ -24,17 +24,24 @@ if (typeof Worker !== "undefined") {
              */
             function onError(ev) {
                 var error = ev.error;
+                var msg =
+                    "An error happened at line " +
+                    [ev.lineno, ev.colno].join(":") +
+                    " in file " +
+                    ev.filename +
+                    ":  " +
+                    ev.message;
 
                 if (!error) {
-                    var msg =
-                        "An error happened at line " +
-                        [ev.lineno, ev.colno].join(":") +
-                        " in file " +
-                        ev.filename +
-                        ":  " +
-                        ev.message;
                     error = new Error(msg);
                 }
+
+                // `ev.error` might be defined, but still might include
+                // very little information, which is why we want to
+                // include as much information as we possibly can
+
+                console.error(msg); // eslint-disable-line no-console
+
                 done(error);
             }