Commit 928aecc
test: replace fs.accessSync with fs.existsSync
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>1 parent 49402b1 commit 928aecc
1 file changed
Lines changed: 1 addition & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -57,7 +57,5 @@ function handleExit(error, stdout, stderr) { | |||
| 57 | 57 | ||
| 58 | 58 | assert.strictEqual(code, 0, `npm install got error code ${code}`); | |
| 59 | 59 | assert.strictEqual(signalCode, null, `unexpected signal: ${signalCode}`); | |
| 60 | - assert.doesNotThrow(function() { | ||
| 61 | - fs.accessSync(`${installDir}/node_modules/package-name`); | ||
| 62 | - }); | ||
| 60 | + assert(fs.existsSync(`${installDir}/node_modules/package-name`)); | ||
| 63 | 61 | } | |
0 commit comments