fix reporter when without --test

nodejs · Apr 12, 2023 · b475dd8 · b475dd8
1 parent 57caf13
commit b475dd8
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 3 deletions.
diff --git a/lib/internal/test_runner/reporter/spec.js b/lib/internal/test_runner/reporter/spec.js
@@ -23,12 +23,16 @@ const colors = {
   '__proto__': null,
   'test:fail': red,
   'test:pass': green,
+  'test:todo': blue,
+  'test:skip': gray,
   'test:diagnostic': blue,
 };
 const symbols = {
   '__proto__': null,
   'test:fail': '\u2716 ',
   'test:pass': '\u2714 ',
+  'test:skip': '\u002D ',
+  'test:todo': '\u25A1 ',
   'test:diagnostic': '\u2139 ',
   'test:coverage': '\u2139 ',
   'arrow:right': '\u25B6 ',
@@ -111,6 +115,10 @@ class SpecReporter extends Transform {
         return this.#handleTestReportEvent(type, data);
       case 'test:pass':
         return this.#handleTestReportEvent(type, data);
+      case 'test:skip':
+        return this.#handleTestReportEvent(type, data);
+      case 'test:todo':
+        return this.#handleTestReportEvent(type, data);
       case 'test:start':
         ArrayPrototypeUnshift(this.#stack, { __proto__: null, data, type });
         break;

diff --git a/lib/internal/test_runner/test.js b/lib/internal/test_runner/test.js
@@ -53,6 +53,7 @@ const {
 const { setTimeout } = require('timers/promises');
 const { TIMEOUT_MAX } = require('internal/timers');
 const { availableParallelism } = require('os');
+const console = require('console')
 const { bigint: hrtime } = process.hrtime;
 const kCallbackAndPromisePresent = 'callbackAndPromisePresent';
 const kCancelledByParent = 'cancelledByParent';
@@ -691,8 +692,14 @@ class Test extends AsyncResource {
       details.type = this.reportedType;
     }
 
-    if (this.passed) {
-      this.reporter.ok(this.nesting, kFilename, this.testNumber, this.name, details, directive);
+    if(this.skipped) {
+      this.reporter.skip(this.nesting, kFilename, this.testNumber, this.name, details, directive);
+    }
+    else if (this.isTodo) {
+      this.reporter.todo(this.nesting, kFilename, this.testNumber, this.name, details, directive);
+    }
+    else if (this.passed) {
+        this.reporter.ok(this.nesting, kFilename, this.testNumber, this.name, details, directive);
     } else {
       details.error = this.error;
       this.reporter.fail(this.nesting, kFilename, this.testNumber, this.name, details, directive);

diff --git a/lib/internal/test_runner/tests_stream.js b/lib/internal/test_runner/tests_stream.js
@@ -3,6 +3,7 @@ const {
   ArrayPrototypePush,
   ArrayPrototypeShift,
 } = primordials;
+const console = require('console')
 const Readable = require('internal/streams/readable');
 
 class TestsStream extends Readable {
@@ -31,7 +32,15 @@ class TestsStream extends Readable {
     this.#emit('test:fail', { __proto__: null, name, nesting, file, testNumber, details, ...directive });
   }
 
-  ok(nesting, file, testNumber, name, details, directive) {
+  skip(nesting, file, testNumber, name, details, directive) {
+    this.#emit('test:skip', { __proto__: null, name, nesting, file, testNumber, details, ...directive });
+  }
+
+  todo(nesting, file, testNumber, name, details, directive) {
+    this.#emit('test:todo', { __proto__: null, name, nesting, file, testNumber, details, ...directive });
+  }
+
+ ok(nesting, file, testNumber, name, details, directive) {
     this.#emit('test:pass', { __proto__: null, name, nesting, file, testNumber, details, ...directive });
   }