1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

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

Browse Source 
use common.fail instead of assert(false)
change var to let or const as appropriate

PR-URL: https://github.com/nodejs/node/pull/9900
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
This commit is contained in:
Jay Brownlee 2016-12-01 10:06:21 -06:00 committed by James M Snell
parent 5e5ec12c10
commit fa228547d3
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
test/parallel/test-vm-syntax-error-stderr.js
View File

@ -1,22 +1,22 @@
'use strict';
var common = require('../common');
var assert = require('assert');
var path = require('path');
var child_process = require('child_process');
const common = require('../common');
const assert = require('assert');
const path = require('path');
const child_process = require('child_process');
var wrong_script = path.join(common.fixturesDir, 'cert.pem');
const wrong_script = path.join(common.fixturesDir, 'cert.pem');
var p = child_process.spawn(process.execPath, [
const p = 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);
});
var output = '';
let output = '';
p.stderr.on('data', function(data) {
output += data;