test: replace forEach with for of in file test-trace-events-net.js

Ianna83 · richardlau · commit c3c2b2b041e8 · 2024-03-25T18:14:35.000Z
PR-URL: #50789 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
diff --git a/test/parallel/test-trace-events-net.js b/test/parallel/test-trace-events-net.js
@@ -32,12 +32,11 @@ proc.once('exit', common.mustCall(() =>{
 const traces = JSON.parse(data.toString()).traceEvents;
 assert(traces.length > 0);
 let count = 0;
- traces.forEach((trace) =>{
- if (trace.cat === 'node,node.net,node.net.native' &&
- trace.name === 'connect'){
+ for (const trace of traces){
+ if (trace.cat === 'node,node.net,node.net.native' && trace.name === 'connect'){
 count++;
 }
- });
+ }
 // Two begin, two end
 assert.strictEqual(count, 4);
 }));

-Original file line number
+Diff line change
 consttraces=JSON.parse(data.toString()).traceEvents;
 assert(traces.length>0);
 letcount=0;
 -traces.forEach((trace)=>{
 -if(trace.cat==='node,node.net,node.net.native'&&
 -trace.name==='connect'){
 +for(consttraceoftraces){
 +if(trace.cat==='node,node.net,node.net.native'&&trace.name==='connect'){
 count++;
+}
 -});
 +}
 // Two begin, two end
 assert.strictEqual(count,4);
 }));