diff --git a/alfa-client/apps/admin-e2e/src/e2e/main-tests/benutzer_rollen/benutzer-anlegen.cy.ts b/alfa-client/apps/admin-e2e/src/e2e/main-tests/benutzer_rollen/benutzer-anlegen.cy.ts index 3d3ad6752448a0b229c8074610f0ae0c080b1738..43af53698b48a680377197bdcd7093c61fdd5911 100644 --- a/alfa-client/apps/admin-e2e/src/e2e/main-tests/benutzer_rollen/benutzer-anlegen.cy.ts +++ b/alfa-client/apps/admin-e2e/src/e2e/main-tests/benutzer_rollen/benutzer-anlegen.cy.ts @@ -64,66 +64,66 @@ describe('Benutzer anlegen', () => { benutzerVerifier.verifyUserNotInList(newUser.username); }); - describe('client validation errors', () => { - it('should show all if empty form', () => { + describe('when filling formular', () => { + it('should show validation errors on empty formular', () => { benutzerHelper.openNewBenutzerPage(); benutzerHelper.saveBenutzer(); - benutzerVerifier.verifyMissingVornameError(); - benutzerVerifier.verifyLastNameError(); - benutzerVerifier.verifyUsernameSizeError(); - benutzerVerifier.verifyEmailInvalidError(); - benutzerVerifier.verifyRollenError(); + benutzerVerifier.verifyEmptyVornameError(); + benutzerVerifier.verifyEmptyNameError(); + benutzerVerifier.verifyUsernameLengthError(); + benutzerVerifier.verifyInvalidEmailError(); + benutzerVerifier.verifyEmptyRollenError(); }); - it('should hide Vorname validation error', () => { + it('should hide Vorname validation error on filled', () => { benutzerHelper.editVorname('Max'); - benutzerVerifier.verifyNoFirstNameValidationError(); - benutzerVerifier.verifyLastNameError(); - benutzerVerifier.verifyUsernameSizeError(); - benutzerVerifier.verifyEmailInvalidError(); - benutzerVerifier.verifyRollenError(); + benutzerVerifier.verifyVornameHasNoError(); + benutzerVerifier.verifyEmptyNameError(); + benutzerVerifier.verifyUsernameLengthError(); + benutzerVerifier.verifyInvalidEmailError(); + benutzerVerifier.verifyEmptyRollenError(); }); - it('should hide Nachname validation error', () => { + it('should hide Nachname validation error on filled', () => { benutzerHelper.editNachname('Mustermann'); - benutzerVerifier.verifyNoFirstNameValidationError(); - benutzerVerifier.verifyNoLastNameValidationError(); - benutzerVerifier.verifyUsernameSizeError(); - benutzerVerifier.verifyEmailInvalidError(); - benutzerVerifier.verifyRollenError(); + benutzerVerifier.verifyVornameHasNoError(); + benutzerVerifier.verifyLastNameHasNoError(); + benutzerVerifier.verifyUsernameLengthError(); + benutzerVerifier.verifyInvalidEmailError(); + benutzerVerifier.verifyEmptyRollenError(); }); - it('should hide Benutzername validation error', () => { + it('should hide Benutzername validation error on filled', () => { benutzerHelper.editBenutzername('Max'); - benutzerVerifier.verifyNoFirstNameValidationError(); - benutzerVerifier.verifyNoLastNameValidationError(); - benutzerVerifier.verifyNoUsernameSizeValidationError(); - benutzerVerifier.verifyEmailInvalidError(); - benutzerVerifier.verifyRollenError(); + benutzerVerifier.verifyVornameHasNoError(); + benutzerVerifier.verifyLastNameHasNoError(); + benutzerVerifier.verifyUsernameHasNoError(); + benutzerVerifier.verifyInvalidEmailError(); + benutzerVerifier.verifyEmptyRollenError(); }); - it('should hide Email validation error', () => { + it('should hide Email validation error on filled', () => { benutzerHelper.editEmail('max@max.local'); - benutzerVerifier.verifyNoFirstNameValidationError(); - benutzerVerifier.verifyNoLastNameValidationError(); - benutzerVerifier.verifyNoUsernameSizeValidationError(); - benutzerVerifier.verifyNoEmailInvalidValidationError(); - benutzerVerifier.verifyRollenError(); + benutzerVerifier.verifyVornameHasNoError(); + benutzerVerifier.verifyLastNameHasNoError(); + benutzerVerifier.verifyUsernameHasNoError(); + benutzerVerifier.verifyEmailHasNoError(); + benutzerVerifier.verifyEmptyRollenError(); }); - it('should hide Rollen validation error', () => { + it('should hide Rollen validation error on filled', () => { benutzerHelper.addUserRole(); - benutzerVerifier.verifyNoFirstNameValidationError(); - benutzerVerifier.verifyNoLastNameValidationError(); - benutzerVerifier.verifyNoUsernameSizeValidationError(); - benutzerVerifier.verifyNoEmailInvalidValidationError(); - benutzerVerifier.verifyNoRollenValidationError(); + benutzerVerifier.verifyVornameHasNoError(); + benutzerVerifier.verifyLastNameHasNoError(); + benutzerVerifier.verifyUsernameHasNoError(); + benutzerVerifier.verifyEmailHasNoError(); + benutzerVerifier.verifyRollenHasNoError(); }); }); - describe('server validation errors', () => { - it('should create user', () => { + describe('after formular submission', () => { + it('should have user created', () => { benutzerHelper.openNewBenutzerPage(); benutzerHelper.addBenutzer(newUser); @@ -145,7 +145,7 @@ describe('Benutzer anlegen', () => { benutzerHelper.addBenutzer({ ...newUser, email: faker.internet.email() + '.local' }); benutzerHelper.saveBenutzer(); - benutzerVerifier.verifyUserExistsError(); + benutzerVerifier.verifyUsernameExistsError(); }); it('should remove benutzer', () => { diff --git a/alfa-client/apps/admin-e2e/src/helper/benutzer/benutzer.verifier.ts b/alfa-client/apps/admin-e2e/src/helper/benutzer/benutzer.verifier.ts index 0b12ee86799ba8def648795404a68fdab47775ff..5618af6a9f59e943b95a65632c9b1ce56056d0d8 100644 --- a/alfa-client/apps/admin-e2e/src/helper/benutzer/benutzer.verifier.ts +++ b/alfa-client/apps/admin-e2e/src/helper/benutzer/benutzer.verifier.ts @@ -70,27 +70,27 @@ export class E2EBenutzerVerifier { return this.benutzerListPage.getItem(userName); } - public verifyMissingVornameError(): void { + public verifyEmptyVornameError(): void { exist(this.benutzerPage.getVornameValidationError()); haveText(this.benutzerPage.getVornameValidationError(), BenutzerValidationMessages.VORNAME_EMPTY_ERROR); } - public verifyLastNameError(): void { + public verifyEmptyNameError(): void { exist(this.benutzerPage.getNachnameValidationError()); haveText(this.benutzerPage.getNachnameValidationError(), BenutzerValidationMessages.NACHNAME_EMPTY_ERROR); } - public verifyUsernameSizeError(): void { + public verifyUsernameLengthError(): void { exist(this.benutzerPage.getBenutzernameValidationError()); haveText(this.benutzerPage.getBenutzernameValidationError(), BenutzerValidationMessages.BENUTZERNAME_SIZE_ERROR); } - public verifyUserExistsError(): void { + public verifyUsernameExistsError(): void { exist(this.benutzerPage.getBenutzernameValidationError()); haveText(this.benutzerPage.getBenutzernameValidationError(), BenutzerValidationMessages.BENUTZER_NAME_EXISTS); } - public verifyEmailInvalidError(): void { + public verifyInvalidEmailError(): void { exist(this.benutzerPage.getEmailValidationError()); haveText(this.benutzerPage.getEmailValidationError(), BenutzerValidationMessages.EMAIL_INVALID_ERROR); } @@ -100,28 +100,28 @@ export class E2EBenutzerVerifier { haveText(this.benutzerPage.getEmailValidationError(), BenutzerValidationMessages.EMAIL_EXISTS); } - public verifyRollenError(): void { + public verifyEmptyRollenError(): void { exist(this.benutzerPage.getRollenValidationError()); contains(this.benutzerPage.getRollenValidationError(), BenutzerValidationMessages.ROLLEN_EMPTY_ERROR); } - public verifyNoFirstNameValidationError(): void { + public verifyVornameHasNoError(): void { haveText(this.benutzerPage.getVornameValidationError(), EMPTY_STRING); } - public verifyNoLastNameValidationError(): void { + public verifyLastNameHasNoError(): void { haveText(this.benutzerPage.getNachnameValidationError(), EMPTY_STRING); } - public verifyNoUsernameSizeValidationError(): void { + public verifyUsernameHasNoError(): void { haveText(this.benutzerPage.getBenutzernameValidationError(), EMPTY_STRING); } - public verifyNoEmailInvalidValidationError(): void { + public verifyEmailHasNoError(): void { haveText(this.benutzerPage.getEmailValidationError(), EMPTY_STRING); } - public verifyNoRollenValidationError(): void { + public verifyRollenHasNoError(): void { notContains(this.benutzerPage.getRollenValidationError(), BenutzerValidationMessages.ROLLEN_EMPTY_ERROR); } }