test_runner: fix spec skip detection

PR-URL: #47537
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Benjamin Gruenbaum <[email protected]>

Author: MoLow

lib/internal/test_runner/runner.js

@@ -190,9 +190,9 @@ class FileTest extends Test {
         let directive;
 
         if (skip) {
-          directive = this.reporter.getSkip(node.reason);
+          directive = this.reporter.getSkip(node.reason || true);
         } else if (todo) {
-          directive = this.reporter.getTodo(node.reason);
+          directive = this.reporter.getTodo(node.reason || true);
         } else {
           directive = kEmptyObject;
         }

test/fixtures/test-runner/output/spec_reporter.snapshot

@@ -150,7 +150,7 @@
  test with a name and options provided (*ms) # SKIP
  functionAndOptions (*ms) # SKIP
  escaped description \ # \#\ 
- 	     (*ms)
+ 	     (*ms)
  escaped skip message (*ms) # SKIP
  escaped todo message (*ms)
  escaped diagnostic (*ms)

test/fixtures/test-runner/output/spec_reporter_cli.js

@@ -0,0 +1,12 @@
+// Flags: --no-warnings
+'use strict';
+require('../../../common');
+const fixtures = require('../../../common/fixtures');
+const spawn = require('node:child_process').spawn;
+
+const child = spawn(process.execPath,
+                    ['--no-warnings', '--test', '--test-reporter', 'spec', fixtures.path('test-runner/output/output.js')],
+                    { stdio: 'pipe' });
+// eslint-disable-next-line no-control-regex
+child.stdout.on('data', (d) => process.stdout.write(d.toString().replace(/[^\x00-\x7F]/g, '').replace(/\u001b\[\d+m/g, '')));
+child.stderr.pipe(process.stderr);