Rename to error handler and fix issues

src/bearbyWallet/BearbyAccount.ts

@@ -11,8 +11,8 @@ import { IAccount } from '../account/IAccount';
 import { AddressInfo, web3 } from '@hicaru/bearby.js';
 import { postRequest } from '../massaStation/RequestHandler';
 import { IAccountSignOutput } from '../account/AccountSign';
-import { operationErrorMapping } from '../errors/utils/errorMapping';
-import { operationType } from 'src/utils/constants';
+import { errorHandler } from '../errors/utils/errorHandler';
+import { operationType } from '../utils/constants';
 
 export class BearbyAccount implements IAccount {
   private _providerName: string;
@@ -91,7 +91,7 @@ export class BearbyAccount implements IAccount {
         base58Encoded: signature.signature,
       };
     } catch (error) {
-      throw operationErrorMapping(operationType.SIGN, error);
+      throw errorHandler(operationType.Sign, error);
     }
   }
 
@@ -103,7 +103,7 @@ export class BearbyAccount implements IAccount {
         operationId,
       };
     } catch (error) {
-      throw operationErrorMapping(operationType.BUY_ROLLS, error);
+      throw errorHandler(operationType.BuyRolls, error);
     }
   }
 
@@ -115,7 +115,7 @@ export class BearbyAccount implements IAccount {
         operationId,
       };
     } catch (error) {
-      throw operationErrorMapping('sellRolls', error);
+      throw errorHandler(operationType.SellRolls, error);
     }
   }
 
@@ -133,7 +133,7 @@ export class BearbyAccount implements IAccount {
 
       return { operationId };
     } catch (error) {
-      throw operationErrorMapping(operationType.SEND_TRANSACTION, error);
+      throw errorHandler(operationType.SendTransaction, error);
     }
   }
 
@@ -174,7 +174,7 @@ export class BearbyAccount implements IAccount {
         unsafeParameters,
       });
     } catch (error) {
-      throw operationErrorMapping(operationType.CALL_SC, error);
+      throw errorHandler(operationType.CallSC, error);
     }
     return { operationId };
   }

src/errors/base.spec.ts

@@ -33,9 +33,9 @@ test('BaseError with cause', () => {
   const cause = new Error('The cause of the error');
   const baseError = new BaseError('An error occurred.', { cause });
   expect(baseError.cause).toBe(cause);
-  expect(baseError.message).toBe(`An error occurred.`);
+  expect(baseError.message).toBe('An error occurred.');
   if (baseError.cause instanceof Error) {
-    expect(baseError.cause.message).toBe(`The cause of the error`);
+    expect(baseError.cause.message).toBe('The cause of the error');
   }
 });
 

src/errors/utils/errorMapping.spec.ts → src/errors/utils/errorHandler.spec..ts

@@ -1,5 +1,5 @@
 import { UserRejectionError } from '../userRejection';
-import { isUserRejectionError, operationErrorMapping } from './errorMapping';
+import { isUserRejectionError, errorHandler } from './errorHandler';
 
 const massaUserRejectionErrorMessage = 'Action canceled by user';
 const bearbyUserRejectionErrorMessage = 'User rejected';
@@ -22,19 +22,19 @@ describe('isUserRejectionError', () => {
   });
 });
 
