From f225459496ec46f14e48f06444ac38b14948a8ec Mon Sep 17 00:00:00 2001 From: Santiago Gimeno Date: Wed, 9 Mar 2016 00:59:59 +0100 Subject: [PATCH] test: improve test-npm-install 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 Reviewed-By: Claudio Rodriguez --- test/parallel/test-npm-install.js | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/test/parallel/test-npm-install.js b/test/parallel/test-npm-install.js index 5275b17b39ebb7..0c2e4df52f9863 100644 --- a/test/parallel/test-npm-install.js +++ b/test/parallel/test-npm-install.js @@ -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));