test: increase strictness for test-trace-event

Change test-trace-event such that it checks that all expected values are within the same trace object rather than scattered across multiple trace objects. PR-URL: #11065 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: Trott

test/parallel/test-trace-event.js

@@ -21,15 +21,19 @@ proc_no_categories.once('exit', common.mustCall(() =>{
 
 proc.once('exit',common.mustCall(()=>{
 assert(common.fileExists(FILE_NAME));
-fs.readFile(FILE_NAME,(err,data)=>{
+fs.readFile(FILE_NAME,common.mustCall((err,data)=>{
 consttraces=JSON.parse(data.toString()).traceEvents;
 assert(traces.length>0);
 // Values that should be present on all runs to approximate correctness.
-assert(traces.some((trace)=>{returntrace.pid===proc.pid;}));
-assert(traces.some((trace)=>{returntrace.cat==='v8';}));
 assert(traces.some((trace)=>{
-returntrace.name==='V8.ScriptCompiler';
+if(trace.pid!==proc.pid)
+returnfalse;
+if(trace.cat!=='v8')
+returnfalse;
+if(trace.name!=='V8.ScriptCompiler')
+returnfalse;
+returntrue;
 }));
-});
+}));
 }));
 }));