adds back quotes around named values in error messages

digitalcredentials · Dec 13, 2023 · 7b0b213 · 7b0b213
1 parent 79cbf41
commit 7b0b213
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/src/credential-status-manager-base.ts b/src/credential-status-manager-base.ts
@@ -459,7 +459,7 @@ export abstract class BaseCredentialStatusManager {
         throw new BadRequestError({
           message:
             '"credentialStatus" must be one of the following values: ' +
-            `${Object.values(CredentialState).join(', ')}.`
+            `${Object.values(CredentialState).map(s => `"${s}"`).join(', ')}.`
         });
     }
     const statusCredentialUrlBase = this.getStatusCredentialUrlBase();

diff --git a/src/credential-status-manager-github.ts b/src/credential-status-manager-github.ts
@@ -88,7 +88,7 @@ export class GitHubCredentialStatusManager extends BaseCredentialStatusManager {
         message:
           'You have neglected to set the following required options for the ' +
           'GitHub credential status manager: ' +
-          `${missingOptions.join(', ')}.`
+          `${missingOptions.map(o => `"${o}"`).join(', ')}.`
       });
     }
 

diff --git a/src/credential-status-manager-gitlab.ts b/src/credential-status-manager-gitlab.ts
@@ -153,7 +153,7 @@ export class GitLabCredentialStatusManager extends BaseCredentialStatusManager {
         message:
           'You have neglected to set the following required options for the ' +
           'GitLab credential status manager: ' +
-          `${missingOptions.join(', ')}.`
+          `${missingOptions.map(o => `"${o}"`).join(', ')}.`
       });
     }
 

diff --git a/src/credential-status-manager-index.ts b/src/credential-status-manager-index.ts
@@ -88,7 +88,7 @@ export async function createStatusManager(options: CredentialStatusManagerOption
       throw new BadRequestError({
         message:
           '"service" must be one of the following values: ' +
-          `${Object.values(CredentialStatusManagerService).join(', ')}.`
+          `${Object.values(CredentialStatusManagerService).map(s => `"${s}"`).join(', ')}.`
       });
   }
 

diff --git a/src/helpers.ts b/src/helpers.ts
@@ -105,7 +105,7 @@ export async function getSigningMaterial({
       throw new BadRequestError({
         message:
           '"didMethod" must be one of the following values: ' +
-          `${Object.values(DidMethod).join(', ')}.`
+          `${Object.values(DidMethod).map(m => `"${m}"`).join(', ')}.`
       });
   }
   const issuerDid = didDocument.id;