test: improve test-npm-install
authorSantiago Gimeno <santiago.gimeno@gmail.com>
Tue, 8 Mar 2016 23:59:59 +0000 (00:59 +0100)
committerMyles Borins <mborins@us.ibm.com>
Wed, 30 Mar 2016 20:12:14 +0000 (13:12 -0700)
Make npm install a dependency that is defined as a relative path, so it
avoids any network interaction.

PR-URL: https://github.com/nodejs/node/pull/5613
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Claudio Rodriguez <cjrodr@yahoo.com>
test/parallel/test-npm-install.js

index 5275b17..0c2e4df 100644 (file)
@@ -22,7 +22,11 @@ const args = [
   'install'
 ];
 
-const pkgContent = '{}';
+const pkgContent = JSON.stringify({
+  dependencies: {
+    'package-name': common.fixturesDir + '/packages/main'
+  }
+});
 
 const pkgPath = path.join(common.tmpDir, 'package.json');
 
@@ -35,6 +39,9 @@ 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.doesNotThrow(function() {
+    fs.accessSync(common.tmpDir + '/node_modules/package-name');
+  });
 }
 
 proc.on('exit', common.mustCall(handleExit));