test: fix test runner colored output test

PR-URL: #51064 Reviewed-By: Raz Luvaton <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Chemi Atlow <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: MoLow

lib/internal/test_runner/reporter/spec.js

@@ -14,19 +14,11 @@ const{
 constassert=require('assert');
 constTransform=require('internal/streams/transform');
 const{ inspectWithNoCustomRetry }=require('internal/errors');
-const{ green, blue, red, white, gray, shouldColorize }=require('internal/util/colors');
+constcolors=require('internal/util/colors');
 const{ kSubtestsFailed }=require('internal/test_runner/test');
 const{ getCoverageReport }=require('internal/test_runner/utils');
 const{ relative }=require('path');
 
-constinspectOptions={__proto__: null,colors: shouldColorize(process.stdout),breakLength: Infinity};
-
-constcolors={
-'__proto__': null,
-'test:fail': red,
-'test:pass': green,
-'test:diagnostic': blue,
-};
 constsymbols={
 '__proto__': null,
 'test:fail': '\u2716 ',
@@ -42,9 +34,19 @@ class SpecReporter extends Transform{
  #indentMemo =newSafeMap();
  #failedTests =[];
  #cwd =process.cwd();
+ #inspectOptions;
+ #colors;
 
 constructor(){
 super({__proto__: null,writableObjectMode: true});
+colors.refresh();
+this.#inspectOptions ={__proto__: null,colors: colors.shouldColorize(process.stdout),breakLength: Infinity};
+this.#colors ={
+'__proto__': null,
+'test:fail': colors.red,
+'test:pass': colors.green,
+'test:diagnostic': colors.blue,
+};
 }
 
  #indent(nesting){
@@ -62,15 +64,15 @@ class SpecReporter extends Transform{
 constmessage=ArrayPrototypeJoin(
 RegExpPrototypeSymbolSplit(
 hardenRegExp(/\r?\n/),
-inspectWithNoCustomRetry(err,inspectOptions),
+inspectWithNoCustomRetry(err,this.#inspectOptions),
 ),`\n${indent} `);
 return`\n${indent}${message}\n`;
 }
  #formatTestReport(type,data,prefix='',indent='',hasChildren=false){
-letcolor=colors[type]??white;
+letcolor=this.#colors[type]??colors.white;
 letsymbol=symbols[type]??' ';
 const{ skip, todo }=data;
-constduration_ms=data.details?.duration_ms ? ` ${gray}(${data.details.duration_ms}ms)${white}` : '';
+constduration_ms=data.details?.duration_ms ? ` ${colors.gray}(${data.details.duration_ms}ms)${colors.white}` : '';
 lettitle=`${data.name}${duration_ms}`;
 
 if(skip!==undefined){
@@ -82,13 +84,13 @@ class SpecReporter extends Transform{
 if(hasChildren){
 // If this test has had children - it was already reported, so slightly modify the output
 consterr=data.details?.error?.failureType==='subtestsFailed' ? '' : error;
-return`${prefix}${indent}${color}${symbols['arrow:right']}${white}${title}\n${err}`;
+return`${prefix}${indent}${color}${symbols['arrow:right']}${colors.white}${title}\n${err}`;
 }
 if(skip!==undefined){
-color=gray;
+color=colors.gray;
 symbol=symbols['hyphen:minus'];
 }
-return`${prefix}${indent}${color}${symbol}${title}${white}${error}`;
+return`${prefix}${indent}${color}${symbol}${title}${colors.white}${error}`;
 }
  #handleTestReportEvent(type,data){
 constsubtest=ArrayPrototypeShift(this.#stack);// This is the matching `test:start` event
@@ -130,9 +132,9 @@ class SpecReporter extends Transform{
 case'test:stdout':
 returndata.message;
 case'test:diagnostic':
-return`${colors[type]}${this.#indent(data.nesting)}${symbols[type]}${data.message}${white}\n`;
+return`${this.#colors[type]}${this.#indent(data.nesting)}${symbols[type]}${data.message}${colors.white}\n`;
 case'test:coverage':
-returngetCoverageReport(this.#indent(data.nesting),data.summary,symbols['test:coverage'],blue,true);
+returngetCoverageReport(this.#indent(data.nesting),data.summary,symbols['test:coverage'],colors.blue,true);
 }
 }
 _transform({ type, data },encoding,callback){
@@ -143,7 +145,7 @@ class SpecReporter extends Transform{
 callback(null,'');
 return;
 }
-constresults=[`\n${colors['test:fail']}${symbols['test:fail']}failing tests:${white}\n`];
+constresults=[`\n${this.#colors['test:fail']}${symbols['test:fail']}failing tests:${colors.white}\n`];
 for(leti=0;i<this.#failedTests.length;i++){
 consttest=this.#failedTests[i];
 constrelPath=relative(this.#cwd,test.file);

test/fixtures/test-runner/output/arbitrary-output-colored.js

@@ -6,6 +6,7 @@ const fixtures = require('../../../common/fixtures');
 
 (asyncfunctionrun(){
 consttest=fixtures.path('test-runner/output/arbitrary-output-colored-1.js');
-awaitonce(spawn(process.execPath,['--test',test],{stdio: 'inherit',env: {FORCE_COLOR: 1}}),'exit');
-awaitonce(spawn(process.execPath,['--test','--test-reporter','tap',test],{stdio: 'inherit',env: {FORCE_COLOR: 1}}),'exit');
+constreset=fixtures.path('test-runner/output/reset-color-depth.js');
+awaitonce(spawn(process.execPath,['-r',reset,'--test',test],{stdio: 'inherit'}),'exit');
+awaitonce(spawn(process.execPath,['-r',reset,'--test','--test-reporter','tap',test],{stdio: 'inherit'}),'exit');
 })().then(common.mustCall());

test/fixtures/test-runner/output/reset-color-depth.js

@@ -0,0 +1,5 @@
+'use strict';
+
+// Make tests OS-independent by overriding stdio getColorDepth().
+process.stdout.getColorDepth=()=>8;
+process.stderr.getColorDepth=()=>8;