Export createHttpError()

tkrotoff · Aug 1, 2019 · a534716 · a534716
1 parent 8a13801
commit a534716
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 4 deletions.
diff --git a/src/Http.test.ts b/src/Http.test.ts
@@ -405,3 +405,10 @@ describe('checkStatus()', () => {
     }
   });
 });
+
+test('createHttpError()', () => {
+  const error = Http.createHttpError('Bad Request', HttpStatus._400_BadRequest, { error: 400 });
+  expect(error).toEqual(new HttpError('Bad Request'));
+  expect(error.status).toEqual(HttpStatus._400_BadRequest);
+  expect(error.response).toEqual({ error: 400 });
+});
diff --git a/src/Http.ts b/src/Http.ts
@@ -1,4 +1,5 @@
 import { HttpError } from './HttpError';
+import { HttpStatus } from './HttpStatus';
 
 const JSON_MIME_TYPE = 'application/json';
 
@@ -16,6 +17,17 @@ export async function parseResponseBody(response: Response) {
   return response.text();
 }
 
+export function createHttpError(
+  statusText: string,
+  status: HttpStatus,
+  parsedResponseBody: unknown
+) {
+  const error = new HttpError(statusText);
+  error.status = status;
+  error.response = parsedResponseBody;
+  return error;
+}
+
 // See [Handling HTTP error statuses](https://github.com/github/fetch/blob/v3.0.0/README.md#handling-http-error-statuses)
 // Exported for testing purpose only
 export function checkStatus(response: Response, parsedResponseBody: unknown) {
@@ -46,10 +58,7 @@ export function checkStatus(response: Response, parsedResponseBody: unknown) {
   // }
 
   if (!response.ok) {
-    const error = new HttpError(response.statusText);
-    error.status = response.status;
-    error.response = parsedResponseBody;
-    throw error;
+    throw createHttpError(response.statusText, response.status, parsedResponseBody);
   }
 }