Merge pull request #47 from ApplauseOSS/fix-auto-api-error-logging

Fix Auto-API Error Logging
ApplauseOSS · Aug 2, 2024 · 007baba · 007baba
2 parents e341e41 + 0ab3acc
commit 007baba
Show file tree

Hide file tree

Showing 7 changed files with 15 additions and 21 deletions.
diff --git a/dist/index.cjs b/dist/index.cjs
diff --git a/dist/index.cjs.map b/dist/index.cjs.map
diff --git a/dist/index.min.js b/dist/index.min.js
diff --git a/dist/index.min.js.map b/dist/index.min.js.map
diff --git a/dist/index.mjs b/dist/index.mjs
diff --git a/dist/index.mjs.map b/dist/index.mjs.map
diff --git a/src/auto-api/auto-api.ts b/src/auto-api/auto-api.ts
@@ -1,4 +1,4 @@
-import axios, { AxiosInstance, AxiosResponse } from 'axios';
+import axios, { AxiosError, AxiosInstance, AxiosResponse } from 'axios';
 import {
   CreateTestCaseResultDto,
   CreateTestCaseResultResponseDto,
@@ -39,15 +39,13 @@ export class AutoApi {
       function (response: AxiosResponse<any, any>) {
         return response;
       },
-      function (error) {
+      function (error: AxiosError) {
         // log and rethrow
         const errText =
           // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-          error.data !== undefined
-            ? // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-              error.data
-            : // eslint-disable-next-line @typescript-eslint/no-unsafe-member-access
-              `error-code [${error.response.status}] with error [${error.response.statusText}]`;
+          error.response?.data !== undefined
+            ? JSON.stringify(error.response.data)
+            : `error-code [${error.response?.status}] with error [${error.response?.statusText}]`;
         console.error(`Auto-Api returned ${errText}`);
         return Promise.reject(error);
       }