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 e0b6bc7 commit d693dac
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 d693dac

Please sign in to comment.