Commit b14d7b3
test: improve error messages in test-npm-install
PR-URL: #11027
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>1 parent 87488ba commit b14d7b3
1 file changed
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -49,8 +49,8 @@ const proc = spawn(process.execPath, args, { | |||
| 49 | 49 | }); | |
| 50 | 50 | ||
| 51 | 51 | function handleExit(code, signalCode) { | |
| 52 | - assert.equal(code, 0, 'npm install should run without an error'); | ||
| 53 | - assert.ok(signalCode === null, 'signalCode should be null'); | ||
| 52 | + assert.strictEqual(code, 0, `npm install got error code ${code}`); | ||
| 53 | + assert.strictEqual(signalCode, null, `unexpected signal: ${signalCode}`); | ||
| 54 | 54 | assert.doesNotThrow(function() { | |
| 55 | 55 | fs.accessSync(installDir + '/node_modules/package-name'); | |
| 56 | 56 | }); | |
0 commit comments