test: improve error messages in test-npm-install

gonenduk · Italo A. Casas · commit 8d2a9138fc5f · 2017-02-14T11:51:48.000-05:00
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>
diff --git a/test/parallel/test-npm-install.js b/test/parallel/test-npm-install.js
@@ -53,8 +53,8 @@ const proc = spawn(process.execPath, args, {
 });
 
 function handleExit(code, signalCode) {
-  assert.strictEqual(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');
   });
