Skip to content

Commit ad27555

Browse files
NigelKibodeauxMylesBorins
authored andcommitted
test: use assert.strictEqual in test-cli-eval
PR-URL: #9919 Reviewed-By: Teddy Katz <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
1 parent cffd51e commit ad27555

File tree

1 file changed

+18
-17
lines changed

1 file changed

+18
-17
lines changed

‎test/parallel/test-cli-eval.js‎

Lines changed: 18 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -19,15 +19,15 @@ var filename = __filename.replace(/\\/g, '/');
1919
// assert that nothing is written to stdout
2020
child.exec(nodejs+' --eval 42',
2121
function(err,stdout,stderr){
22-
assert.equal(stdout,'');
23-
assert.equal(stderr,'');
22+
assert.strictEqual(stdout,'');
23+
assert.strictEqual(stderr,'');
2424
});
2525

2626
// assert that "42\n" is written to stderr
2727
child.exec(nodejs+' --eval "console.error(42)"',
2828
function(err,stdout,stderr){
29-
assert.equal(stdout,'');
30-
assert.equal(stderr,'42\n');
29+
assert.strictEqual(stdout,'');
30+
assert.strictEqual(stderr,'42\n');
3131
});
3232

3333
// assert that the expected output is written to stdout
@@ -36,27 +36,27 @@ child.exec(nodejs + ' --eval "console.error(42)"',
3636

3737
child.exec(cmd+'42',
3838
function(err,stdout,stderr){
39-
assert.equal(stdout,'42\n');
40-
assert.equal(stderr,'');
39+
assert.strictEqual(stdout,'42\n');
40+
assert.strictEqual(stderr,'');
4141
});
4242

4343
child.exec(cmd+"'[]'",common.mustCall(
4444
function(err,stdout,stderr){
45-
assert.equal(stdout,'[]\n');
46-
assert.equal(stderr,'');
45+
assert.strictEqual(stdout,'[]\n');
46+
assert.strictEqual(stderr,'');
4747
}));
4848
});
4949

5050
// assert that module loading works
5151
child.exec(nodejs+' --eval "require(\''+filename+'\')"',
5252
function(status,stdout,stderr){
53-
assert.equal(status.code,42);
53+
assert.strictEqual(status.code,42);
5454
});
5555

5656
// module path resolve bug, regression test
5757
child.exec(nodejs+' --eval "require(\'./test/parallel/test-cli-eval.js\')"',
5858
function(status,stdout,stderr){
59-
assert.equal(status.code,42);
59+
assert.strictEqual(status.code,42);
6060
});
6161

6262
// Missing argument should not crash
@@ -67,26 +67,27 @@ child.exec(nodejs + ' -e', common.mustCall(function(status, stdout, stderr){
6767

6868
// empty program should do nothing
6969
child.exec(nodejs+' -e ""',function(status,stdout,stderr){
70-
assert.equal(stdout,'');
71-
assert.equal(stderr,'');
70+
assert.strictEqual(stdout,'');
71+
assert.strictEqual(stderr,'');
7272
});
7373

7474
// "\\-42" should be interpreted as an escaped expression, not a switch
7575
child.exec(nodejs+' -p "\\-42"',
7676
function(err,stdout,stderr){
77-
assert.equal(stdout,'-42\n');
78-
assert.equal(stderr,'');
77+
assert.strictEqual(stdout,'-42\n');
78+
assert.strictEqual(stderr,'');
7979
});
8080

8181
child.exec(nodejs+' --use-strict -p process.execArgv',
8282
function(status,stdout,stderr){
83-
assert.equal(stdout,"[ '--use-strict', '-p', 'process.execArgv' ]\n");
83+
assert.strictEqual(stdout,
84+
"[ '--use-strict', '-p', 'process.execArgv' ]\n");
8485
});
8586

8687
// Regression test for https://github.com/nodejs/node/issues/3574
8788
constemptyFile=path.join(common.fixturesDir,'empty.js');
8889
child.exec(nodejs+` -e 'require("child_process").fork("${emptyFile}")'`,
8990
function(status,stdout,stderr){
90-
assert.equal(stdout,'');
91-
assert.equal(stderr,'');
91+
assert.strictEqual(stdout,'');
92+
assert.strictEqual(stderr,'');
9293
});

0 commit comments

Comments
(0)