Skip to content

Commit 8b367c5

Browse files
edsadritaloacasas
authored andcommitted
test: improve test-child-process-fork-and-spawn
* use const instead of var for required modules * use assert.strictEqual instead of assert.equal * remove unnecessary process.nextTick PR-URL: #10273 Reviewed-By: Santiago Gimeno <[email protected]> Reviewed-By: Evan Lucas <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Minwoo Jung <[email protected]> Reviewed-By: Italo A. Casas <[email protected]>
1 parent 7cb9813 commit 8b367c5

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
'use strict';
22
const common = require('../common');
3-
var assert = require('assert');
4-
var spawn = require('child_process').spawn;
5-
var fork = require('child_process').fork;
3+
const assert = require('assert');
4+
const spawn = require('child_process').spawn;
5+
const fork = require('child_process').fork;
66

77
// Fork, then spawn. The spawned process should not hang.
88
switch (process.argv[2] || '') {
@@ -16,10 +16,9 @@ switch (process.argv[2] || '') {
1616
case 'spawn':
1717
break;
1818
default:
19-
assert(0);
19+
common.fail();
2020
}
2121

2222
function checkExit(statusCode) {
23-
assert.equal(statusCode, 0);
24-
process.nextTick(process.exit);
23+
assert.strictEqual(statusCode, 0);
2524
}

0 commit comments

Comments
 (0)