E2E fix tests

alfa-client/apps/admin-e2e/src/components/benutzer/benutzer.e2e.component.ts

@@ -13,11 +13,16 @@ export class BenutzerE2EComponent {
   private readonly loeschenCheckbox: string = 'Loschen-checkbox-editor';
   private readonly userCheckbox: string = 'User-checkbox-editor';
   private readonly postCheckbox: string = 'Poststelle-checkbox-editor';
+  private readonly userRolesTable: string = 'User-roles-table';
 
   public getHinzufuegenButton(): Cypress.Chainable<Element> {
     return cy.getTestElement(this.benutzerHinzufuegenButton);
   }
 
+  public getUserRolesTable(): Cypress.Chainable<Element> {
+    return cy.getTestElement(this.userRolesTable);
+  }
+
   public addUser(): void {
     this.getHinzufuegenButton().click();
   }

alfa-client/apps/admin-e2e/src/e2e/main-tests/benutzer_rollen/benutzer_rollen.cy.ts

@@ -25,6 +25,10 @@ describe('Benutzer und Rollen', () => {
     exist(benutzerPage.getHinzufuegenButton());
   });
 
+  it('should show table with users and roles', () => {
+    exist(benutzerPage.getUserRolesTable());
+  });
+
   it('should show users and attributes in table', () => {
     exist(benutzerPage.getUserEntry('ariane'));
     benutzerPage.stringExistsInUserEntry(role1, 'dorothea');

alfa-client/apps/alfa-e2e/src/e2e/einheitlicher-ansprechpartner/vorgang-detail/vorgang-abschliessen.cy.ts

@@ -23,27 +23,13 @@
  */
 import { VorgangFormularButtonsE2EComponent } from 'apps/alfa-e2e/src/components/vorgang/vorgang-formular-buttons.e2e.components';
 import { loginAsEmil } from 'apps/alfa-e2e/src/support/user-util';
-import {
-  buildVorgang,
-  createVorgang,
-  initVorgaenge,
-  objectIds,
-} from 'apps/alfa-e2e/src/support/vorgang-util';
+import { buildVorgang, createVorgang, initVorgaenge, objectIds } from 'apps/alfa-e2e/src/support/vorgang-util';
 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 {
-  MainPage,
-  waitForSpinnerToDisappear,
-  waitforSpinnerToAppear,
-} from '../../../page-objects/main.po';
+import { VorgangE2E, VorgangMessagesE2E, VorgangStatusE2E, vorgangStatusLabelE2E } from '../../../model/vorgang';
+import { MainPage, waitForSpinnerToDisappear, waitforSpinnerToAppear } from '../../../page-objects/main.po';
 import { VorgangPage } from '../../../page-objects/vorgang.po';
-import { dropCollections } from '../../../support/cypress-helper';
+import { dropCollections, wait } from '../../../support/cypress-helper';
 import { contains, exist, haveText, notExist } from '../../../support/cypress.util';
 
 describe('Vorgang abschliessen', () => {
@@ -52,8 +38,7 @@ describe('Vorgang abschliessen', () => {
   const snackBar: SnackBarE2EComponent = mainPage.getSnackBar();
 
   const vorgangPage: VorgangPage = new VorgangPage();
-  const vorgangFormularButtons: VorgangFormularButtonsE2EComponent =
-    vorgangPage.getFormularButtons();
+  const vorgangFormularButtons: VorgangFormularButtonsE2EComponent = vorgangPage.getFormularButtons();
 
   const vorgangAbschliessen: VorgangE2E = {
     ...createVorgang(),
@@ -88,10 +73,7 @@ describe('Vorgang abschliessen', () => {
 
     describe('by button', () => {
       it('should have status Beschieden', () => {
-        haveText(
-          vorgangPage.getVorgangDetailHeader().getStatus(),
-          vorgangStatusLabelE2E[vorgangAbschliessen.status],
-        );
+        haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[vorgangAbschliessen.status]);
       });
 
       it('should show snackBar', () => {
@@ -108,10 +90,7 @@ describe('Vorgang abschliessen', () => {
       });
 
       it('should have status Abgeschlossen', () => {
-        haveText(
-          vorgangPage.getVorgangDetailHeader().getStatus(),
-          vorgangStatusLabelE2E[VorgangStatusE2E.ABGESCHLOSSEN],
-        );
+        haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.ABGESCHLOSSEN]);
       });
 
       it('back to vorgang list', () => {
@@ -142,10 +121,7 @@ describe('Vorgang abschliessen', () => {
 
     describe('by icon-button', () => {
       it('should have status Beschieden', () => {
-        haveText(
-          vorgangPage.getVorgangDetailHeader().getStatus(),
-          vorgangStatusLabelE2E[vorgangAbschliessenRevoke.status],
-        );
+        haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[vorgangAbschliessenRevoke.status]);
       });
 
       it('should show snackBar', () => {
@@ -157,6 +133,7 @@ describe('Vorgang abschliessen', () => {
 
       it('should close snackbar on revoke', () => {
         snackBar.getRevokeButton().click();
+        wait(500);
         waitForSpinnerToDisappear();
 
         notExist(snackBar.getMessage());
@@ -165,10 +142,7 @@ describe('Vorgang abschliessen', () => {
       it('should show status Beschieden after progressbar dissapear', () => {
         notExist(vorgangPage.getProgressBar());
 
-        haveText(
-          vorgangPage.getVorgangDetailHeader().getStatus(),
-          vorgangStatusLabelE2E[VorgangStatusE2E.IN_BEARBEITUNG],
-        );
+        haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.IN_BEARBEITUNG]);
       });
 
       it('back to vorgang list', () => {

alfa-client/apps/alfa-e2e/src/e2e/main-tests/vorgang-detailansicht/vorgang-wiedereroeffnen.cy.ts

@@ -24,23 +24,13 @@
 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, wait } 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 wiedereroeffnen', () => {
   const mainPage: MainPage = new MainPage();
@@ -48,8 +38,7 @@ describe('Vorgang wiedereroeffnen', () => {
   const snackbar: SnackBarE2EComponent = mainPage.getSnackBar();
 
   const vorgangPage: VorgangPage = new VorgangPage();
-  const vorgangFormularButtons: VorgangFormularButtonsE2EComponent =
-    vorgangPage.getFormularButtons();
+  const vorgangFormularButtons: VorgangFormularButtonsE2EComponent = vorgangPage.getFormularButtons();
 
   const vorgangBeschiedenWiedereroeffnen: VorgangE2E = {
     ...createVorgang(),
@@ -119,10 +108,7 @@ describe('Vorgang wiedereroeffnen', () => {
         });
 
         it('should have status In Bearbeitung', () => {
-          haveText(
-            vorgangPage.getVorgangDetailHeader().getStatus(),
-            vorgangStatusLabelE2E[VorgangStatusE2E.IN_BEARBEITUNG],
-          );
+          haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.IN_BEARBEITUNG]);
         });
 
         it('back to vorgang list', () => {
@@ -177,10 +163,7 @@ describe('Vorgang wiedereroeffnen', () => {
         it('should show status Beschieden', () => {
           wait(500);
 
-          haveText(
-            vorgangPage.getVorgangDetailHeader().getStatus(),
-            vorgangStatusLabelE2E[VorgangStatusE2E.BESCHIEDEN],
-          );
+          haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.BESCHIEDEN]);
         });
 
         it('back to vorgang list', () => {
@@ -232,10 +215,7 @@ describe('Vorgang wiedereroeffnen', () => {
         });
 
         it('should have status In Bearbeitung', () => {
-          haveText(
-            vorgangPage.getVorgangDetailHeader().getStatus(),
-            vorgangStatusLabelE2E[VorgangStatusE2E.IN_BEARBEITUNG],
-          );
+          haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.IN_BEARBEITUNG]);
         });
 
         it('back to vorgang list', () => {
@@ -280,6 +260,7 @@ describe('Vorgang wiedereroeffnen', () => {
 
         it('should close snackbar on revoke', () => {
           snackbar.getRevokeButton().click();
+          wait(500);
           waitForSpinnerToDisappear();
 
           notExist(snackbar.getMessage());
@@ -288,10 +269,7 @@ describe('Vorgang wiedereroeffnen', () => {
         it('should show status Abgeschlossen', () => {
           notExist(vorgangPage.getProgressBar());
 
-          haveText(
-            vorgangPage.getVorgangDetailHeader().getStatus(),
-            vorgangStatusLabelE2E[VorgangStatusE2E.ABGESCHLOSSEN],
-          );
+          haveText(vorgangPage.getVorgangDetailHeader().getStatus(), vorgangStatusLabelE2E[VorgangStatusE2E.ABGESCHLOSSEN]);
         });
 
         it('back to vorgang list', () => {

alfa-client/libs/admin/settings/src/lib/users-roles/users-roles.component.html

@@ -5,7 +5,7 @@
   dataTestId="Add-user-button"
   (clickEmitter)="navigateToAddUser()"
 />
-<ods-list *ngIf="users$ | async as users">
+<ods-list data-test-id="User-roles-table" *ngIf="users$ | async as users">
   <ods-list-item *ngFor="let user of users.resource" [path]="user.username" [attr.data-test-id]="'User-entry-' + user.username">
     <div class="flex-1 basis-1/2">
       <div class="mb-2 flex flex-wrap items-center gap-3">