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>
This commit is contained in:
parent
25eedf02a8
commit
fbe0b444ef
@ -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));
|
||||
|
Loading…
x
Reference in New Issue
Block a user