LPD-22188 LPD-24750 Clean up with test steps and await

gianmarcobrunialti · May 22, 2024 · d9a370a · d9a370a
1 parent 9921d6d
commit d9a370a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 8 deletions.
diff --git a/modules/test/playwright/pages/portal-search-admin-web/SearchAdminPage.ts b/modules/test/playwright/pages/portal-search-admin-web/SearchAdminPage.ts
@@ -56,15 +56,15 @@ export class SearchAdminPage {
 	}
 
 	async goToConnectionsTab() {
-		this.connectionsTab.click();
+		await this.connectionsTab.click();
 	}
 
 	async goToFieldMappingsTab() {
-		this.fieldMappingsTab.click();
+		await this.fieldMappingsTab.click();
 	}
 
 	async goToIndexActionsTab() {
-		this.indexActionsTab.click();
+		await this.indexActionsTab.click();
 	}
 
 	async reindexAllSearchIndexes() {

diff --git a/modules/test/playwright/tests/portal-search-admin-web/indexActions.spec.ts b/modules/test/playwright/tests/portal-search-admin-web/indexActions.spec.ts
@@ -13,12 +13,11 @@ export const test = mergeTests(loginTest(), searchAdminPageTest);
 test('should have a human-readable label for every index action', async ({
 	searchAdminPage,
 }) => {
+	await test.step('Navigate to Search Admin page', async () => {
+		await searchAdminPage.goto();
 
-	// Navigate to Search Admin page
-
-	await searchAdminPage.goto();
-
-	await searchAdminPage.goToIndexActionsTab();
+		await searchAdminPage.goToIndexActionsTab();
+	});
 
 	// Assert that the first two items would reindex all indexes and dictionaries