refactor: update error handler i18n message identifier and JSONAPI er…

…ror code
adonisjs · Jan 22, 2024 · 4d3ec8b · 4d3ec8b
1 parent 676338d
commit 4d3ec8b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/errors.ts b/src/errors.ts
@@ -24,7 +24,7 @@ class AuthorizationException extends Exception {
   /**
    * Error identifier to lookup translation message
    */
-  identifier = 'access.denied'
+  identifier = 'errors.E_AUTHORIZATION_FAILURE'
 
   constructor(
     public response: AuthorizationResponse,
@@ -86,7 +86,7 @@ class AuthorizationException extends Exception {
         ctx.response.status(status).send({
           errors: [
             {
-              code: this.identifier,
+              code: this.code,
               title: message,
             },
           ],

diff --git a/tests/authorization_exception.spec.ts b/tests/authorization_exception.spec.ts
@@ -42,7 +42,7 @@ test.group('AuthorizationException', () => {
                 async load() {
                   return {
                     en: {
-                      'access.denied': 'Access denied from translations',
+                      'errors.E_AUTHORIZATION_FAILURE': 'Access denied from translations',
                     },
                   }
                 },
@@ -74,7 +74,7 @@ test.group('AuthorizationException', () => {
                 async load() {
                   return {
                     en: {
-                      'access.denied': 'Access denied from translations',
+                      'errors.E_AUTHORIZATION_FAILURE': 'Access denied from translations',
                       'errors.not_found': 'Page not found',
                     },
                   }
@@ -116,7 +116,7 @@ test.group('AuthorizationException', () => {
 
     await exception.handle(exception, ctx)
     assert.deepEqual(ctx.response.getBody(), {
-      errors: [{ code: 'access.denied', title: 'Access denied' }],
+      errors: [{ code: 'E_AUTHORIZATION_FAILURE', title: 'Access denied' }],
     })
     assert.equal(ctx.response.getStatus(), 403)
   })