update test

Author: ilicfilip

tests/e2e/auth.setup.js

@@ -12,10 +12,16 @@ async function cleanup() {
 	}
 }
 
+// Handle async cleanup properly
+async function handleCleanup() {
+	await cleanup();
+	process.exit( 0 );
+}
+
 // Register cleanup on process exit
-// process.on( 'exit', cleanup ); // it gets triggered between sequential & parallel tests
-process.on( 'SIGINT', cleanup );
-process.on( 'SIGTERM', cleanup );
+process.on( 'exit', () => cleanup() ); // exit event doesn't support async, it gets triggered between sequential & parallel tests
+process.on( 'SIGINT', () => handleCleanup() );
+process.on( 'SIGTERM', () => handleCleanup() );
 
 async function globalSetup() {
 	const authFile = path.join( process.cwd(), 'auth.json' );

tests/e2e/tour.spec.js

@@ -13,22 +13,29 @@ test.describe( 'PRPL Tour', () => {
 		await tourButton.click();
 
 		// Wait for and verify the tour popover is visible
-		const tourPopover = page.locator( '.driver-popover' );
+		let tourPopover = page.locator( '.driver-popover' );
 		await expect( tourPopover ).toBeVisible();
 
 		// Get the number of steps from the window object
 		const numberOfSteps = await page.evaluate(
 			() => window.progressPlannerTour.steps.length
 		);
 
-		// Click the next button for each step
-		const nextButton = page.locator( '.driver-popover-next-btn' );
-		for ( let i = 0; i < numberOfSteps; i++ ) {
-			await nextButton.click();
-			// Verify the tour popover remains visible for each step
+		for ( let i = 0; i < numberOfSteps - 1; i++ ) {
+			tourPopover = page.locator( '.driver-popover' );
+
+			// Wait for the popover to be visible before interacting
 			await expect( tourPopover ).toBeVisible();
+
+			// Click the "Next" button if it's not the last step
+			if ( i < numberOfSteps - 1 ) {
+				const nextButton = page.locator( '.driver-popover-next-btn' );
+				await nextButton.click();
+			}
 		}
 
+		const nextButton = page.locator( '.driver-popover-next-btn' );
+
 		// Verify the button text changes to "Finish" on the last step
 		await expect( nextButton ).toHaveText( 'Finish' );