Skip to content

Commit

Permalink
test: replace fs.accessSync with fs.existsSync
Browse files Browse the repository at this point in the history
PR-URL: #17446
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Anatoli Papirovski <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
  • Loading branch information
Leko authored and MylesBorins committed Dec 12, 2017
1 parent 49402b1 commit 928aecc
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions test/parallel/test-npm-install.js
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,5 @@ function handleExit(error, stdout, stderr) {

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`);
});
assert(fs.existsSync(`${installDir}/node_modules/package-name`));
}

0 comments on commit 928aecc

Please sign in to comment.