test: refactor test-vm-syntax-error-stderr.js

use common.fail instead of assert(false) change var to let or const as appropriate PR-URL: #9900 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>

Author: Jay Brownlee

test/parallel/test-vm-syntax-error-stderr.js

@@ -1,22 +1,22 @@
 'use strict';
-varcommon=require('../common');
-varassert=require('assert');
-varpath=require('path');
-varchild_process=require('child_process');
+constcommon=require('../common');
+constassert=require('assert');
+constpath=require('path');
+constchild_process=require('child_process');
 
-varwrong_script=path.join(common.fixturesDir,'cert.pem');
+constwrong_script=path.join(common.fixturesDir,'cert.pem');
 
-varp=child_process.spawn(process.execPath,[
+constp=child_process.spawn(process.execPath,[
 '-e',
 'require(process.argv[1]);',
 wrong_script
 ]);
 
 p.stdout.on('data',function(data){
-assert(false,'Unexpected stdout data: '+data);
+common.fail('Unexpected stdout data: '+data);
 });
 
-varoutput='';
+letoutput='';
 
 p.stderr.on('data',function(data){
 output+=data;