test: enable skipped colorize test

islandryu · RafaelGSS · commit 7039173398a9 · 2025-05-02T17:06:31.000-03:00
PR-URL: #57887 Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
diff --git a/test/fixtures/test-runner/output/coverage-width-80-color.snapshot b/test/fixtures/test-runner/output/coverage-width-80-color.snapshot
@@ -20,7 +20,7 @@
 ℹ output | [31m [34m | [31m [34m | [31m [34m | 
 ℹ [32mcoverage-width-80-color.mjs [34m | [32m100.00[34m | [32m 100.00[34m | [32m 100.00[34m | 
 ℹ ------------------------------------------------------------------------------
-ℹ all files | [33m 61.33[34m | [32m 100.00[34m | [31m 0.00[34m | 
+ℹ all files | [33m 62.34[34m | [32m 100.00[34m | [31m 0.00[34m | 
 ℹ ------------------------------------------------------------------------------
 ℹ end of coverage report
 [39m
diff --git a/test/fixtures/test-runner/output/coverage-width-80-uncovered-lines-color.snapshot b/test/fixtures/test-runner/output/coverage-width-80-uncovered-lines-color.snapshot
@@ -21,7 +21,7 @@
 ℹ output | [31m [34m | [31m [34m | [31m [34m | 
 ℹ [32mcoverage-width-80-uncovered-lines-color.mjs [34m | [32m100.00[34m | [32m 100.00[34m | [32m 100.00[34m | 
 ℹ --------------------------------------------------------------------------------------------------
-ℹ all files | [33m 52.91[34m | [33m 60.00[34m | [31m 1.61[34m | 
+ℹ all files | [33m 53.24[34m | [33m 60.00[34m | [31m 1.61[34m | 
 ℹ --------------------------------------------------------------------------------------------------
 ℹ end of coverage report
 [39m
diff --git a/test/parallel/test-runner-output.mjs b/test/parallel/test-runner-output.mjs
@@ -1,18 +1,18 @@
+// Flags: --expose-internals
 import * as common from '../common/index.mjs';
 import * as fixtures from '../common/fixtures.mjs';
 import * as snapshot from '../common/assertSnapshot.js';
 import { describe, it } from 'node:test';
 import { hostname } from 'node:os';
 import { chdir, cwd } from 'node:process';
 import { fileURLToPath } from 'node:url';
+import internalTTy from 'internal/tty';
 
 const skipForceColors =
 process.config.variables.icu_gyp_path !== 'tools/icu/icu-generic.gyp' ||
 process.config.variables.node_shared_openssl;
 
-const canColorize = process.stderr?.isTTY && (
- typeof process.stderr?.getColorDepth === 'function' ?
- process.stderr?.getColorDepth() > 2 : true);
+const canColorize = internalTTy.getColorDepth() > 2;
 const skipCoverageColors = !canColorize;
 
 function replaceTestDuration(str) {

-Original file line number
+Diff line change
@@ @@ -1,18 +1,18 @@ @@
 +// Flags: --expose-internals
 import*ascommonfrom'../common/index.mjs';
 import*asfixturesfrom'../common/fixtures.mjs';
 import*assnapshotfrom'../common/assertSnapshot.js';
 import{describe,it}from'node:test';
 import{hostname}from'node:os';
 import{chdir,cwd}from'node:process';
 import{fileURLToPath}from'node:url';
 +importinternalTTyfrom'internal/tty';
 constskipForceColors=
 process.config.variables.icu_gyp_path!=='tools/icu/icu-generic.gyp'||
 process.config.variables.node_shared_openssl;
 -constcanColorize=process.stderr?.isTTY&&(
 -typeofprocess.stderr?.getColorDepth==='function' ?
 -process.stderr?.getColorDepth()>2 : true);
 +constcanColorize=internalTTy.getColorDepth()>2;
 constskipCoverageColors=!canColorize;
 functionreplaceTestDuration(str){