fix(utils): statPath: do not throw on EACCES (#506)

Author: bcmarinacci

packages/mongodb-memory-server-core/src/util/__tests__/utils.test.ts

@@ -31,7 +31,7 @@ describe('utils', () => {
   });
 
   describe('statPath', () => {
-    it('should throw an error if code is not "ENOENT"', async () => {
+    it('should throw an error if code is not "ENOENT" or "EACCES"', async () => {
       const retError = new Error();
       // @ts-expect-error because there is no FSError, or an error with an "code" property - but still being used
       retError.code = 'EPERM';
@@ -97,7 +97,20 @@ describe('utils', () => {
       ).resolves.toBeUndefined();
     });
 
-    it('should throw an error if error is not "ENOENT"', async () => {
+    it('should return "undefined" if "EACCES"', async () => {
+      const retError = new Error();
+      // @ts-expect-error because there is no FSError, or an error with an "code" property - but still being used
+      retError.code = 'EACCES';
+      jest.spyOn(fspromises, 'readFile').mockRejectedValueOnce(retError);
+
+      await expect(
+        utils.tryReleaseFile('/some/path', () => {
+          fail('This Function should never run');
+        })
+      ).resolves.toBeUndefined();
+    });
+
+    it('should throw an error if error is not "ENOENT" or "EACCES"', async () => {
       const retError = new Error();
       // @ts-expect-error because there is no FSError, or an error with an "code" property - but still being used
       retError.code = 'EPERM';

packages/mongodb-memory-server-core/src/util/utils.ts

@@ -145,15 +145,16 @@ export function authDefault(opts: AutomaticAuth): Required<AutomaticAuth> {
 }
 
 /**
- * Run "fs.promises.stat", but return "undefined" if error is "ENOENT"
+ * Run "fs.promises.stat", but return "undefined" if error is "ENOENT" or "EACCES"
  * follows symlinks
  * @param path The Path to Stat
- * @throws if the error is not "ENOENT"
+ * @throws if the error is not "ENOENT" or "EACCES"
  */
 export async function statPath(path: string): Promise<Stats | undefined> {
   return fspromises.stat(path).catch((err) => {
-    if (err.code === 'ENOENT') {
-      return undefined; // catch the error if the directory dosnt exist, without throwing an error
+    // catch the error if the directory doesn't exist or permission is denied, without throwing an error
+    if (['ENOENT', 'EACCES'].includes(err.code)) {
+      return undefined;
     }
 
     throw err;
@@ -184,7 +185,7 @@ export async function tryReleaseFile(
 
     return parser(output.toString());
   } catch (err) {
-    if (err.code !== 'ENOENT') {
+    if (!['ENOENT', 'EACCES'].includes(err.code)) {
       throw err;
     }