fix: unit test

Author: micaelae

app/core/Engine/controllers/bridge-status-controller/bridge-status-controller-init.test.ts

@@ -29,10 +29,7 @@ function buildTransactionControllerMock(
   partialMock?: Partial<TransactionController>,
 ): TransactionController {
   const defaultMocks = {
-    addTransaction: jest.fn().mockResolvedValue('txId'),
-    estimateGasFee: jest.fn().mockResolvedValue({ gasLimit: '0x5208' }),
     addTransactionBatch: jest.fn().mockResolvedValue(['txId1', 'txId2']),
-    updateTransaction: jest.fn().mockResolvedValue(undefined),
   };
 
   // @ts-expect-error Incomplete mock, just includes properties used by code-under-test
@@ -197,62 +194,6 @@ describe('BridgeStatusController Init', () => {
       });
     });
 
-    it('correctly sets up addTransactionFn', () => {
-      // Arrange
-      const mockTransactionController = buildTransactionControllerMock({
-        addTransaction: jest.fn().mockResolvedValue('newTxId'),
-      });
-      const requestMock = buildInitRequestMock({
-        getController: jest.fn().mockReturnValue(mockTransactionController),
-      });
-
-      // Act
-      bridgeStatusControllerInit(requestMock);
-
-      // Assert
-      const constructorOptions =
-        bridgeStatusControllerClassMock.mock.calls[0][0];
-      const addTransactionFn = constructorOptions.addTransactionFn;
-      const mockTxParams = { from: '0xabc', to: '0x123', value: '0x0' };
-      const mockOrigin = 'test-origin';
-
-      addTransactionFn(mockTxParams, {
-        origin: mockOrigin,
-        networkClientId: 'mainnet',
-      });
-      expect(mockTransactionController.addTransaction).toHaveBeenCalledWith(
-        mockTxParams,
-        { origin: mockOrigin, networkClientId: 'mainnet' },
-      );
-    });
-
-    it('correctly sets up estimateGasFeeFn', () => {
-      // Arrange
-      const mockTransactionController = buildTransactionControllerMock({
-        estimateGasFee: jest.fn().mockResolvedValue({ gasLimit: '0x7530' }),
-      });
-      const requestMock = buildInitRequestMock({
-        getController: jest.fn().mockReturnValue(mockTransactionController),
-      });
-
-      // Act
-      bridgeStatusControllerInit(requestMock);
-
-      // Assert
-      const constructorOptions =
-        bridgeStatusControllerClassMock.mock.calls[0][0];
-      const estimateGasFeeFn = constructorOptions.estimateGasFeeFn;
-      const mockTxParams = {
-        transactionParams: { from: '0xabc', to: '0x123', value: '0x0' },
-        chainId: '0x1' as const,
-      };
-
-      estimateGasFeeFn(mockTxParams);
-      expect(mockTransactionController.estimateGasFee).toHaveBeenCalledWith(
-        mockTxParams,
-      );
-    });
-
     it('correctly sets up addTransactionBatchFn', () => {
       // Arrange
       const mockTransactionController = buildTransactionControllerMock({
@@ -284,40 +225,6 @@ describe('BridgeStatusController Init', () => {
       ).toHaveBeenCalledWith(mockTxBatch);
     });
 
-    it('correctly sets up updateTransactionFn', () => {
-      // Arrange
-      const mockTransactionController = buildTransactionControllerMock({
-        updateTransaction: jest.fn().mockResolvedValue(undefined),
-      });
-      const requestMock = buildInitRequestMock({
-        getController: jest.fn().mockReturnValue(mockTransactionController),
-      });
-
-      // Act
-      bridgeStatusControllerInit(requestMock);
-
-      // Assert
-      const constructorOptions =
-        bridgeStatusControllerClassMock.mock.calls[0][0];
-      const updateTransactionFn = constructorOptions.updateTransactionFn;
-      const mockTxUpdate = {
-        id: 'txId',
-        chainId: '0x1' as const,
-        networkClientId: 'mainnet',
-        time: Date.now(),
-        txParams: { from: '0xabc', to: '0x123', value: '0x0' },
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        status: 'confirmed' as any,
-      };
-      const mockNote = 'test note';
-
-      updateTransactionFn(mockTxUpdate, mockNote);
-      expect(mockTransactionController.updateTransaction).toHaveBeenCalledWith(
-        mockTxUpdate,
-        mockNote,
-      );
-    });
-
     it('handles undefined persistedState', () => {
       // Arrange
       const requestMock = buildInitRequestMock({