Skip to content

Commit

Permalink
test: improve error messages in test-npm-install
Browse files Browse the repository at this point in the history
PR-URL: #11027
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Jeremiah Senkpiel <[email protected]>
  • Loading branch information
gonenduk authored and MylesBorins committed Mar 9, 2017
1 parent 1db89d4 commit f24aa7e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions test/parallel/test-npm-install.js
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,8 @@ const proc = spawn(process.execPath, args, {
});

function handleExit(code, signalCode) {
assert.equal(code, 0, 'npm install should run without an error');
assert.ok(signalCode === null, 'signalCode should be null');
assert.strictEqual(code, 0, `npm install got error code ${code}`);
assert.strictEqual(signalCode, null, `unexpected signal: ${signalCode}`);
assert.doesNotThrow(function() {
fs.accessSync(installDir + '/node_modules/package-name');
});
Expand Down

0 comments on commit f24aa7e

Please sign in to comment.