Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

build(deps-dev): bump testing packages #5882

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions packages/app-content-pages/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -48,14 +48,14 @@
"@storybook/nextjs": "~7.6.11",
"@storybook/react": "~7.6.11",
"@testing-library/dom": "~9.3.0",
"@testing-library/react": "~14.1.0",
"@testing-library/react": "~14.2.0",
"@testing-library/user-event": "~14.5.0",
"chai": "~4.3.4",
"chai-dom": "~1.12.0",
"dirty-chai": "~2.0.1",
"eslint-config-next": "~13.5.5",
"eslint-plugin-jsx-a11y": "~6.8.0",
"jsdom": "~23.0.0",
"jsdom": "~24.0.0",
"mocha": "~10.2.0",
"proxyquire": "~2.1.0",
"selfsigned": "~2.4.1",
Expand Down
4 changes: 2 additions & 2 deletions packages/app-project/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@
"@storybook/nextjs": "~7.6.11",
"@storybook/react": "~7.6.11",
"@testing-library/dom": "~9.3.0",
"@testing-library/react": "~14.1.0",
"@testing-library/react": "~14.2.0",
"@testing-library/user-event": "~14.5.0",
"@wojtekmaj/enzyme-adapter-react-17": "~0.8.0",
"babel-loader": "~9.1.0",
Expand All @@ -80,7 +80,7 @@
"enzyme": "~3.11.0",
"eslint-config-next": "~13.5.6",
"eslint-plugin-jsx-a11y": "~6.8.0",
"jsdom": "~23.0.0",
"jsdom": "~24.0.0",
"mocha": "~10.2.0",
"nock": "~13.5.1",
"selfsigned": "~2.4.1",
Expand Down
4 changes: 2 additions & 2 deletions packages/lib-classifier/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@
"@storybook/react": "~7.6.11",
"@storybook/react-webpack5": "~7.6.11",
"@testing-library/dom": "~9.3.0",
"@testing-library/react": "~14.1.0",
"@testing-library/react": "~14.2.0",
"@testing-library/user-event": "~14.5.0",
"@visx/mock-data": "~3.3.0",
"@wojtekmaj/enzyme-adapter-react-17": "~0.8.0",
Expand All @@ -96,7 +96,7 @@
"grommet-icons": "~4.12.0",
"html-webpack-plugin": "~5.6.0",
"ignore-styles": "~5.0.1",
"jsdom": "~23.0.0",
"jsdom": "~24.0.0",
"million": "~2.6.4",
"mobx": "~6.12.0",
"mobx-devtools-mst": "~0.9.21",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -679,7 +679,7 @@ describe('Component > TranscribedLines', function () {
expect(popup).to.be.null()
line.focus()
await user.keyboard('{Enter}')
popup = screen.getByRole('heading', { level: 2, name: popupTitle})
popup = await screen.findByRole('heading', { level: 2, name: popupTitle})
expect(popup).to.exist()
const closeButton = screen.queryByRole('button', { name: 'Close' })
closeButton.focus()
Expand All @@ -703,7 +703,7 @@ describe('Component > TranscribedLines', function () {
expect(popup).to.be.null()
line.focus()
await user.keyboard('{ }')
popup = screen.getByRole('heading', { level: 2, name: popupTitle})
popup = await screen.findByRole('heading', { level: 2, name: popupTitle})
expect(popup).to.exist()
const closeButton = screen.queryByRole('button', { name: 'Close' })
closeButton.focus()
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { render, screen } from '@testing-library/react'
import { render, screen, waitFor } from '@testing-library/react'
import userEvent from '@testing-library/user-event'

import TaskHelp from './TaskHelp'
Expand Down Expand Up @@ -40,7 +40,7 @@ describe('TaskHelp', function () {
const needHelpButton = screen.getByRole('button', { name: 'TaskArea.Tasks.TaskHelp.label' })
await user.click(needHelpButton)

expect(screen.getByText('Try this')).to.be.ok()
await waitFor(() => expect(screen.getByText('Try this')).to.be.ok())
})

it('should not show any <hr />', async function () {
Expand All @@ -58,7 +58,7 @@ describe('TaskHelp', function () {
const needHelpButton = screen.getByRole('button', { name: 'TaskArea.Tasks.TaskHelp.label' })
await user.click(needHelpButton)

expect(screen.getByText('Try this')).to.be.ok()
await waitFor(() => expect(screen.getByText('Try this')).to.be.ok())
const closeButton = screen.getByRole('button', { name: 'TaskArea.Tasks.TaskHelp.close' })
await user.click(closeButton)

Expand All @@ -79,7 +79,7 @@ describe('TaskHelp', function () {
const needHelpButton = screen.getByRole('button', { name: 'TaskArea.Tasks.TaskHelp.label' })
await user.click(needHelpButton)

expect(screen.getByText('Try this')).to.be.ok()
await waitFor(() => expect(screen.getByText('Try this')).to.be.ok())
expect(screen.getByText('Try this again')).to.be.ok()
})

Expand All @@ -89,7 +89,7 @@ describe('TaskHelp', function () {
const needHelpButton = screen.getByRole('button', { name: 'TaskArea.Tasks.TaskHelp.label' })
await user.click(needHelpButton)

expect(screen.getAllByRole('separator')).to.have.lengthOf(1)
await waitFor(() => expect(screen.getAllByRole('separator')).to.have.lengthOf(1))
})
})
})
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ describe('SurveyTask with user keystrokes', function () {

it('should show characteristic filter sections', async function () {
// the filterSections are the characteristic filter sections, i.e. the sections for "Like", "Pattern", and "Color" for the mock task
const characteristicsSection = screen.getByTestId('characteristics')
const characteristicsSection = await screen.findByTestId('characteristics')
const filterSections = within(characteristicsSection).getAllByRole('radiogroup')
expect(filterSections.length).to.equal(3)
})
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ describe('TextTask > Components > TextTaskWithSuggestions', function () {
keys: '[MouseLeft]',
target: textInput
})
options = document.querySelectorAll('[role=option]')
options = await screen.findAllByRole('option')
})


Expand Down
2 changes: 1 addition & 1 deletion packages/lib-panoptes-js/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
"@zooniverse/standard": "~2.0.0",
"chai": "~4.3.4",
"dirty-chai": "~2.0.1",
"jsdom": "~23.0.0",
"jsdom": "~24.0.0",
"mocha": "~10.2.0",
"mock-local-storage": "~1.1.19",
"nock": "~13.5.1",
Expand Down
4 changes: 2 additions & 2 deletions packages/lib-react-components/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@
"@storybook/react": "~7.6.11",
"@storybook/react-webpack5": "~7.6.11",
"@testing-library/dom": "~9.3.0",
"@testing-library/react": "~14.1.0",
"@testing-library/react": "~14.2.0",
"@testing-library/user-event": "~14.5.0",
"@wojtekmaj/enzyme-adapter-react-17": "~0.8.0",
"@zooniverse/grommet-theme": "~3.2.0",
Expand All @@ -90,7 +90,7 @@
"enzyme": "~3.11.0",
"grommet": "~2.34.0",
"grommet-icons": "~4.12.0",
"jsdom": "~23.0.0",
"jsdom": "~24.0.0",
"lodash": "~4.17.11",
"mobx-state-tree": "~5.4.0",
"mocha": "~10.2.0",
Expand Down
6 changes: 3 additions & 3 deletions packages/lib-react-components/src/Tabs/Tabs.spec.js
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
import { render, screen, within } from '@testing-library/react'
import { render, screen, waitFor, within } from '@testing-library/react'
import userEvent from '@testing-library/user-event'
import zooTheme from '@zooniverse/grommet-theme'
import { Grommet } from 'grommet'
Expand Down Expand Up @@ -43,9 +43,9 @@ describe('Component > Tabs', function () {
expect(within(tabPanel).getByText('An apple is a red fruit.')).to.exist()

await user.click(tabButton1)
expect(within(tabPanel).getByText('A banana is a long fruit.')).to.exist()
await waitFor(() => expect(within(tabPanel).getByText('A banana is a long fruit.')).to.exist())

await user.click(tabButton2)
expect(within(tabPanel).getByText('A cherry is a stone fruit.')).to.exist()
await waitFor(() => expect(within(tabPanel).getByText('A cherry is a stone fruit.')).to.exist())
})
})
4 changes: 2 additions & 2 deletions packages/lib-user/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@
"@storybook/react": "~7.6.11",
"@storybook/react-webpack5": "~7.6.11",
"@testing-library/dom": "~9.3.0",
"@testing-library/react": "~14.1.0",
"@testing-library/react": "~14.2.0",
"@testing-library/user-event": "~14.5.0",
"babel-loader": "~9.1.0",
"babel-plugin-module-resolver": "~5.0.0",
Expand All @@ -69,7 +69,7 @@
"eslint-plugin-react": "~7.33.0",
"html-webpack-plugin": "~5.6.0",
"ignore-styles": "~5.0.1",
"jsdom": "~23.0.0",
"jsdom": "~24.0.0",
"mocha": "~10.2.0",
"nock": "~13.5.1",
"panoptes-client": "~5.6.0",
Expand Down
Loading