console: fix trace function

A recent refactoring made the slight mistake of calling `trace()` instead of `this.trace()`. PR-URL: #26764 Fixes: #26763 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Gus Caplan <me@gus.host> Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com> Reviewed-By: Weijia Wang <starkwang@126.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
nodejs · Mar 27, 2019 · e49cd40 · e49cd40
1 parent c7b34cd
commit e49cd40
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 1 deletion.
diff --git a/lib/internal/console/constructor.js b/lib/internal/console/constructor.js
@@ -330,7 +330,7 @@ const consoleMethods = {
       name: 'Trace',
       message: this[kFormatForStderr](args)
     };
-    Error.captureStackTrace(err, trace);
+    Error.captureStackTrace(err, this.trace);
     this.error(err.stack);
   },
 

diff --git a/test/message/console.js b/test/message/console.js
@@ -0,0 +1,5 @@
+'use strict';
+
+require('../common');
+
+console.trace('foo');
diff --git a/test/message/console.out b/test/message/console.out
@@ -0,0 +1,10 @@
+Trace: foo
+    at Console.trace (*)
+    at Object.<anonymous> (*console.js:*:*)
+    at *
+    at *
+    at *
+    at *
+    at *
+    at *
+    at *