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 <luigipinca@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
  • Loading branch information
Leko authored and gibfahn committed Dec 20, 2017
1 parent a44f085 commit 6e7ace2
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 @@ -55,7 +55,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 6e7ace2

Please sign in to comment.