provided the final fix for error handling

Author: anujachakraborty

src/clients/AasRepositoryClient.ts

@@ -71,7 +71,6 @@ export class AasRepositoryClient {
             };
         } catch (err) {
             const customError = handleApiError(err);
-            console.log('Error:', customError);
             return { success: false, error: customError };
         }
     }
@@ -101,7 +100,6 @@ export class AasRepositoryClient {
             return { success: true, data: convertApiAasToCoreAas(result) };
         } catch (err) {
             const customError = handleApiError(err);
-            console.log('Error:', customError);
             return { success: false, error: customError };
         }
     }
@@ -133,7 +131,6 @@ export class AasRepositoryClient {
             return { success: true, data: result };
         } catch (err) {
             const customError = handleApiError(err);
-            console.log('Error:', customError);
             return { success: false, error: customError };
         }
     }
@@ -165,8 +162,6 @@ export class AasRepositoryClient {
             return { success: true, data: convertApiAasToCoreAas(result) };
         } catch (err) {
             const customError = handleApiError(err);
-            //console.log('Error:', customError);
-            console.log(`Error: field: ${customError.field}, message: ${customError.message}`);
             return { success: false, error: customError };
         }
     }
@@ -201,7 +196,6 @@ export class AasRepositoryClient {
             return { success: true, data: result };
         } catch (err) {
             const customError = handleApiError(err);
-            console.log('Error:', customError);
             return { success: false, error: customError };
         }
     }
@@ -236,7 +230,6 @@ export class AasRepositoryClient {
             };
         } catch (err) {
             const customError = handleApiError(err);
-            console.log('Error:', customError);
             return { success: false, error: customError };
         }
     }
@@ -271,7 +264,6 @@ export class AasRepositoryClient {
             return { success: true, data: result };
         } catch (err) {
             const customError = handleApiError(err);
-            console.log('Error:', customError);
             return { success: false, error: customError };
         }
     }
@@ -302,7 +294,6 @@ export class AasRepositoryClient {
             return { success: true, data: result };
         } catch (err) {
             const customError = handleApiError(err);
-            console.log('Error:', customError);
             return { success: false, error: customError };
         }
     }
@@ -334,7 +325,6 @@ export class AasRepositoryClient {
             return { success: true, data: result };
         } catch (err) {
             const customError = handleApiError(err);
-            console.log('Error:', customError);
             return { success: false, error: customError };
         }
     }
@@ -372,7 +362,6 @@ export class AasRepositoryClient {
             return { success: true, data: result };
         } catch (err) {
             const customError = handleApiError(err);
-            console.log('Error:', customError);
             return { success: false, error: customError };
         }
     }
@@ -417,7 +406,6 @@ export class AasRepositoryClient {
             };
         } catch (err) {
             const customError = handleApiError(err);
-            console.log('Error:', customError);
             return { success: false, error: customError };
         }
     }
@@ -452,7 +440,6 @@ export class AasRepositoryClient {
             return { success: true, data: convertApiReferenceToCoreReference(result) };
         } catch (err) {
             const customError = handleApiError(err);
-            console.log('Error:', customError);
             return { success: false, error: customError };
         }
     }
@@ -488,7 +475,6 @@ export class AasRepositoryClient {
             return { success: true, data: result };
         } catch (err) {
             const customError = handleApiError(err);
-            console.log('Error:', customError);
             return { success: false, error: customError };
         }
     }

src/integration-tests/aasRepo.integration.test.ts

@@ -44,21 +44,9 @@ describe('AAS Repository Integration Tests', () => {
         expect(response.success).toBe(false);
         if (!response.success) {
             expect(response.error).toBeDefined();
-
-            //console.log('Error:', response.error);
-            //console.log(`❌ Error fetching AAS with ID '${nonExistingId}':`, response.error.message || response.error);
-            
-            let errorMessage = "Unknown error occurred.";
-            if (response.error?.field) {
-                errorMessage = ` Missing required field: ${response.error.field} - ${response.error.message}`;
-            } else if (response.error?.message) {
-                errorMessage = `${response.error.message}`;
-            } 
-            console.log(
-                `❌ Failed to fetch AAS with AAS ID: ${nonExistingId}, Error: ${errorMessage}` );
-                       
-         }
-     });
+            console.log('Error:', response.error);
+        }
+    });
 
     test('should fetch all Asset Administration Shells', async () => {
         const response = await client.getAllAssetAdministrationShells({

src/lib/errorHandler.ts

@@ -1,11 +1,8 @@
 import { RequiredError } from '../generated';
 
 export function handleApiError(err: unknown): RequiredError {
-    //export function handleApiError(err: unknown, fields: Record<string, unknown>): RequiredError {, defaultField: string
-
     let message = 'Unknown error';
     let statusCode = 'N/A';
-    //let field = defaultField;
     let field = 'N/A';
 
     if (err instanceof RequiredError) {
@@ -22,5 +19,4 @@ export function handleApiError(err: unknown): RequiredError {
     }
 
     return new RequiredError(field, ` message: ${message}`);
-    //return new RequiredError(field, message);
 }