test: update test-stdout-to-file

* changed vars to const * changed assert.equal to assert.strictEqual * added a common.mustCall in the childProcess.exec callback * replaced 2 console.log strings with template strings for readability * had to break up line 9 because it was causing a line max length (80) listing err PR-URL: #9939 Reviewed-By: Prince John Wesley <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: scalkpdev

test/parallel/test-stdout-to-file.js

@@ -1,13 +1,14 @@
 'use strict';
-varcommon=require('../common');
-varassert=require('assert');
-varpath=require('path');
-varchildProcess=require('child_process');
-varfs=require('fs');
+constcommon=require('../common');
+constassert=require('assert');
+constpath=require('path');
+constchildProcess=require('child_process');
+constfs=require('fs');
 
-varscriptString=path.join(common.fixturesDir,'print-chars.js');
-varscriptBuffer=path.join(common.fixturesDir,'print-chars-from-buffer.js');
-vartmpFile=path.join(common.tmpDir,'stdout.txt');
+constscriptString=path.join(common.fixturesDir,'print-chars.js');
+constscriptBuffer=path.join(common.fixturesDir,
+'print-chars-from-buffer.js');
+consttmpFile=path.join(common.tmpDir,'stdout.txt');
 
 common.refreshTmpDir();
 
@@ -24,22 +25,22 @@ function test(size, useBuffer, cb){
 fs.unlinkSync(tmpFile);
 }catch(e){}
 
-console.log(size+' chars to '+tmpFile+'...');
+console.log(`${size}chars to ${tmpFile}...`);
 
-childProcess.exec(cmd,function(err){
+childProcess.exec(cmd,common.mustCall(function(err){
 if(err)throwerr;
 
 console.log('done!');
 
 varstat=fs.statSync(tmpFile);
 
-console.log(tmpFile+' has '+stat.size+' bytes');
+console.log(`${tmpFile}has ${stat.size}bytes`);
 
-assert.equal(size,stat.size);
+assert.strictEqual(size,stat.size);
 fs.unlinkSync(tmpFile);
 
 cb();
-});
+}));
 }
 
 test(1024*1024,false,common.mustCall(function(){