test_runner: replace ansi clear with ansi reset

pmarchini · marco-ippolito · commit 5830931c5513 · 2024-11-16T12:29:24.000+01:00
PR-URL: #55013 Fixes: #55009 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Moshe Atlow <moshe@atlow.co.il> Reviewed-By: Chemi Atlow <chemi@atlow.co.il> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
diff --git a/lib/internal/test_runner/reporter/dot.js b/lib/internal/test_runner/reporter/dot.js
@@ -12,10 +12,10 @@ module.exports = async function* dot(source){
 const failedTests = [];
 for await (const{type, data } of source){
 if (type === 'test:pass'){
- yield `${colors.green}.${colors.clear}`;
+ yield `${colors.green}.${colors.reset}`;
 }
 if (type === 'test:fail'){
- yield `${colors.red}X${colors.clear}`;
+ yield `${colors.red}X${colors.reset}`;
 ArrayPrototypePush(failedTests, data);
 }
 if ((type === 'test:fail' || type === 'test:pass') && ++count === columns){
diff --git a/lib/internal/util/colors.js b/lib/internal/util/colors.js
@@ -13,6 +13,7 @@ module.exports ={
 red: '',
 gray: '',
 clear: '',
+ reset: '',
 hasColors: false,
 shouldColorize(stream){
 if (process.env.FORCE_COLOR !== undefined){
@@ -32,6 +33,7 @@ module.exports ={
 module.exports.red = hasColors ? '\u001b[31m' : '';
 module.exports.gray = hasColors ? '\u001b[90m' : '';
 module.exports.clear = hasColors ? '\u001bc' : '';
+ module.exports.reset = hasColors ? '\u001b[0m' : '';
 module.exports.hasColors = hasColors;
 }
 },
diff --git a/test/fixtures/test-runner/output/dot_output_custom_columns.snapshot b/test/fixtures/test-runner/output/dot_output_custom_columns.snapshot
@@ -1,3 +1,3 @@
-[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c
-[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c
-[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c[32m.c
+[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m
+[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m
+[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m[32m.[0m

Original file line number	Diff line number	Diff line change
`@@ -12,10 +12,10 @@ module.exports = async function* dot(source){`
`12`	`12`	`constfailedTests=[];`
`13`	`13`	`forawait(const{ type, data }ofsource){`
`14`	`14`	`if(type==='test:pass'){`
`15`		-yield`${colors.green}.${colors.clear}`;
	`15`	+yield`${colors.green}.${colors.reset}`;
`16`	`16`	`}`
`17`	`17`	`if(type==='test:fail'){`
`18`		-yield`${colors.red}X${colors.clear}`;
	`18`	+yield`${colors.red}X${colors.reset}`;
`19`	`19`	`ArrayPrototypePush(failedTests,data);`
`20`	`20`	`}`
`21`	`21`	`if((type==='test:fail'\|\|type==='test:pass')&&++count===columns){`
-Original file line number
+Diff line change
 red: '',
 gray: '',
 clear: '',
 +reset: '',
 hasColors: false,
 shouldColorize(stream){
 if(process.env.FORCE_COLOR!==undefined){
 module.exports.red=hasColors ? '\u001b[31m' : '';
 module.exports.gray=hasColors ? '\u001b[90m' : '';
 module.exports.clear=hasColors ? '\u001bc' : '';
 +module.exports.reset=hasColors ? '\u001b[0m' : '';
 module.exports.hasColors=hasColors;
+}
 },