Clean up code

- remove unused code
- update FromLedger test, to reflect runtimeIs changes

src/app/pages/OpenWalletPage/Features/FromBleLedger/index.tsx

@@ -2,7 +2,6 @@ import { importAccountsActions } from 'app/state/importaccounts'
 import { Box } from 'grommet/es6/components/Box'
 import { Button } from 'grommet/es6/components/Button'
 import { Heading } from 'grommet/es6/components/Heading'
-import React from 'react'
 import { useTranslation } from 'react-i18next'
 import { useDispatch, useSelector } from 'react-redux'
 import {

src/app/pages/OpenWalletPage/Features/FromLedger/__tests__/index.test.tsx

@@ -10,7 +10,7 @@ jest.mock('react-redux', () => ({
 jest.mock('../../../../../lib/ledger', () => ({
   ...jest.requireActual('../../../../../lib/ledger'),
   // Throws BLE not supported
-  canAccessBle: () => jest.fn().mockReturnValue(false),
+  canAccessBle: jest.fn().mockResolvedValue(false),
 }))
 
 const renderComponent = () =>
@@ -24,15 +24,15 @@ describe('<FromLedger />', () => {
   it('should render component in disabled state', async () => {
     renderComponent()
 
-    await waitFor(async () => {
+    await waitFor(() => {
       expect(screen.queryByText('openWallet.importAccounts.usbLedger')).toBeInTheDocument()
       expect(screen.queryByText('openWallet.importAccounts.bluetoothLedger')).toBeInTheDocument()
 
       expect(screen.getByText('errors.usbTransportNotSupported')).toBeInTheDocument()
       expect(screen.getByText('errors.bluetoothTransportNotSupported')).toBeInTheDocument()
 
-      const usbLedgerBtn = await screen.getByRole('button', { name: 'openWallet.importAccounts.usbLedger' })
-      const bluetoothLedgerBtn = await screen.getByRole('button', {
+      const usbLedgerBtn = screen.getByRole('button', { name: 'openWallet.importAccounts.usbLedger' })
+      const bluetoothLedgerBtn = screen.getByRole('button', {
         name: 'openWallet.importAccounts.bluetoothLedger',
       })
 

src/app/pages/OpenWalletPage/Features/FromUsbLedger/index.tsx

@@ -2,7 +2,6 @@ import { importAccountsActions } from 'app/state/importaccounts'
 import { Box } from 'grommet/es6/components/Box'
 import { Button } from 'grommet/es6/components/Button'
 import { Heading } from 'grommet/es6/components/Heading'
-import React from 'react'
 import { useTranslation } from 'react-i18next'
 import { useDispatch, useSelector } from 'react-redux'
 import { ImportAccountsSelectionModal } from 'app/pages/OpenWalletPage/Features/ImportAccountsSelectionModal'

src/app/pages/OpenWalletPage/Features/ListBleLedgerDevicesModal/index.tsx

@@ -1,4 +1,3 @@
-import React from 'react'
 import { useTranslation } from 'react-i18next'
 import { useDispatch, useSelector } from 'react-redux'
 import { AlertBox } from 'app/components/AlertBox'

src/app/pages/OpenWalletPage/__tests__/index.test.tsx

@@ -1,12 +1,20 @@
 import React from 'react'
-import { render } from '@testing-library/react'
+import { render, screen, waitFor } from '@testing-library/react'
 import { MemoryRouter } from 'react-router-dom'
 import { useSelector } from 'react-redux'
 import { when } from 'jest-when'
 import { selectShowAccountsSelectionModal } from 'app/state/importaccounts/selectors'
-import { canAccessNavigatorUsb } from 'app/lib/ledger'
 import { SelectOpenMethod } from '..'
 
+jest.mock('app/lib/ledger', () => ({
+  ...jest.requireActual('app/lib/ledger'),
+  canAccessBle: jest.fn().mockResolvedValue(false),
+  canAccessNavigatorUsb: jest.fn().mockResolvedValue(false),
+}))
+jest.mock('config', () => ({
+  ...jest.requireActual('config'),
+  runtimeIs: 'extension',
+}))
 jest.mock('react-redux', () => ({
   useSelector: jest.fn(),
 }))
@@ -18,31 +26,31 @@ jest.mock('react-router-dom', () => ({
   useNavigate: () => mockNavigate,
 }))
 
-const renderComponent = (webExtensionLedgerAccess?: () => void) =>
+const renderComponent = () =>
   render(
     <MemoryRouter>
-      <SelectOpenMethod webExtensionLedgerAccess={webExtensionLedgerAccess} />
+      <SelectOpenMethod />
     </MemoryRouter>,
   )
 
 describe('<SelectOpenMethod />', () => {
-  beforeEach(() => {
-    jest.mocked(canAccessNavigatorUsb).mockResolvedValue(false)
-  })
-
   it('should render component', () => {
     const { container } = renderComponent()
 
     expect(container).toMatchSnapshot()
   })
 
-  it('should redirect user to ledger page', () => {
+  it('should redirect user to ledger page', async () => {
     when(useSelector as any)
       .calledWith(selectShowAccountsSelectionModal)
       .mockReturnValue(true)
 
-    renderComponent(() => {})
+    renderComponent()
+
+    await waitFor(() => {
+      expect(screen.queryByText('openWallet.method.ledger')).toBeInTheDocument()
 
-    expect(mockNavigate).toHaveBeenCalledWith('/open-wallet/ledger/usb')
+      expect(mockNavigate).toHaveBeenCalledWith('/open-wallet/ledger/usb')
+    })
   })
 })