diff --git a/ui/pages/multichain-accounts/base-account-details/base-account-details.test.tsx b/ui/pages/multichain-accounts/base-account-details/base-account-details.test.tsx index 7ddb01605aa6..3151d89713fe 100644 --- a/ui/pages/multichain-accounts/base-account-details/base-account-details.test.tsx +++ b/ui/pages/multichain-accounts/base-account-details/base-account-details.test.tsx @@ -8,10 +8,7 @@ import { MOCK_ACCOUNT_EOA, MOCK_ACCOUNT_SOLANA_MAINNET, } from '../../../../test/data/mock-accounts'; -import { - ACCOUNT_DETAILS_QR_CODE_ROUTE, - DEFAULT_ROUTE, -} from '../../../helpers/constants/routes'; +import { ACCOUNT_DETAILS_QR_CODE_ROUTE } from '../../../helpers/constants/routes'; import { KeyringType } from '../../../../shared/constants/keyring'; import { BaseAccountDetails } from './base-account-details'; @@ -20,10 +17,12 @@ const mockStore = configureMockStore(middleware); // Mock the useHistory hook const mockPush = jest.fn(); +const mockGoBack = jest.fn(); jest.mock('react-router-dom', () => ({ ...jest.requireActual('react-router-dom'), useHistory: () => ({ push: mockPush, + goBack: mockGoBack, }), })); @@ -135,6 +134,7 @@ const createMockState = ( describe('BaseAccountDetails', () => { beforeEach(() => { mockPush.mockClear(); + mockGoBack.mockClear(); }); describe('Component Rendering', () => { @@ -216,7 +216,7 @@ describe('BaseAccountDetails', () => { }); describe('Navigation', () => { - it('should navigate to default route when back button is clicked', () => { + it('should go back when back button is clicked', () => { const state = createMockState(MOCK_ACCOUNT_EOA.address); const store = mockStore(state); @@ -233,7 +233,7 @@ describe('BaseAccountDetails', () => { const backButton = screen.getByLabelText('Back'); fireEvent.click(backButton); - expect(mockPush).toHaveBeenCalledWith(DEFAULT_ROUTE); + expect(mockGoBack).toHaveBeenCalledTimes(1); }); it('should navigate to QR code route when address row is clicked', () => { diff --git a/ui/pages/multichain-accounts/base-account-details/base-account-details.tsx b/ui/pages/multichain-accounts/base-account-details/base-account-details.tsx index 259c23b9a319..cb23d8aa7259 100644 --- a/ui/pages/multichain-accounts/base-account-details/base-account-details.tsx +++ b/ui/pages/multichain-accounts/base-account-details/base-account-details.tsx @@ -100,7 +100,7 @@ export const BaseAccountDetails = ({ const handleNavigation = useCallback(() => { dispatch(setAccountDetailsAddress('')); - history.push(DEFAULT_ROUTE); + history.goBack(); }, [history, dispatch]); // we can never have a scenario where an account is not associated with a wallet.