test: improve debug output in trace-events test

test-trace-events-fs-sync is swallowing useful information when it fails. This change results in more information being displayed. PR-URL: #21120 Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Lance Ball <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>

Author: Trott

test/parallel/test-trace-events-fs-sync.js

@@ -3,6 +3,7 @@ const common = require('../common');
 constassert=require('assert');
 constcp=require('child_process');
 constfs=require('fs');
+constutil=require('util');
 
 if(!common.isMainThread)
 common.skip('process.chdir is not available in Workers');
@@ -121,7 +122,8 @@ for (const tr in tests){
 constproc=cp.spawnSync(process.execPath,
 ['--trace-events-enabled',
 '--trace-event-categories','node.fs.sync',
-'-e',tests[tr]]);
+'-e',tests[tr]],
+{encoding: 'utf8'});
 // Some AIX versions don't support futimes or utimes, so skip.
 if(common.isAIX&&proc.status!==0&&tr==='fs.sync.futimes'){
 continue;
@@ -131,7 +133,7 @@ for (const tr in tests){
 }
 
 // Make sure the operation is successful.
-assert.strictEqual(proc.status,0,tr+': '+proc.stderr);
+assert.strictEqual(proc.status,0,`${tr}:\n${util.inspect(proc)}`);
 
 // Confirm that trace log file is created.
 assert(common.fileExists(traceFile));