Commit f9f8e4e
test: refactor test-preload
* assert.equal() -> assert.strictEqual()
* replace template string with a string; no variable substitution or
concatenation or anything like that
PR-URL: #9803
Reviewed-By: Prince John Wesley <princejohnwesley@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent e7c4dfb commit f9f8e4e
1 file changed
Lines changed: 5 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -29,7 +29,7 @@ child_process.exec(nodeBinary + ' ' | |||
| 29 | 29 | + fixtureB, | |
| 30 | 30 | function(err, stdout, stderr) { | |
| 31 | 31 | if (err) throw err; | |
| 32 | - assert.equal(stdout, 'A\nB\n'); | ||
| 32 | + assert.strictEqual(stdout, 'A\nB\n'); | ||
| 33 | 33 | }); | |
| 34 | 34 | ||
| 35 | 35 | // test preloading multiple modules works | |
@@ -38,7 +38,7 @@ child_process.exec(nodeBinary + ' ' | |||
| 38 | 38 | + fixtureC, | |
| 39 | 39 | function(err, stdout, stderr) { | |
| 40 | 40 | if (err) throw err; | |
| 41 | - assert.equal(stdout, 'A\nB\nC\n'); | ||
| 41 | + assert.strictEqual(stdout, 'A\nB\nC\n'); | ||
| 42 | 42 | }); | |
| 43 | 43 | ||
| 44 | 44 | // test that preloading a throwing module aborts | |
@@ -47,7 +47,7 @@ child_process.exec(nodeBinary + ' ' | |||
| 47 | 47 | + fixtureB, | |
| 48 | 48 | function(err, stdout, stderr) { | |
| 49 | 49 | if (err) { | |
| 50 | - assert.equal(stdout, 'A\n'); | ||
| 50 | + assert.strictEqual(stdout, 'A\n'); | ||
| 51 | 51 | } else { | |
| 52 | 52 | throw new Error('Preload should have failed'); | |
| 53 | 53 | } | |
@@ -59,7 +59,7 @@ child_process.exec(nodeBinary + ' ' | |||
| 59 | 59 | + '-e "console.log(\'hello\');"', | |
| 60 | 60 | function(err, stdout, stderr) { | |
| 61 | 61 | if (err) throw err; | |
| 62 | - assert.equal(stdout, 'A\nhello\n'); | ||
| 62 | + assert.strictEqual(stdout, 'A\nhello\n'); | ||
| 63 | 63 | }); | |
| 64 | 64 | ||
| 65 | 65 | // test that preload placement at other points in the cmdline | |
@@ -70,7 +70,7 @@ child_process.exec(nodeBinary + ' ' | |||
| 70 | 70 | + preloadOption([fixtureA, fixtureB]), | |
| 71 | 71 | function(err, stdout, stderr) { | |
| 72 | 72 | if (err) throw err; | |
| 73 | - assert.equal(stdout, 'A\nB\nhello\n'); | ||
| 73 | + assert.strictEqual(stdout, 'A\nB\nhello\n'); | ||
| 74 | 74 | }); | |
| 75 | 75 | ||
| 76 | 76 | child_process.exec(nodeBinary + ' ' | |
0 commit comments