From ff8c51d9499a87ef5f50ff8d7154fa94527268cd Mon Sep 17 00:00:00 2001
From: OZGCloud <ozgcloud@mgm-tp.com>
Date: Tue, 19 Nov 2024 10:10:21 +0100
Subject: [PATCH] implement 3 waits to make tests more robust
---
.../vorgang-bescheiden.cy.ts | 38 ++++-----------
.../vorgang-verwerfen.cy.ts | 48 ++++---------------
.../vorgang-zurueckholen.cy.ts | 43 ++++-------------
3 files changed, 27 insertions(+), 102 deletions(-)
diff --git a/alfa-client/apps/alfa-e2e/src/e2e/main-tests/vorgang-detailansicht/vorgang-bescheiden.cy.ts b/alfa-client/apps/alfa-e2e/src/e2e/main-tests/vorgang-detailansicht/vorgang-bescheiden.cy.ts
index f735f51103..cb1f62b6ce 100644
--- a/alfa-client/apps/alfa-e2e/src/e2e/main-tests/vorgang-detailansicht/vorgang-bescheiden.cy.ts
+++ b/alfa-client/apps/alfa-e2e/src/e2e/main-tests/vorgang-detailansicht/vorgang-bescheiden.cy.ts
@@ -25,23 +25,13 @@ import { VorgangBescheidWizardE2EComponent } from 'apps/alfa-e2e/src/components/
import { VorgangFormularButtonsE2EComponent } from 'apps/alfa-e2e/src/components/vorgang/vorgang-formular-buttons.e2e.components';
import { SnackBarE2EComponent } from '../../../components/ui/snackbar.e2e.component';
import { VorgangListE2EComponent } from '../../../components/vorgang/vorgang-list.e2e.component';
-import {
- VorgangE2E,
- VorgangMessagesE2E,
- VorgangStatusE2E,
- vorgangStatusLabelE2E,
-} from '../../../model/vorgang';
+import { VorgangE2E, VorgangMessagesE2E, VorgangStatusE2E, vorgangStatusLabelE2E } from '../../../model/vorgang';
import { MainPage, waitForSpinnerToDisappear } from '../../../page-objects/main.po';
import { VorgangPage } from '../../../page-objects/vorgang.po';
import { dropCollections } from '../../../support/cypress-helper';
import { contains, exist, haveText, notExist } from '../../../support/cypress.util';
import { loginAsSabine } from '../../../support/user-util';
-import {
- buildVorgang,
- createVorgang,
- initVorgaenge,
- objectIds,
-} from '../../../support/vorgang-util';
+import { buildVorgang, createVorgang, initVorgaenge, objectIds } from '../../../support/vorgang-util';
describe('Vorgang bescheiden', () => {
const mainPage: MainPage = new MainPage();
@@ -49,8 +39,7 @@ describe('Vorgang bescheiden', () => {
const snackBar: SnackBarE2EComponent = mainPage.getSnackBar();
const vorgangPage: VorgangPage = new VorgangPage();
- const vorgangFormularButtons: VorgangFormularButtonsE2EComponent =
- vorgangPage.getFormularButtons();
+ const vorgangFormularButtons: VorgangFormularButtonsE2EComponent = vorgangPage.getFormularButtons();
const vorgangBescheiden: VorgangE2E = {
...createVorgang(),
@@ -87,10 +76,7 @@ describe('Vorgang bescheiden', () => {
describe('by button', () => {
it('should have status In Bearbeitung', () => {
- haveText(
- vorgangPage.getVorgangDetailHeader().getStatus(),
- vorgangStatusLabelE2E[vorgangBescheiden.status],
- );
+ haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[vorgangBescheiden.status]);
});
it('should show snackBar message', () => {
@@ -113,10 +99,7 @@ describe('Vorgang bescheiden', () => {
});
it('should have status Abgeschlossen', () => {
- haveText(
- vorgangPage.getVorgangDetailHeader().getStatus(),
- vorgangStatusLabelE2E[VorgangStatusE2E.ABGESCHLOSSEN],
- );
+ haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.ABGESCHLOSSEN]);
});
it('back to vorgang list', () => {
@@ -147,10 +130,7 @@ describe('Vorgang bescheiden', () => {
describe('by icon-button', () => {
it('should have status In Bearbeitung', () => {
- haveText(
- vorgangPage.getVorgangDetailHeader().getStatus(),
- vorgangStatusLabelE2E[vorgangBescheidenRevoke.status],
- );
+ haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[vorgangBescheidenRevoke.status]);
});
it('should show snackBar message', () => {
@@ -175,11 +155,9 @@ describe('Vorgang bescheiden', () => {
it('should show status In Bearbeitung', () => {
notExist(vorgangPage.getProgressBar());
+ waitForSpinnerToDisappear();
- haveText(
- vorgangPage.getVorgangDetailHeader().getStatus(),
- vorgangStatusLabelE2E[VorgangStatusE2E.IN_BEARBEITUNG],
- );
+ haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.IN_BEARBEITUNG]);
});
it('back to vorgang list', () => {
diff --git a/alfa-client/apps/alfa-e2e/src/e2e/main-tests/vorgang-detailansicht/vorgang-verwerfen.cy.ts b/alfa-client/apps/alfa-e2e/src/e2e/main-tests/vorgang-detailansicht/vorgang-verwerfen.cy.ts
index 08eb091172..9763fe29a9 100644
--- a/alfa-client/apps/alfa-e2e/src/e2e/main-tests/vorgang-detailansicht/vorgang-verwerfen.cy.ts
+++ b/alfa-client/apps/alfa-e2e/src/e2e/main-tests/vorgang-detailansicht/vorgang-verwerfen.cy.ts
@@ -27,23 +27,13 @@ import localeDeExtra from '@angular/common/locales/extra/de';
import { VorgangFormularButtonsE2EComponent } from 'apps/alfa-e2e/src/components/vorgang/vorgang-formular-buttons.e2e.components';
import { SnackBarE2EComponent } from '../../../components/ui/snackbar.e2e.component';
import { VorgangListE2EComponent } from '../../../components/vorgang/vorgang-list.e2e.component';
-import {
- VorgangE2E,
- VorgangMessagesE2E,
- VorgangStatusE2E,
- vorgangStatusLabelE2E,
-} from '../../../model/vorgang';
+import { VorgangE2E, VorgangMessagesE2E, VorgangStatusE2E, vorgangStatusLabelE2E } from '../../../model/vorgang';
import { MainPage, waitForSpinnerToDisappear } from '../../../page-objects/main.po';
import { VorgangPage } from '../../../page-objects/vorgang.po';
import { dropCollections } from '../../../support/cypress-helper';
import { contains, exist, haveText, notExist } from '../../../support/cypress.util';
import { loginAsSabine } from '../../../support/user-util';
-import {
- buildVorgang,
- createVorgang,
- initVorgaenge,
- objectIds,
-} from '../../../support/vorgang-util';
+import { buildVorgang, createVorgang, initVorgaenge, objectIds } from '../../../support/vorgang-util';
registerLocaleData(localeDe, 'de', localeDeExtra);
@@ -53,8 +43,7 @@ describe('Vorgang Verwerfen', () => {
const snackbar: SnackBarE2EComponent = mainPage.getSnackBar();
const vorgangPage: VorgangPage = new VorgangPage();
- const vorgangFormularButtons: VorgangFormularButtonsE2EComponent =
- vorgangPage.getFormularButtons();
+ const vorgangFormularButtons: VorgangFormularButtonsE2EComponent = vorgangPage.getFormularButtons();
const vorgangVerwerfen: VorgangE2E = { ...createVorgang(), name: 'DoVerwerfen' };
const vorgangVerwerfenRevoke: VorgangE2E = { ...buildVorgang(objectIds[0], 'DoRevokeVerwerfen') };
@@ -82,10 +71,7 @@ describe('Vorgang Verwerfen', () => {
describe('by button', () => {
it('should have status Neu', () => {
- haveText(
- vorgangPage.getVorgangDetailHeader().getStatus(),
- vorgangStatusLabelE2E[vorgangVerwerfen.status],
- );
+ haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[vorgangVerwerfen.status]);
});
it('should show snackBar', () => {
@@ -103,10 +89,7 @@ describe('Vorgang Verwerfen', () => {
});
it('should have status Verworfen', () => {
- haveText(
- vorgangPage.getVorgangDetailHeader().getStatus(),
- vorgangStatusLabelE2E[VorgangStatusE2E.VERWORFEN],
- );
+ haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.VERWORFEN]);
});
it('should have button zurückholen', () => {
@@ -122,10 +105,7 @@ describe('Vorgang Verwerfen', () => {
});
it('should have status Verworfen', () => {
- haveText(
- vorgangList.getListItem(vorgangVerwerfen.name).getStatus(),
- vorgangStatusLabelE2E[VorgangStatusE2E.VERWORFEN],
- );
+ haveText(vorgangList.getListItem(vorgangVerwerfen.name).getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.VERWORFEN]);
});
});
});
@@ -140,10 +120,7 @@ describe('Vorgang Verwerfen', () => {
describe('by icon-button', () => {
it('should have status Neu', () => {
- haveText(
- vorgangPage.getVorgangDetailHeader().getStatus(),
- vorgangStatusLabelE2E[vorgangVerwerfenRevoke.status],
- );
+ haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[vorgangVerwerfenRevoke.status]);
});
it('should show snackBar', () => {
@@ -163,11 +140,9 @@ describe('Vorgang Verwerfen', () => {
it('should show status Neu', () => {
notExist(vorgangPage.getProgressBar());
+ waitForSpinnerToDisappear();
- haveText(
- vorgangPage.getVorgangDetailHeader().getStatus(),
- vorgangStatusLabelE2E[VorgangStatusE2E.NEU],
- );
+ haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.NEU]);
});
it('should not have button zurückholen', () => {
@@ -183,10 +158,7 @@ describe('Vorgang Verwerfen', () => {
});
it('should have status Neu', () => {
- haveText(
- vorgangList.getListItem(vorgangVerwerfenRevoke.name).getStatus(),
- vorgangStatusLabelE2E[VorgangStatusE2E.NEU],
- );
+ haveText(vorgangList.getListItem(vorgangVerwerfenRevoke.name).getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.NEU]);
});
});
});
diff --git a/alfa-client/apps/alfa-e2e/src/e2e/main-tests/vorgang-detailansicht/vorgang-zurueckholen.cy.ts b/alfa-client/apps/alfa-e2e/src/e2e/main-tests/vorgang-detailansicht/vorgang-zurueckholen.cy.ts
index 4c9784b5de..fb1ae0fe41 100644
--- a/alfa-client/apps/alfa-e2e/src/e2e/main-tests/vorgang-detailansicht/vorgang-zurueckholen.cy.ts
+++ b/alfa-client/apps/alfa-e2e/src/e2e/main-tests/vorgang-detailansicht/vorgang-zurueckholen.cy.ts
@@ -27,23 +27,13 @@ import localeDeExtra from '@angular/common/locales/extra/de';
import { VorgangFormularButtonsE2EComponent } from 'apps/alfa-e2e/src/components/vorgang/vorgang-formular-buttons.e2e.components';
import { SnackBarE2EComponent } from '../../../components/ui/snackbar.e2e.component';
import { VorgangListE2EComponent } from '../../../components/vorgang/vorgang-list.e2e.component';
-import {
- VorgangE2E,
- VorgangMessagesE2E,
- VorgangStatusE2E,
- vorgangStatusLabelE2E,
-} from '../../../model/vorgang';
+import { VorgangE2E, VorgangMessagesE2E, VorgangStatusE2E, vorgangStatusLabelE2E } from '../../../model/vorgang';
import { MainPage, waitForSpinnerToDisappear } from '../../../page-objects/main.po';
import { VorgangPage } from '../../../page-objects/vorgang.po';
import { dropCollections } from '../../../support/cypress-helper';
import { contains, exist, haveText, notExist } from '../../../support/cypress.util';
import { loginAsSabine } from '../../../support/user-util';
-import {
- buildVorgang,
- createVorgang,
- initVorgaenge,
- objectIds,
-} from '../../../support/vorgang-util';
+import { buildVorgang, createVorgang, initVorgaenge, objectIds } from '../../../support/vorgang-util';
registerLocaleData(localeDe, 'de', localeDeExtra);
@@ -53,8 +43,7 @@ describe('Vorgang Zurueckholen', () => {
const snackBar: SnackBarE2EComponent = mainPage.getSnackBar();
const vorgangPage: VorgangPage = new VorgangPage();
- const vorgangFormularButtons: VorgangFormularButtonsE2EComponent =
- vorgangPage.getFormularButtons();
+ const vorgangFormularButtons: VorgangFormularButtonsE2EComponent = vorgangPage.getFormularButtons();
const vorgangZurueckholen: VorgangE2E = {
...createVorgang(),
@@ -89,10 +78,7 @@ describe('Vorgang Zurueckholen', () => {
describe('by button', () => {
it('should have status Verworfen', () => {
- haveText(
- vorgangPage.getVorgangDetailHeader().getStatus(),
- vorgangStatusLabelE2E[vorgangZurueckholen.status],
- );
+ haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[vorgangZurueckholen.status]);
});
it('should show snackBar message', () => {
@@ -110,10 +96,7 @@ describe('Vorgang Zurueckholen', () => {
});
it('should have status Neu', () => {
- haveText(
- vorgangPage.getVorgangDetailHeader().getStatus(),
- vorgangStatusLabelE2E[VorgangStatusE2E.NEU],
- );
+ haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.NEU]);
});
it('should have button annehmen und verwerfen', () => {
@@ -128,10 +111,7 @@ describe('Vorgang Zurueckholen', () => {
});
it('should have status Neu', () => {
- haveText(
- vorgangList.getListItem(vorgangZurueckholen.name).getStatus(),
- vorgangStatusLabelE2E[VorgangStatusE2E.NEU],
- );
+ haveText(vorgangList.getListItem(vorgangZurueckholen.name).getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.NEU]);
});
});
});
@@ -146,10 +126,7 @@ describe('Vorgang Zurueckholen', () => {
describe('by icon-button', () => {
it('should have status Neu', () => {
- haveText(
- vorgangPage.getVorgangDetailHeader().getStatus(),
- vorgangStatusLabelE2E[vorgangZurueckholenRevoke.status],
- );
+ haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[vorgangZurueckholenRevoke.status]);
});
it('should show snackBar', () => {
@@ -169,11 +146,9 @@ describe('Vorgang Zurueckholen', () => {
it('should show status Abgeschlossen', () => {
notExist(vorgangPage.getProgressBar());
+ waitForSpinnerToDisappear();
- haveText(
- vorgangPage.getVorgangDetailHeader().getStatus(),
- vorgangStatusLabelE2E[VorgangStatusE2E.VERWORFEN],
- );
+ haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.VERWORFEN]);
});
it('should have button zurueckholen', () => {
--
GitLab