Skip to content

Commit 904fb14

Browse files
committed
fix tests
1 parent 3753b53 commit 904fb14

File tree

5 files changed

+16
-19
lines changed

5 files changed

+16
-19
lines changed

src/containers/Accounts/AccountNFTTable/test/AccountNFTTable.test.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
1-
import { render, screen, cleanup } from '@testing-library/react'
2-
import { userEvent } from '@testing-library/user-event'
1+
import { render, screen, cleanup, fireEvent } from '@testing-library/react'
32
import { QueryClientProvider } from 'react-query'
43
import { I18nextProvider } from 'react-i18next'
54
import { BrowserRouter } from 'react-router-dom'
@@ -84,7 +83,8 @@ describe('AccountNFTTable component', () => {
8483
expect(columns[1]).toHaveTextContent(`rGymBL8Huct6euA8jtEcLagYXpRgQKh6EC`)
8584
expect(columns[2]).toHaveTextContent(`0`)
8685
expect(screen.getByRole('button')).toBeDefined()
87-
userEvent.click(screen.getByRole('button'))
86+
fireEvent.click(screen.getByRole('button'))
87+
expect(mockedGetAccountNFTs).toHaveBeenCalledTimes(2)
8888
expect(mockedGetAccountNFTs.mock.calls[1][2]).toEqual('hello')
8989
})
9090

src/containers/Accounts/AccountTransactionTable/test/AccountTransactionTable.test.tsx

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
1-
import { cleanup, render, screen } from '@testing-library/react'
2-
import { userEvent } from '@testing-library/user-event'
1+
import { cleanup, fireEvent, render, screen } from '@testing-library/react'
32
import i18n from '../../../../i18n/testConfig'
43
import { AccountTransactionTable } from '../index'
54
import TEST_TRANSACTIONS_DATA from './mockTransactions.json'
@@ -62,7 +61,7 @@ describe('AccountTransactionsTable container', () => {
6261
expect(screen.getAllByRole('link')).toHaveLength(60)
6362

6463
const button = screen.getByRole('button')
65-
userEvent.click(button)
64+
fireEvent.click(button)
6665
expect(getAccountTransactions).toHaveBeenCalledWith(
6766
TEST_ACCOUNT_ID,
6867
undefined,
@@ -107,7 +106,7 @@ describe('AccountTransactionsTable container', () => {
107106
expect(screen.getAllByRole('link')).toHaveLength(60)
108107

109108
const button = screen.getByRole('button')
110-
userEvent.click(button)
109+
fireEvent.click(button)
111110
expect(getAccountTransactions).toHaveBeenCalledWith(
112111
TEST_ACCOUNT_ID,
113112
undefined,

src/containers/Header/NetworkPicker/NetworkPicker.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,6 @@ export const NetworkPicker = () => {
142142
: networkPickerTitle
143143
}
144144
className={`network network-${currentMode}`}
145-
data-testid="dropdown"
146145
>
147146
<>
148147
{networks.map(({ network, title, url = '' }) => {

src/containers/shared/components/Dropdown/test/Dropdown.test.tsx

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
1-
import { cleanup, render, screen } from '@testing-library/react'
2-
import { userEvent } from '@testing-library/user-event'
1+
import { cleanup, fireEvent, render, screen } from '@testing-library/react'
32
import { Dropdown } from '../Dropdown'
43

54
describe('Dropdown', () => {
@@ -49,10 +48,11 @@ describe('Dropdown', () => {
4948
it('shows menu when clicking toggle', () => {
5049
render(<Dropdown title="Woo">Menu Contents</Dropdown>)
5150
expect(screen.getByTestId('dropdown')).not.toHaveClass('dropdown-expanded')
51+
screen.debug()
5252
const button = screen.getByRole('button')
53-
userEvent.click(button)
53+
fireEvent.click(button)
5454
expect(screen.getByTestId('dropdown')).toHaveClass('dropdown-expanded')
55-
userEvent.click(button)
55+
fireEvent.click(button)
5656
expect(screen.getByTestId('dropdown')).not.toHaveClass('dropdown-expanded')
5757
})
5858

@@ -72,9 +72,9 @@ describe('Dropdown', () => {
7272

7373
const child = screen.getAllByRole('button')[0]
7474
const outside = screen.getAllByRole('button')[1]
75-
userEvent.click(child)
75+
fireEvent.click(child)
7676
expect(screen.getByTestId('dropdown')).toHaveClass('dropdown-expanded')
77-
userEvent.click(outside)
77+
fireEvent.click(outside)
7878
expect(screen.getByTestId('dropdown')).not.toHaveClass('dropdown-expanded')
7979
})
8080

@@ -86,7 +86,7 @@ describe('Dropdown', () => {
8686
expect(toggle).toHaveAttribute('tabIndex', '0')
8787
expect(menu).toHaveAttribute('role', 'menu')
8888
expect(menu).toHaveAttribute('tabIndex', '0')
89-
userEvent.click(toggle)
89+
fireEvent.click(toggle)
9090
expect(toggle).toHaveAttribute('aria-expanded', 'true')
9191
expect(menu).toHaveAttribute('aria-hidden', 'false')
9292
})

src/containers/shared/components/Dropdown/test/DropdownItem.test.tsx

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
1-
import { cleanup, render, screen } from '@testing-library/react'
2-
import { userEvent } from '@testing-library/user-event'
1+
import { cleanup, fireEvent, render, screen } from '@testing-library/react'
32
import { DropdownItem } from '../DropdownItem'
43
import createSpy = jasmine.createSpy
54

@@ -27,12 +26,12 @@ describe('DropdownItem', () => {
2726
})
2827

2928
it('executes handler on click', () => {
30-
userEvent.click(item)
29+
fireEvent.click(item)
3130
expect(handler).toHaveBeenCalled()
3231
})
3332

3433
it('executes handler on keyup', () => {
35-
userEvent.click(item)
34+
fireEvent.click(item)
3635
expect(handler).toHaveBeenCalled()
3736
})
3837
})

0 commit comments

Comments
 (0)