fix suggestions

nodejs · Dec 12, 2023 · 175af14 · 175af14
1 parent e4eb26b
commit 175af14
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/lib/internal/test_runner/utils.js b/lib/internal/test_runner/utils.js
@@ -132,7 +132,7 @@ function tryBuiltinReporter(name) {
 function colorizeTestFiles(rootTest) {
   const { reporters, destinations } = parseCommandLine();
   ArrayPrototypeForEach(reporters, (_, index) => {
-    const destination = kBuiltinDestinations.get(destinations[index]) ?? createWriteStream(destinations[index]);
+    const destination = kBuiltinDestinations.get(destinations[index]);
     rootTest.harness.shouldColorizeTestFiles ||= shouldColorize(destination);
   });
 }

diff --git a/test/parallel/test-runner-run.mjs b/test/parallel/test-runner-run.mjs
@@ -1,14 +1,10 @@
-// Flags: --expose-internals
-
 import * as common from '../common/index.mjs';
 import * as fixtures from '../common/fixtures.mjs';
 import { join } from 'node:path';
 import { describe, it, run } from 'node:test';
 import { dot, spec, tap } from 'node:test/reporters';
 import assert from 'node:assert';
-import stream from 'internal/test_runner/tests_stream';
 
-const { TestsStream } = stream;
 const testFixtures = fixtures.path('test-runner');
 
 describe('require(\'node:test\').run', { concurrency: true }, () => {
@@ -474,7 +470,7 @@ describe('require(\'node:test\').run', { concurrency: true }, () => {
       const stream = run({
         files: [join(testFixtures, 'default-behavior/test/random.cjs')],
         setup: common.mustCall((root) => {
-          assert(root instanceof TestsStream);
+          assert.strictEqual(root.constructor.name, "TestsStream");
         }),
       });
       stream.on('test:fail', common.mustNotCall());