diff options
author | Santiago Gimeno <santiago.gimeno@gmail.com> | 2016-03-09 00:59:59 +0100 |
---|---|---|
committer | James M Snell <jasnell@gmail.com> | 2016-03-14 08:25:55 -0700 |
commit | fbe0b444ef87a49da537fc165dde62172ce1b721 (patch) | |
tree | 0ccaf409d0d2a27a1a08f092632622adb630e62f /test/parallel/test-npm-install.js | |
parent | 25eedf02a85bf3cac25f8f9b2c38d91969f52fcd (diff) | |
download | android-node-v8-fbe0b444ef87a49da537fc165dde62172ce1b721.tar.gz android-node-v8-fbe0b444ef87a49da537fc165dde62172ce1b721.tar.bz2 android-node-v8-fbe0b444ef87a49da537fc165dde62172ce1b721.zip |
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 <jasnell@gmail.com>
Reviewed-By: Claudio Rodriguez <cjrodr@yahoo.com>
Diffstat (limited to 'test/parallel/test-npm-install.js')
-rw-r--r-- | test/parallel/test-npm-install.js | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/test/parallel/test-npm-install.js b/test/parallel/test-npm-install.js index 5275b17b39..0c2e4df52f 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)); |