benchmark: add validateStream to styleText bench

RafaelGSS · marco-ippolito · commit d84be843e39a · 2025-03-06T13:38:10.000+01:00
PR-URL: #56556 Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
diff --git a/benchmark/util/style-text.js b/benchmark/util/style-text.js
@@ -3,14 +3,16 @@
 const common = require('../common.js');
 
 const{styleText } = require('node:util');
+const assert = require('node:assert');
 
 const bench = common.createBenchmark(main,{
 messageType: ['string', 'number', 'boolean', 'invalid'],
 format: ['red', 'italic', 'invalid'],
+ validateStream: [1, 0],
 n: [1e3],
 });
 
-function main({messageType, format, n }){
+function main({messageType, format, validateStream, n }){
 let str;
 switch (messageType){
 case 'string':
@@ -29,8 +31,10 @@ function main({messageType, format, n }){
 
 bench.start();
 for (let i = 0; i < n; i++){
+ let colored = '';
 try{
- styleText(format, str);
+ colored = styleText(format, str,{validateStream });
+ assert.ok(colored); // Attempt to avoid dead-code elimination
 } catch{
 // eslint-disable no-empty
 }

-Original file line number
+Diff line change
 constcommon=require('../common.js');
 const{ styleText }=require('node:util');
 +constassert=require('node:assert');
 constbench=common.createBenchmark(main,{
 messageType: ['string','number','boolean','invalid'],
 format: ['red','italic','invalid'],
 +validateStream: [1,0],
 n: [1e3],
 });
 -functionmain({ messageType, format, n }){
 +functionmain({ messageType, format,validateStream,n }){
 letstr;
 switch(messageType){
 case'string':
 bench.start();
 for(leti=0;i<n;i++){
 +letcolored='';
 try{
 -styleText(format,str);
 +colored=styleText(format,str,{ validateStream });
 +assert.ok(colored);// Attempt to avoid dead-code elimination
 }catch{
 // eslint-disable no-empty
+}