Make tests more stable.

Squidex · Sep 20, 2024 · c5b5be9 · c5b5be9
1 parent ce769da
commit c5b5be9
Show file tree

Hide file tree

Showing 6 changed files with 15 additions and 23 deletions.
diff --git a/tools/e2e/tests/given-app/_setup.ts b/tools/e2e/tests/given-app/_setup.ts
@@ -10,12 +10,7 @@ import { test as setup } from './../given-login/_fixture';
 
 setup('prepare app', async ({ appsPage }) => {
     const appName = `my-app-${getRandomId()}`;
-
-    await appsPage.goto();
-
-    const dialog = await appsPage.openAppDialog();
-    await dialog.enterName(appName);
-    await dialog.save();
+    await appsPage.createNewApp(appName);
 
     await appsPage.gotoApp(appName);
 

diff --git a/tools/e2e/tests/given-login/apps.spec.ts b/tools/e2e/tests/given-login/apps.spec.ts
@@ -20,6 +20,5 @@ test('create app', async ({ page, appsPage }) => {
     await appDialog.save();
 
     const newApp = page.getByRole('heading', { name: appName });
-
     await expect(newApp).toBeVisible();
 });
diff --git a/tools/e2e/tests/given-login/clients.spec.ts b/tools/e2e/tests/given-login/clients.spec.ts
@@ -13,11 +13,7 @@ test.beforeEach(async ({ context, appsPage, clientsPage }) => {
     await context.grantPermissions(['clipboard-read', 'clipboard-write']);
 
     const appName = `my-app-${getRandomId()}`;
-
-    await appsPage.goto();
-    const appDialog = await appsPage.openAppDialog();
-    await appDialog.enterName(appName);
-    await appDialog.save();
+    await appsPage.createNewApp(appName);
 
     await clientsPage.goto(appName);
 });

diff --git a/tools/e2e/tests/given-login/languages.spec.ts b/tools/e2e/tests/given-login/languages.spec.ts
@@ -11,11 +11,7 @@ import { expect, test } from './_fixture';
 
 test.beforeEach(async ({ languagesPage, appsPage }) => {
     const appName = `my-app-${getRandomId()}`;
-
-    await appsPage.goto();
-    const appDialog = await appsPage.openAppDialog();
-    await appDialog.enterName(appName);
-    await appDialog.save();
+    await appsPage.createNewApp(appName);
 
     await languagesPage.goto(appName);
 });

diff --git a/tools/e2e/tests/given-login/settings.spec.ts b/tools/e2e/tests/given-login/settings.spec.ts
@@ -10,11 +10,7 @@ import { getRandomId } from '../utils';
 
 test.beforeEach(async ({ appsPage, settingsPage }) => {
     const appName = `my-app-${getRandomId()}`;
-
-    await appsPage.goto();
-    const appDialog = await appsPage.openAppDialog();
-    await appDialog.enterName(appName);
-    await appDialog.save();
+    await appsPage.createNewApp(appName);
 
     await settingsPage.goto(appName);
 });

diff --git a/tools/e2e/tests/pages/apps.ts b/tools/e2e/tests/pages/apps.ts
@@ -5,7 +5,7 @@
  * Copyright (c) Squidex UG (haftungsbeschränkt). All rights reserved.
  */
 
-import { Page } from '@playwright/test';
+import { expect, Page } from '@playwright/test';
 
 export class AppsPage {
     constructor(private readonly page: Page) {}
@@ -23,6 +23,16 @@ export class AppsPage {
 
         return new AppDialog(this.page);
     }
+
+    public async createNewApp(appName: string) {
+        await this.goto();
+        const appDialog = await this.openAppDialog();
+        await appDialog.enterName(appName);
+        await appDialog.save();
+
+        const newApp = this.page.getByRole('heading', { name: appName });
+        await expect(newApp).toBeVisible();
+    }
 }
 
 class AppDialog {