chore: added error handling for invalid patterns

MatrixAI · Aug 20, 2024 · d1b05b9 · d1b05b9
1 parent e0f062d
commit d1b05b9
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 9 deletions.
diff --git a/src/client/errors.ts b/src/client/errors.ts
@@ -18,8 +18,8 @@ class ErrorClientAuthDenied<T> extends ErrorClient<T> {
   exitCode = sysexits.NOPERM;
 }
 
-class ErrorClientFileReadFailed<T> extends ErrorClient<T> {
-  static description = 'Failed to read file or directory';
+class ErrorClientFSReadFailed<T> extends ErrorClient<T> {
+  static description = 'Failed to read from filesystem';
   exitCode = sysexits.IOERR;
 }
 
@@ -50,7 +50,7 @@ export {
   ErrorClientAuthMissing,
   ErrorClientAuthFormat,
   ErrorClientAuthDenied,
-  ErrorClientFileReadFailed,
+  ErrorClientFSReadFailed,
   ErrorClientService,
   ErrorClientServiceRunning,
   ErrorClientServiceNotRunning,

diff --git a/src/client/handlers/VaultsSecretsGetFileTree.ts b/src/client/handlers/VaultsSecretsGetFileTree.ts
@@ -54,14 +54,14 @@ class VaultsSecretsGetFileTree extends ServerHandler<
                 stat: input.yieldStats ? generateStats(stat) : undefined,
               };
             } catch (e) {
-              throw new clientErrors.ErrorClientFileReadFailed(
+              throw new clientErrors.ErrorClientFSReadFailed(
                 `Failed to read file: ${file}`,
                 { cause: e },
               );
             }
           }
         } catch (e) {
-          throw new clientErrors.ErrorClientFileReadFailed(
+          throw new clientErrors.ErrorClientFSReadFailed(
             `Failed to read directory: ${input.pattern}`,
             { cause: e },
           );

diff --git a/tests/client/handlers/vaults.test.ts b/tests/client/handlers/vaults.test.ts
@@ -1589,19 +1589,18 @@ describe('vaultsSecretsNewDir and vaultsSecretsList', () => {
     });
     expect(addResponse.success).toBeTruthy();
 
-    expect(async () => {
+    await expect(async () => {
       const files = await rpcClient.methods.vaultsSecretsGetFileTree({
         nameOrId: vaultsIdEncoded,
         pattern: 'doesntExist',
         yieldStats: false,
       });
       try {
         for await (const _ of files); // Consume values
-      }
-      catch (e) {
+      } catch (e) {
         throw e.cause;
       }
-    }).rejects.toThrow(clientErrors.ErrorClientFileReadFailed);
+    }).rejects.toThrow(clientErrors.ErrorClientFSReadFailed);
 
     // List secrets with names of directories
     const secrets = await rpcClient.methods.vaultsSecretsGetFileTree({