test: replace fs.accessSync with fs.existsSync

Leko · gibfahn · commit d693dacc78ae · 2017-12-20T00:51:27.000Z
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>
diff --git a/test/parallel/test-npm-install.js b/test/parallel/test-npm-install.js
@@ -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`));
 }

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`	`}`