Fix tests

Automattic · Sep 24, 2024 · 55e1bc4 · 55e1bc4
1 parent c520eff
commit 55e1bc4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/...low/internals/steps-repository/site-migration-already-wordpress/components/form/index.tsx b/...low/internals/steps-repository/site-migration-already-wordpress/components/form/index.tsx
@@ -105,7 +105,9 @@ const Form: FC< FormProps > = ( { onComplete } ) => {
 
 	useEffect( () => {
 		if ( isSuccess ) {
-			alert( 'Success!' );
+			if ( process.env.NODE_ENV !== 'development' ) {
+				alert( 'Success!' );
+			}
 			onComplete();
 		}
 	}, [ isSuccess, onComplete ] );

diff --git a/...clarative-flow/internals/steps-repository/site-migration-already-wordpress/test/index.tsx b/...clarative-flow/internals/steps-repository/site-migration-already-wordpress/test/index.tsx
@@ -28,15 +28,15 @@ describe( 'SiteMigrationAlreadyWordPress', () => {
 		render( { navigation: { submit } }, { initialEntry: '/some-path?from=https://example.com' } );
 
 		await userEvent.click( continueButton() );
-		expect( await screen.findByText( 'Please select an option' ) ).toBeVisible();
+		expect( await screen.findByText( /Please select an option/ ) ).toBeVisible();
 	} );
 
 	it( 'shows an error when the user selects other but does not provide details', async () => {
 		render( {}, { initialEntry: '/some-path?from=https://example.com' } );
 		userEvent.click( intentByName( 'Other' ) );
 		await userEvent.click( continueButton() );
 
-		expect( await screen.findByText( 'Please describe your needs' ) ).toBeVisible();
+		expect( await screen.findByText( /Please, provide more details/ ) ).toBeVisible();
 	} );
 
 	it( 'navigate to next step when the submits an intent', async () => {