Merge branch 'main' into SPSH-1481-fixTests

base/api/testHelperPerson.page.ts

@@ -6,7 +6,7 @@ import { HeaderPage } from '../../pages/Header.page';
 import { LoginPage } from '../../pages/LoginView.page';
 import { faker } from '@faker-js/faker';
 import { lehrkraftOeffentlichRolle } from '../roles';
-import { generateLehrerNachname, generateLehrerVorname, generateKopersNr } from "../testHelperGenerateTestdataNames";
+import { generateNachname, generateVorname, generateKopersNr } from "../testHelperGenerateTestdataNames";
 import { testschule } from "../organisation";
 
 const FRONTEND_URL: string | undefined = process.env.FRONTEND_URL || "";
@@ -99,7 +99,10 @@ export async function getPersonId(page: Page, Benutzername: string): Promise<str
 export async function createTeacherAndLogin(page: Page) {
     const header = new HeaderPage(page);
     const login: LoginPage = new LoginPage(page);
+    const vorname = await generateVorname();
+    const nachname = await generateNachname();
     const organisation = testschule;
+    const kopersNr = '0815' + faker.string.numeric({ length: 3 });
 
     const userInfo: UserInfo = await createPersonWithUserContext(page, organisation, await generateLehrerNachname(), await generateLehrerVorname(), lehrkraftOeffentlichRolle, await generateKopersNr());
     await header.logout();

base/organisation.ts

@@ -1,2 +1,3 @@
 export const testschule = "Testschule Schulportal";
-export const testschule665 = "Testschule-PW665";
+export const testschule665 = "Testschule-PW665";
+export const landSH = "Land Schleswig-Holstein";

base/rollentypen.ts

@@ -1 +1,2 @@
-export const typelehrer = "LEHR";
+export const typelehrer = "LEHR";
+export const typeSchueler = "LERN";

base/sp.ts

@@ -1 +1,2 @@
-export const email = "E-Mail";
+export const email = "E-Mail";
+export const itslearning = "itslearning";

base/testHelperGenerateTestdataNames.ts

@@ -1,12 +1,12 @@
 import { faker } from "@faker-js/faker/locale/de";
 import { generateRandomString, CharacterSetType } from "ts-randomstring/lib/index.js"
 
-export async function generateLehrerVorname(){  
-  return "TAuto-PW-V-" + faker.person.firstName() + generateRandomString({length: 3,charSetType: CharacterSetType.Alphabetic});
+export async function generateVorname(){  
+  return "TAuto-PW-V-" + faker.person.firstName() + generateRandomString({length: 3,charSetType: CharacterSetType.Alphabetic})
 }
 
 export async function generateLehrerNachname(){  
-  return "TAuto-PW-N-" + faker.person.lastName() + generateRandomString({length: 3,charSetType: CharacterSetType.Alphabetic});
+  return "TAuto-PW-N-" + faker.person.lastName() + generateRandomString({length: 3,charSetType: CharacterSetType.Alphabetic})
 }
 
 export async function generateRolleName(){  

pages/LoginView.page.ts

@@ -43,7 +43,7 @@ export class LoginPage {
     await this.button_login.click();
     // BE requests laufen zeitverzögert zum FE; dieses muss im FE behoben werden; solange dies nicht der Fall ist, brauchen wir diesen workaround
     // Wenn auf login/logout geklickt wird, sind teilweise noch requests am laufen
-    await this.page.waitForTimeout(1000); 
+    await this.page.waitForTimeout(1000);
 
     return new StartPage(this.page);
   }

pages/ProfileView.page.ts

@@ -85,6 +85,5 @@ export class ProfilePage{
          this.cardHeadline_2FA = page.getByTestId('two-factor-card');
          this.text_no2FA = page.getByText('Es wurde noch kein zweiter Faktor für Sie eingerichtet.');
          this.button_2FAEinrichten = page.getByTestId('open-2FA-self-service-dialog-icon');
-
     }
 }

pages/admin/PersonDetailsView.page.ts

@@ -5,6 +5,7 @@ export class PersonDetailsViewPage{
     readonly text_h2_benutzerBearbeiten: Locator;
 
     // Passwort
+    readonly text_h3_passwort_headline: Locator;
     readonly button_pwChange: Locator;
     readonly button_pwReset: Locator;
     readonly text_pwResetInfo: Locator; 
@@ -17,6 +18,7 @@ export class PersonDetailsViewPage{
     readonly button_closeDeletePersonConfirm: Locator;
 
     // Schulzuordnungen
+    readonly text_h3_schulzuordnung_headline: Locator;
     readonly button_editSchulzuordnung: Locator;
     readonly button_addSchulzuordnung: Locator;
     readonly combobox_organisation: Locator;
@@ -27,6 +29,9 @@ export class PersonDetailsViewPage{
     readonly button_confirmAddSchulzuordnung: Locator;
     readonly button_saveAssignmentChanges: Locator;
     readonly button_closeSaveAssignmentChanges: Locator;
+
+    // Benutzer sperren
+    readonly text_h3_lockPerson_headline: Locator;
     readonly button_lockPerson: Locator;
     readonly button_lockPersonConfirm: Locator;
     readonly text_h2_dialogBenutzerSperren: Locator;
@@ -38,12 +43,29 @@ export class PersonDetailsViewPage{
     readonly radio_button_befristet: Locator;
     readonly text_sperrdatumAb: Locator;
     readonly text_sperrdatumBis: Locator;
+
+     // 2FA
+     readonly text_h3_2FA: Locator;
+     readonly text_token_IstEingerichtet_info: Locator;
+     readonly text_neuen_token_einrichten_info: Locator;
+     readonly text_kein_token_ist_Eingerichtet: Locator;
+     readonly button_2FAEinrichten: Locator;
+     readonly text_2FA_info: Locator;
+     readonly dialog_2FA_Einrichten: Locator;
+     readonly text_h2_2FA_cardheadline: Locator;
+     readonly selectOption_2FA_softwareToken: Locator;
+     readonly text_2FA_softwareToken_info: Locator;
+     readonly button_2FA_Einrichten_Weiter: Locator;
+     readonly button_close_softwareToken_dialog: Locator;
+     readonly button_2FA_Zuruecksetzen_Weiter: Locator;
+
 
     constructor(page){
         this.page = page;  
         this.text_h2_benutzerBearbeiten = page.getByTestId('layout-card-headline');
 
         // Passwort
+        this.text_h3_passwort_headline = page.locator( `//h3[text()='Passwort']`);
         this.button_pwChange = page.getByTestId('open-password-reset-dialog-icon');     
         this.button_pwReset = page.getByTestId('password-reset-button');
         this.text_pwResetInfo = page.getByTestId('password-reset-info-text');
@@ -56,6 +78,7 @@ export class PersonDetailsViewPage{
         this.button_closeDeletePersonConfirm = page.getByTestId('close-person-delete-success-dialog-button');
 
         // Schulzuordnungen
+        this.text_h3_schulzuordnung_headline = page.getByText('Schulzuordnung(en)');
         this.button_editSchulzuordnung = page.locator('div').filter({ hasText: /^Schulzuordnung\(en\)Bearbeiten$/ }).getByTestId('zuordnung-edit-button');
         this.button_addSchulzuordnung = page.getByTestId('zuordnung-create-button');
         this.combobox_organisationDialogBenutzerSperren = page.getByTestId('person-lock-card').locator('.v-field__input');
@@ -66,6 +89,9 @@ export class PersonDetailsViewPage{
         this.button_confirmAddSchulzuordnung = page.getByRole('button', { name: 'Ja' });
         this.button_saveAssignmentChanges = page.getByTestId('zuordnung-changes-save');
         this.button_closeSaveAssignmentChanges = page.getByRole('dialog').getByRole('button', { name: 'Schließen' });
+
+        // Benutzer sperren
+        this.text_h3_lockPerson_headline = page.getByTestId('person-lock-info').getByText('Status');
         this.button_lockPerson = page.getByTestId('open-lock-dialog-button');
         this.button_lockPersonConfirm = page.getByTestId('lock-user-button');
         this.text_h2_dialogBenutzerSperren = page.getByTestId('person-lock-card').getByTestId('layout-card-headline');
@@ -77,6 +103,20 @@ export class PersonDetailsViewPage{
         this.radio_button_befristet = page.getByTestId('befristet-radio-button').getByLabel('Befristet');
         this.text_sperrdatumAb = page.getByTestId('lock-info-1-attribute');
         this.text_sperrdatumBis = page.getByTestId('lock-info-2-attribute');
+
+        //2FA
+        this.text_h3_2FA = page.getByText('Zwei-Faktor-Authentifizierung (2FA)');
+        this.text_kein_token_ist_Eingerichtet = page.getByText('Für diesen Benutzer ist aktuell keine 2FA eingerichtet.');
+        this.text_token_IstEingerichtet_info = page.getByText('Für diesen Benutzer ist aktuell ein Software-Token eingerichtet.');
+        this.text_neuen_token_einrichten_info = page.getByText('Um einen neuen Token einzurichten, muss der aktuelle Token durch die schulischen Administratorinnen und Administratoren zurückgesetzt werden.');
+        this.button_2FAEinrichten = page.getByTestId('open-2FA-dialog-icon');
+        this.dialog_2FA_Einrichten = page.getByTestId('two-factor-authentication-dialog');
+        this.text_h2_2FA_cardheadline = this.dialog_2FA_Einrichten.getByTestId('layout-card-headline');
+        this.text_2FA_softwareToken_info = page.getByText('Ein QR-Code wird generiert, welcher direkt eingescannt oder ausgedruckt werden kann.');
+        this.selectOption_2FA_softwareToken = page.getByTestId('software-token-radio-button');
+        this.button_2FA_Einrichten_Weiter = page.getByTestId('proceed-two-factor-authentication-dialog-button');
+        this.button_close_softwareToken_dialog = page.getByTestId('close-software-token-dialog-button');
+        this.button_2FA_Zuruecksetzen_Weiter = page.getByTestId('two-way-authentification-set-up-button');
     }
 
     public async lockUserWithoutDate() {
@@ -111,4 +151,16 @@ export class PersonDetailsViewPage{
     public async checkLockDateTo(lockDateTo: string) {
         await expect(this.text_sperrdatumBis).toHaveText(lockDateTo);
     }
+
+    public async softwareTokenEinrichten() {
+        await this.button_2FAEinrichten.click();
+        await expect(this.text_h2_2FA_cardheadline).toHaveText('Zwei-Faktor-Authentifizierung einrichten');
+        await expect(this.selectOption_2FA_softwareToken).toHaveText('Software-Token einrichten');
+        await expect(this.text_2FA_softwareToken_info).toBeVisible();
+        await this.button_2FA_Einrichten_Weiter.click();
+
+        await expect(this.text_h2_2FA_cardheadline).toHaveText('Software-Token einrichten');
+        await this.button_close_softwareToken_dialog.click();
+    }
+
 }

tests/Person.spec.ts

@@ -15,7 +15,7 @@ import { addSystemrechtToRolle } from "../base/api/testHelperRolle.page";
 import { LONG, SHORT, STAGE, BROWSER } from "../base/tags";
 import { deletePersonByUsername, deleteRolleById, deleteRolleByName } from "../base/testHelperDeleteTestdata.ts";
 import { landesadminRolle, schuelerRolle, schuladminOeffentlichRolle } from "../base/roles.ts";
-import { generateLehrerNachname, generateLehrerVorname, generateRolleName } from "../base/testHelperGenerateTestdataNames.ts";
+import { generateNachname, generateVorname, generateRolleName } from "../base/testHelperGenerateTestdataNames.ts";
 import { testschule665 } from "../base/organisation.ts";
 import { gotoTargetURL } from "../base/testHelperUtils.ts";
 
@@ -706,8 +706,8 @@ test.describe(`Testfälle für die Administration von Personen": Umgebung: ${pro
     const PersonDetailsView = new PersonDetailsViewPage(page);
     const header = new HeaderPage(page);
 
-    const vorname = await generateLehrerVorname();
-    const nachname = await generateLehrerNachname();
+    const vorname = await generateVorname();
+    const nachname = await generateNachname();
     const rolle = await generateRolleName();
     const berechtigung = 'SYSADMIN';
     const idSP = await getSPId(page, 'Schulportal-Administration');