feat: set default expose option for client- & server-error

tada5hi · Oct 7, 2023 · 4baf64d · 4baf64d
1 parent 0bd38ea
commit 4baf64d
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 3 deletions.
diff --git a/packages/http/build/utils.mjs b/packages/http/build/utils.mjs
@@ -20,6 +20,7 @@ export async function saveFile(content, filePath) {
 /**
  * Load template (.tpl) file from disk.
  * @param file
+ * @return Promise<string>
  */
 export async function loadTemplate(file) {
     const tplPath = path.isAbsolute(file) ?

diff --git a/packages/http/src/errors/base/client.ts b/packages/http/src/errors/base/client.ts
@@ -1,7 +1,10 @@
+import type { Input } from '../../types';
 import { HTTPError, isHTTPError } from './http';
 
 export class ClientError extends HTTPError {
-
+    constructor(...input: Input[]) {
+        super({ expose: true }, ...input);
+    }
 }
 
 export function isClientError(input: unknown): input is ClientError {

diff --git a/packages/http/src/errors/base/server.ts b/packages/http/src/errors/base/server.ts
@@ -1,7 +1,10 @@
+import type { Input } from '../../types';
 import { HTTPError, isHTTPError } from './http';
 
 export class ServerError extends HTTPError {
-
+    constructor(...input: Input[]) {
+        super({ expose: false }, ...input);
+    }
 }
 
 export function isServerError(error: unknown): error is ServerError {

diff --git a/packages/http/test/unit/module.spec.ts b/packages/http/test/unit/module.spec.ts
@@ -1,6 +1,6 @@
 // eslint-disable-next-line max-classes-per-file
 import {
-    HTTPError, InternalServerError, NotFoundError, isClientError, isHTTPError, isServerError,
+    ClientError, HTTPError, InternalServerError, NotFoundError, ServerError, isClientError, isHTTPError, isServerError,
 } from '../../src';
 
 describe('src/module.ts', () => {
@@ -25,13 +25,23 @@ describe('src/module.ts', () => {
         expect(error.statusCode).toEqual(500);
     });
 
+    it('should create client error with default props', () => {
+        const error = new ClientError();
+        expect(error.expose).toBeTruthy();
+    });
+
     it('should recognize client error', () => {
         const error = new NotFoundError();
         expect(isClientError(error)).toBeTruthy();
         expect(isServerError(error)).toBeFalsy();
         expect(isHTTPError(error)).toBeTruthy();
     });
 
+    it('should create server error with default props', () => {
+        const error = new ServerError();
+        expect(error.expose).toBeFalsy();
+    });
+
     it('should recognize server error', () => {
         const error = new InternalServerError();
         expect(isClientError(error)).toBeFalsy();