Update npm dependencies (especially @wordpress)

assets/admin/editor-wizard/helpers.test.js

@@ -116,7 +116,7 @@ describe( 'useWizardOpenState', () => {
 		expect( await findByText( 'closed' ) ).toBeTruthy();
 
 		document.body.classList.remove( 'modal-open' );
-		expect( findByText( 'open' ) ).toBeTruthy();
+		expect( await findByText( 'open' ) ).toBeTruthy();
 	} );
 
 	it( 'Should be closed when wizard is done', async () => {

assets/admin/exit-survey/form.test.js

@@ -18,50 +18,52 @@ describe( '<ExitSurveyForm />', () => {
 		skip: () => screen.getByRole( 'button', { name: 'Skip Feedback' } ),
 	};
 
-	it( 'Submit is disabled until an item is selected and details filled out (if provided)', () => {
+	it( 'Submit is disabled until an item is selected and details filled out (if provided)', async () => {
 		const { getByLabelText, getByPlaceholderText } = render(
 			<ExitSurveyForm />
 		);
 
 		expect( buttons.submit() ).toBeDisabled();
 
 		// This reason does not require details.
-		userEvent.click( getByLabelText( 'I no longer need the plugin' ) );
+		await userEvent.click(
+			getByLabelText( 'I no longer need the plugin' )
+		);
 		expect( buttons.submit() ).not.toBeDisabled();
 
 		// This reason does expect details.
-		userEvent.click( getByLabelText( 'I found a better plugin' ) );
+		await userEvent.click( getByLabelText( 'I found a better plugin' ) );
 		expect( buttons.submit() ).toBeDisabled();
-		userEvent.type(
+		await userEvent.type(
 			getByPlaceholderText( "What's the name of the plugin?" ),
 			'Test detail'
 		);
 
 		expect( buttons.submit() ).not.toBeDisabled();
 	} );
 
-	it( 'Skip button skips submission', () => {
+	it( 'Skip button skips submission', async () => {
 		const skip = jest.fn();
 		const submit = jest.fn();
 		render( <ExitSurveyForm submit={ submit } skip={ skip } /> );
-		userEvent.click( buttons.skip() );
+		await userEvent.click( buttons.skip() );
 
 		expect( skip ).toHaveBeenCalled();
 		expect( submit ).not.toHaveBeenCalled();
 	} );
 
-	it( 'Submits selected reason and details', () => {
+	it( 'Submits selected reason and details', async () => {
 		const submit = jest.fn();
 		const { getByLabelText, getByPlaceholderText } = render(
 			<ExitSurveyForm submit={ submit } />
 		);
 
-		userEvent.click( getByLabelText( 'I found a better plugin' ) );
-		userEvent.type(
+		await userEvent.click( getByLabelText( 'I found a better plugin' ) );
+		await userEvent.type(
 			getByPlaceholderText( "What's the name of the plugin?" ),
 			'Test detail'
 		);
-		userEvent.click( buttons.submit() );
+		await userEvent.click( buttons.submit() );
 
 		expect( submit ).toHaveBeenCalledWith( {
 			reason: 'found-better-plugin',

assets/admin/students/hooks/use-abort-controller/index.test.js

@@ -6,7 +6,7 @@ import useAbortController from '.';
 /**
  * External dependencies
  */
-import { cleanup, renderHook } from '@testing-library/react-hooks';
+import { cleanup, renderHook } from '@testing-library/react';
 
 describe( 'useAbortController', () => {
 	it( 'Should return signal to be used on async operations', () => {

assets/admin/students/student-bulk-action-button/index.test.js

@@ -62,15 +62,16 @@ describe( '<StudentBulkActionButton />', () => {
 		expect( button ).toBeDisabled();
 	} );
 
-	it( 'Should render the `Add to course` modal', () => {
+	it( 'Should render the `Add to course` modal', async () => {
 		setupSelector( [
 			{ value: 'enrol_restore_enrolment', selected: true },
 			{ value: 'remove_progress' },
 			{ value: 'remove_enrolment' },
 		] );
 		render( <StudentBulkActionButton isDisabled={ false } /> );
 
-		selectActionButton().click();
+		await selectActionButton().click();
+
 		expect(
 			screen.getByText(
 				ignoreInlineTags(
@@ -80,15 +81,15 @@ describe( '<StudentBulkActionButton />', () => {
 		).toBeInTheDocument();
 	} );
 
-	it( 'Should render the `Reset Progress` modal', () => {
+	it( 'Should render the `Reset Progress` modal', async () => {
 		setupSelector( [
 			{ value: 'enrol_restore_enrolment' },
 			{ value: 'remove_progress', selected: true },
 			{ value: 'remove_enrolment' },
 		] );
 		render( <StudentBulkActionButton isDisabled={ false } /> );
 
-		selectActionButton().click();
+		await selectActionButton().click();
 
 		expect(
 			screen.getByText(
@@ -98,15 +99,16 @@ describe( '<StudentBulkActionButton />', () => {
 			)
 		).toBeInTheDocument();
 	} );
-	it( 'Should render the `Remove from Course` modal', () => {
+	it( 'Should render the `Remove from Course` modal', async () => {
 		setupSelector( [
 			{ value: 'enrol_restore_enrolment' },
 			{ value: 'remove_progress' },
 			{ value: 'remove_enrolment', selected: true },
 		] );
 		render( <StudentBulkActionButton isDisabled={ false } /> );
 
-		selectActionButton().click();
+		await selectActionButton().click();
+
 		expect(
 			screen.getByText(
 				ignoreInlineTags(

assets/admin/students/student-modal/index.test.js

@@ -337,14 +337,14 @@ describe( '<StudentModal />', () => {
 					/>
 				);
 
-				fireEvent.click( await courseOptionAt( 0 ) );
-				fireEvent.click( await buttonByLabel( 'Add to Course' ) );
+				await fireEvent.click( await courseOptionAt( 0 ) );
+				await fireEvent.click( await buttonByLabel( 'Add to Course' ) );
 
 				expect(
 					await findAllByText(
 						'Unable to add student. Please try again.'
 					)
-				).toHaveLength( 2 ); // ARIA + notice
+				).toBeTruthy();
 			} );
 
 			it( 'Should display an error message when removing a student from a course', async () => {
@@ -363,7 +363,7 @@ describe( '<StudentModal />', () => {
 					await findAllByText(
 						'Unable to remove student. Please try again.'
 					)
-				).toHaveLength( 2 ); // ARIA + notice
+				).toBeTruthy();
 			} );
 
 			it( 'Should display an error message when resetting progress for a single student', async () => {
@@ -383,7 +383,7 @@ describe( '<StudentModal />', () => {
 					await findAllByText(
 						'Unable to reset progress for this student. Please try again.'
 					)
-				).toHaveLength( 2 ); // ARIA + notice
+				).toBeTruthy();
 			} );
 		} );
 
@@ -437,14 +437,14 @@ describe( '<StudentModal />', () => {
 					/>
 				);
 
-				fireEvent.click( await courseOptionAt( 0 ) );
-				fireEvent.click( await buttonByLabel( 'Add to Course' ) );
+				await fireEvent.click( await courseOptionAt( 0 ) );
+				await fireEvent.click( await buttonByLabel( 'Add to Course' ) );
 
 				expect(
 					await findAllByText(
 						'Unable to add students. Please try again.'
 					)
-				).toHaveLength( 2 ); // ARIA + notice
+				).toBeTruthy();
 			} );
 
 			it( 'Should display an error message when removing multiple students from a course', async () => {
@@ -456,14 +456,16 @@ describe( '<StudentModal />', () => {
 					/>
 				);
 
-				fireEvent.click( await courseOptionAt( 0 ) );
-				fireEvent.click( await buttonByLabel( 'Remove from Course' ) );
+				await fireEvent.click( await courseOptionAt( 0 ) );
+				await fireEvent.click(
+					await buttonByLabel( 'Remove from Course' )
+				);
 
 				expect(
 					await findAllByText(
 						'Unable to remove students. Please try again.'
 					)
-				).toHaveLength( 2 ); // ARIA + notice
+				).toBeTruthy();
 			} );
 
 			it( 'Should display an error message when resetting progress for multiple students', async () => {
@@ -482,7 +484,7 @@ describe( '<StudentModal />', () => {
 					await findAllByText(
 						'Unable to reset progress for these students. Please try again.'
 					)
-				).toHaveLength( 2 ); // ARIA + notice
+				).toBeTruthy();
 			} );
 		} );
 	} );

assets/blocks/course-categories-block/hooks/use-course-categories-props/index.test.js

@@ -1,7 +1,7 @@
 /**
  * External dependencies
  */
-import { renderHook } from '@testing-library/react-hooks';
+import { renderHook } from '@testing-library/react';
 /**
  * Internal dependencies
  */

assets/blocks/course-categories-block/hooks/use-course-categories/index.test.js

@@ -1,7 +1,7 @@
 /**
  * External dependencies
  */
-import { renderHook } from '@testing-library/react-hooks';
+import { renderHook } from '@testing-library/react';
 import { when } from 'jest-when';
 /**
  * WordPress dependencies

assets/blocks/course-outline/outline-block/outline-placeholder.test.js

@@ -34,19 +34,21 @@
 		expect( getByText( 'Start with blank' ) ).toBeVisible();
 
 		expect( getByText( 'Generate with AI' ) ).toBeVisible();
 	} );
 
-	it( 'Should create empty lessons', () => {
+	it( 'Should create empty lessons', async () => {
 		const { getByRole } = render(
 			<OutlinePlaceholder addBlocks={ addBlocksMock } />
 		);
 
-		userEvent.click( getByRole( 'button', { name: 'Start with blank' } ) );
+		await userEvent.click(
+			getByRole( 'button', { name: 'Start with blank' } )
+		);
 
 		expect( addBlocksMock ).toHaveBeenCalled();
 	} );
 
-	it( 'Should open the tailored modal', () => {
+	it( 'Should open the tailored modal', async () => {
 		const openTailoredModalMock = jest.fn();
 
 		const { getByRole } = render(
@@ -56,7 +58,7 @@
 			/>
 		);
 
-		userEvent.click(
+		await userEvent.click(
 			getByRole( 'button', { name: 'Generate with AI Pro' } )
 		);
 

assets/blocks/course-outline/outline-block/use-sensei-pro-settings.test.js

@@ -1,7 +1,7 @@
 /**
  * External dependencies
  */
-import { renderHook } from '@testing-library/react-hooks';
+import { renderHook } from '@testing-library/react';
 /**
  * Internal dependencies
  */

assets/blocks/course-outline/status-preview/status-control/index.test.js

@@ -63,10 +63,10 @@ describe( '<StatusControl />', () => {
 				setStatus={ setStatusMock }
 			/>
 		);
-		userEvent.click( getByLabelText( 'Completed' ) );
+		await userEvent.click( getByLabelText( 'Completed' ) );
 		expect( setStatusMock ).toBeCalledWith( Status.COMPLETED );
 
-		userEvent.click( getByLabelText( 'In Progress' ) );
+		await userEvent.click( getByLabelText( 'In Progress' ) );
 		expect( setStatusMock ).toBeCalledWith( Status.IN_PROGRESS );
 	} );
 } );

assets/blocks/editor-components/confirm-dialog/use-confirm-dialog-props.test.js

@@ -1,7 +1,7 @@
 /**
  * External dependencies
  */
-import { renderHook, act } from '@testing-library/react-hooks';
+import { renderHook, act } from '@testing-library/react';
 
 /**
  * Internal dependencies
@@ -34,16 +34,15 @@ describe( 'useConfirmDialogProps()', () => {
 		const { result } = renderHook( () => useConfirmDialogProps() );
 		let [ props, confirm ] = result.current;
 		expect( props.isOpen ).toBe( false );
-		const confirmResponse = act( () =>
+		act( () => {
+			// eslint-disable-next-line jest/valid-expect
 			expect(
 				confirm( 'Hey Content', { title: 'Hey Title' } )
-			).resolves.toBe( true )
-		);
+			).resolves.toBe( true );
+		} );
 		[ props, confirm ] = result.current;
 		expect( props.isOpen ).toBe( true );
 		act( () => props.onConfirm() );
-		// We need to verify AFTER calling the props.on* callback, otherwise, the promise won't be resolved yet.
-		await confirmResponse;
 		[ props, confirm ] = result.current;
 		expect( props.isOpen ).toBe( false );
 	} );
@@ -52,16 +51,15 @@ describe( 'useConfirmDialogProps()', () => {
 		const { result } = renderHook( () => useConfirmDialogProps() );
 		let [ props, confirm ] = result.current;
 		expect( props.isOpen ).toBe( false );
-		const confirmResponse = act( () =>
+		act( () => {
+			// eslint-disable-next-line jest/valid-expect
 			expect(
 				confirm( 'Hey Content', { title: 'Hey Title' } )
-			).resolves.toBe( false )
-		);
+			).resolves.toBe( false );
+		} );
 		[ props, confirm ] = result.current;
 		expect( props.isOpen ).toBe( true );
 		act( () => props.onCancel() );
-		// We need to verify AFTER calling the props.on* callback, otherwise, the promise won't be resolved yet.
-		await confirmResponse;
 		[ props, confirm ] = result.current;
 		expect( props.isOpen ).toBe( false );
 	} );