-describe('operationErrorMapping', () => {
+describe('errorHandler', () => {
   it('should return a UserRejectionError if the error is a user rejection error', () => {
     const error = new Error('User rejected');
-    const mappedError = operationErrorMapping('someOperation', error);
-    expect(mappedError).toBeInstanceOf(UserRejectionError);
-    expect(mappedError.message).toBe(
+    const processedError = errorHandler('someOperation', error);
+    expect(processedError).toBeInstanceOf(UserRejectionError);
+    expect(processedError.message).toBe(
       'The operation someOperation was rejected by the user.',
     );
   });
 
   it('should return the original error if the error is not a user rejection error', () => {
     const error = new Error(someOtherErrorMessage);
-    const mappedError = operationErrorMapping('someOperation', error);
-    expect(mappedError).toBe(error);
+    const processedError = errorHandler('someOperation', error);
+    expect(processedError).toBe(error);
   });
 });

src/errors/utils/errorMapping.ts → src/errors/utils/errorHandler.ts

@@ -10,10 +10,7 @@ export function isUserRejectionError(error: Error): boolean {
   );
 }
 
-export function operationErrorMapping(
-  operationName: string,
-  error: Error,
-): Error {
+export function errorHandler(operationName: string, error: Error): Error {
   if (isUserRejectionError(error)) {
     return new UserRejectionError({
       operationName: operationName,

src/massaStation/MassaStationAccount.ts

@@ -21,7 +21,8 @@ import { argsToBase64, uint8ArrayToBase64 } from '../utils/argsToBase64';
 import { IAccountSignOutput, ISignMessage } from '../account/AccountSign';
 import { encode as base58Encode } from 'bs58check';
 import { ExecuteFunctionBody } from './types';
-import { operationErrorMapping } from '../errors/utils/errorMapping';
+import { errorHandler } from '../errors/utils/errorHandler';
+import { operationType } from '../utils/constants';
 
 /**
  * This interface represents the the individual wallet's final and pending balances returned by MassaStation
@@ -145,7 +146,7 @@ export class MassaStationAccount implements IAccount {
     );
 
     if (signOpResponse.isError || signOpResponse.error) {
-      throw operationErrorMapping('sign', signOpResponse.error);
+      throw errorHandler(operationType.Sign, signOpResponse.error);
     }
 
     const signature = base58Encode(
@@ -180,7 +181,7 @@ export class MassaStationAccount implements IAccount {
     buyRollsOpResponse = await postRequest<ITransactionDetails>(url, body);
 
     if (buyRollsOpResponse.isError || buyRollsOpResponse.error) {
-      operationErrorMapping('buyRolls', buyRollsOpResponse.error);
+      errorHandler(operationType.BuyRolls, buyRollsOpResponse.error);
     }
     return buyRollsOpResponse.result;
   }
@@ -207,7 +208,7 @@ export class MassaStationAccount implements IAccount {
     sellRollsOpResponse = await postRequest<ITransactionDetails>(url, body);
 
     if (sellRollsOpResponse.isError || sellRollsOpResponse.error) {
-      operationErrorMapping('sellRolls', sellRollsOpResponse.error);
+      errorHandler(operationType.SellRolls, sellRollsOpResponse.error);
     }
     return sellRollsOpResponse.result;
   }
@@ -235,7 +236,7 @@ export class MassaStationAccount implements IAccount {
     sendTxOpResponse = await postRequest<ITransactionDetails>(url, body);
 
     if (sendTxOpResponse.isError || sendTxOpResponse.error) {
-      throw operationErrorMapping('sendTransaction', sendTxOpResponse.error);
+      throw errorHandler(operationType.SendTransaction, sendTxOpResponse.error);
     }
 
     return sendTxOpResponse.result;
@@ -302,7 +303,7 @@ export class MassaStationAccount implements IAccount {
     CallSCOpResponse = await postRequest<ITransactionDetails>(url, body);
 
     if (CallSCOpResponse.isError || CallSCOpResponse.error) {
-      throw operationErrorMapping('callSmartContract', CallSCOpResponse.error);
+      throw errorHandler('callSmartContract', CallSCOpResponse.error);
     }
     return CallSCOpResponse.result;
   }

src/utils/constants.ts

@@ -1,7 +1,7 @@
 export enum operationType {
-  CALL_SC = 'callSC',
-  SEND_TRANSACTION = 'sendTransaction',
-  SELL_ROLLS = 'sellRolls',
-  BUY_ROLLS = 'buyRolls',
-  SIGN = 'sign',
+  CallSC = 'callSC',
+  SendTransaction = 'sendTransaction',
+  SellRolls = 'sellRolls',
+  BuyRolls = 'buyRolls',
+  Sign = 'sign',
 }