Skip to content

Commit

Permalink
SPSH-1359: Fixed the issue with the rollenName when the test once fai…
Browse files Browse the repository at this point in the history 
…led. (#97)

Co-authored-by: Youssef Bouchara <[email protected]>
  • Loading branch information
@phaelcg @YoussefBouch
phaelcg and YoussefBouch authored Nov 20, 2024
1 parent 1e9bf76 commit 0625c84
Show file tree
Hide file tree
Showing 7 changed files with 24 additions and 19 deletions.
2 changes: 1 addition & 1 deletion base/testHelperGenerateTestdataNames.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ export async function generateLehrerNachname(){
return "TAuto-PW-N-" + faker.person.lastName();
}

export async function generateRolleName(){
export function generateRolleName(){
return "TAuto-PW-R-" + faker.lorem.word({ length: { min: 8, max: 12 }});
}

Expand Down
7 changes: 4 additions & 3 deletions tests/Person.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +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 { generateRolleName } from "../base/testHelperGenerateTestdataNames.ts";

const PW: string | undefined = process.env.PW;
const ADMIN: string | undefined = process.env.USER;
Expand Down Expand Up @@ -229,7 +230,7 @@ test.describe(`Testfälle für die Administration von Personen": Umgebung: ${pro
// Step 1: Create a Schuladmin as Landesadmin and login as the newly created Schuladmin user
await test.step(`Schuladmin anlegen und mit diesem anmelden`, async () => {
const idSP = await getSPId(page, 'Schulportal-Administration');
userInfo = await createRolleAndPersonWithUserContext(page, schulstrukturknoten, 'LEIT', nachname, vorname, idSP, 'TAuto-PW-E-RolleLEIT');
userInfo = await createRolleAndPersonWithUserContext(page, schulstrukturknoten, 'LEIT', nachname, vorname, idSP, generateRolleName());
await addSystemrechtToRolle(page, userInfo.rolleId, 'PERSONEN_VERWALTEN');
await addSystemrechtToRolle(page, userInfo.rolleId, 'PERSONEN_ANLEGEN');

Expand Down Expand Up @@ -518,7 +519,7 @@ test.describe(`Testfälle für die Administration von Personen": Umgebung: ${pro

await test.step(`Testdaten: Landesadmin anlegen und mit diesem anmelden`, async () => {
const idSP = await getSPId(page, 'Schulportal-Administration');
userInfo = await createRolleAndPersonWithUserContext(page, 'Land Schleswig-Holstein', 'SYSADMIN', 'TAuto-PW-B-Master', 'TAuto-PW-B-Hans', idSP, 'TAuto-PW-R-RolleSYSADMIN');
userInfo = await createRolleAndPersonWithUserContext(page, 'Land Schleswig-Holstein', 'SYSADMIN', 'TAuto-PW-B-Master', 'TAuto-PW-B-Hans', idSP, generateRolleName());
await addSystemrechtToRolle(page, userInfo.rolleId, 'ROLLEN_VERWALTEN');
await addSystemrechtToRolle(page, userInfo.rolleId, 'PERSONEN_SOFORT_LOESCHEN');
await addSystemrechtToRolle(page, userInfo.rolleId, 'PERSONEN_VERWALTEN');
Expand Down Expand Up @@ -680,7 +681,7 @@ test.describe(`Testfälle für die Administration von Personen": Umgebung: ${pro

const vorname = "TAuto-PW-V-" + faker.person.firstName();
const nachname = "TAuto-PW-N-" + faker.person.lastName();
const rolle = "TAuto-PW-R-" + faker.lorem.word({ length: { min: 8, max: 12 }});
const rolle = generateRolleName();
const berechtigung = 'SYSADMIN';
const idSP = await getSPId(page, 'Schulportal-Administration');

Expand Down
8 changes: 4 additions & 4 deletions tests/PersonBearbeiten.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -81,15 +81,15 @@ test.describe(`Testfälle für die Administration von Personen": Umgebung: ${pro

const addminVorname = "TAuto-PW-V-" + faker.person.firstName();
const adminNachname = "TAuto-PW-N-" + faker.person.lastName();
const adminRolle = "TAuto-PW-LEIT-" + faker.lorem.word({length: {min: 8, max: 12}});
const adminRolle = generateRolleName();
const adminRollenart = 'LEIT';
const adminOrganisation = 'Testschule-PW665';
const adminIdSP = await getSPId(page, 'Schulportal-Administration');
let userInfoAdmin: UserInfo;

const lehrerVorname = "TAuto-PW-V-" + faker.person.firstName();
const lehrerNachname = "TAuto-PW-N-" + faker.person.lastName();
const lehrerRolle = "TAuto-PW-LEHR-" + faker.lorem.word({length: {min: 8, max: 12}});
const lehrerRolle = generateRolleName();
const lehrerRollenart = 'LEHR';
const lehrerOrganisation = 'Testschule-PW665';
const lehrerIdSP = await getSPId(page, 'E-Mail');
Expand Down Expand Up @@ -149,7 +149,7 @@ test.describe(`Testfälle für die Administration von Personen": Umgebung: ${pro
const sperrDatumAb = await generateDateToday() // Konkrete Testdaten für diesen Testfall

await test.step(`Testdaten: Lehrer mit einer Rolle(LEHR) und SP(email) über die api anlegen ${ADMIN}`, async () => {
userInfoLehrer = await createRolleAndPersonWithUserContext(page, testschule, typelehrer, await generateLehrerNachname(), await generateLehrerVorname(), await getSPId(page, email), await generateRolleName());
userInfoLehrer = await createRolleAndPersonWithUserContext(page, testschule, typelehrer, await generateLehrerNachname(), await generateLehrerVorname(), await getSPId(page, email), generateRolleName());
username.push(userInfoLehrer.username);
rolleId.push(userInfoLehrer.rolleId);
})
Expand All @@ -175,7 +175,7 @@ test.describe(`Testfälle für die Administration von Personen": Umgebung: ${pro
const sperrDatumBis = await generateDateFuture(5, 2); // Konkrete Testdaten für diesen Testfall

await test.step(`Testdaten: Lehrer mit einer Rolle(LEHR) und SP(email) über die api anlegen ${ADMIN}`, async () => {
userInfoLehrer = await createRolleAndPersonWithUserContext(page, testschule, typelehrer, await generateLehrerNachname(), await generateLehrerVorname(), await getSPId(page, email), await generateRolleName());
userInfoLehrer = await createRolleAndPersonWithUserContext(page, testschule, typelehrer, await generateLehrerNachname(), await generateLehrerVorname(), await getSPId(page, email), generateRolleName());
username.push(userInfoLehrer.username);
rolleId.push(userInfoLehrer.rolleId);
})
Expand Down
13 changes: 7 additions & 6 deletions tests/Profile.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import { UserInfo } from "../base/api/testHelper.page";
import { addSystemrechtToRolle } from "../base/api/testHelperRolle.page";
import { LONG, SHORT, STAGE, BROWSER } from "../base/tags";
import { deleteRolleById, deletePersonByUsername} from "../base/testHelperDeleteTestdata";
import { generateRolleName } from "../base/testHelperGenerateTestdataNames";

const PW: string | undefined = process.env.PW;
const ADMIN: string | undefined = process.env.USER;
Expand Down Expand Up @@ -72,7 +73,7 @@ test.describe(`Testfälle für das eigene Profil anzeigen: Umgebung: ${process.e
const vorname = "TAuto-PW-V-" + faker.person.firstName();
const nachname = "TAuto-PW-N-" + faker.person.lastName();
const organisation = 'Land Schleswig-Holstein';
const rollenname = 'TAuto-PW-R-RolleLandesadmin';
const rollenname = generateRolleName();
const rollenart = 'SYSADMIN'

await test.step(`Landesadmin via api anlegen und mit diesem anmelden`, async () => {
Expand Down Expand Up @@ -140,7 +141,7 @@ test.describe(`Testfälle für das eigene Profil anzeigen: Umgebung: ${process.e
const nachname = "TAuto-PW-N-" + faker.person.lastName();
const organisation = 'Testschule Schulportal';
const dienststellenNr = '1111111';
const rollenname = 'TAuto-PW-R-RolleLehrer';
const rollenname = generateRolleName();
const rollenart = 'LEHR';

await test.step(`Lehrer via api anlegen und mit diesem anmelden`, async () => {
Expand Down Expand Up @@ -198,7 +199,7 @@ test.describe(`Testfälle für das eigene Profil anzeigen: Umgebung: ${process.e
const nachname = "TAuto-PW-N-" + faker.person.lastName();
const organisation = 'Testschule Schulportal';
const dienststellenNr = '1111111';
const rollenname = 'TAuto-PW-R-RolleSchüler';
const rollenname = generateRolleName();
const rollenart = 'LERN';

await test.step(`Lehrer via api anlegen und mit diesem anmelden`, async () => {
Expand Down Expand Up @@ -255,7 +256,7 @@ test.describe(`Testfälle für das eigene Profil anzeigen: Umgebung: ${process.e
const nachname = "TAuto-PW-N-" + faker.person.lastName();
const organisation = 'Testschule Schulportal';
const dienststellenNr = '1111111';
const rollenname = 'TAuto-PW-R-RolleSchuladmin';
const rollenname = generateRolleName();
const rollenart = 'LEIT'

await test.step(`Lehrer via api anlegen und mit diesem anmelden`, async () => {
Expand Down Expand Up @@ -316,7 +317,7 @@ test.describe(`Testfälle für das eigene Profil anzeigen: Umgebung: ${process.e
const organisation2 = 'Carl-Orff-Schule';
const dienststellenNr1 = '1111111';
const dienststellenNr2 = '0702948';
const rollenname = 'TAuto-PW-R-RolleLehrer';
const rollenname = generateRolleName();
const rollenart = 'LEHR';

await test.step(`Lehrer via api anlegen und mit diesem anmelden`, async () => {
Expand Down Expand Up @@ -383,7 +384,7 @@ test.describe(`Testfälle für das eigene Profil anzeigen: Umgebung: ${process.e
const vorname = "TAuto-PW-V-" + faker.person.firstName();
const nachname = "TAuto-PW-N-" + faker.person.lastName();
const organisation = 'Testschule Schulportal';
const rollenname = 'TAuto-PW-R-RolleSchüler';
const rollenname = generateRolleName();
const rollenart = 'LERN';

await test.step(`Lehrer via api anlegen und mit diesem anmelden`, async () => {
Expand Down
3 changes: 2 additions & 1 deletion tests/Schule.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ import { addSystemrechtToRolle } from "../base/api/testHelperRolle.page";
import { FooterDataTablePage } from "../pages/FooterDataTable.page";
import { LONG, SHORT, STAGE, BROWSER } from "../base/tags";
import { deletePersonById, deleteRolleById } from "../base/testHelperDeleteTestdata";
import { generateRolleName } from "../base/testHelperGenerateTestdataNames";

const PW: string | undefined = process.env.PW;
const ADMIN: string | undefined = process.env.USER;
Expand Down Expand Up @@ -149,7 +150,7 @@ test.describe(`Testfälle für die Administration von Schulen: Umgebung: ${proce

const startseite: StartPage = await test.step(`Testdaten: Schuladmin anlegen und mit diesem anmelden`, async () => {
const idSP = await getSPId(page, 'Schulportal-Administration');
userInfo = await createRolleAndPersonWithUserContext(page, 'Testschule Schulportal', 'LEIT', 'TAuto-PW-B-MeierLEIT', 'TAuto-PW-B-Hans', idSP, 'TAuto-PW-R-RolleLEIT');
userInfo = await createRolleAndPersonWithUserContext(page, 'Testschule Schulportal', 'LEIT', 'TAuto-PW-B-MeierLEIT', 'TAuto-PW-B-Hans', idSP, generateRolleName());
personId.push(userInfo.personId);
roleId.push(userInfo.rolleId);
await addSystemrechtToRolle(page, userInfo.rolleId, 'SCHULEN_VERWALTEN');
Expand Down
7 changes: 4 additions & 3 deletions tests/SchulportalAdministration.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -9,6 +9,7 @@ import { addSystemrechtToRolle } from "../base/api/testHelperRolle.page";
import { UserInfo } from "../base/api/testHelper.page";
import { LONG, SHORT, STAGE } from "../base/tags";
import { deletePersonById, deleteRolleById } from "../base/testHelperDeleteTestdata";
import { generateRolleName } from "../base/testHelperGenerateTestdataNames";

const PW: string | undefined = process.env.PW;
const ADMIN: string | undefined = process.env.USER;
Expand Down Expand Up @@ -60,7 +61,7 @@ test.describe(`Testfälle für Schulportal Administration": Umgebung: ${process.
await login.login(ADMIN, PW);

const idSP = await getSPId(page, 'E-Mail');
const userInfo: UserInfo = await createRolleAndPersonWithUserContext(page, 'Testschule Schulportal', 'LEHR', 'TAuto-PW-B-MeierLehrer', 'TAuto-PW-B-Hans', idSP, 'TAuto-PW-R-RolleLehrer');
const userInfo: UserInfo = await createRolleAndPersonWithUserContext(page, 'Testschule Schulportal', 'LEHR', 'TAuto-PW-B-MeierLehrer', 'TAuto-PW-B-Hans', idSP, generateRolleName());
personId.push(userInfo.personId);
rolleId.push(userInfo.rolleId);
await header.logout();
Expand All @@ -87,7 +88,7 @@ test.describe(`Testfälle für Schulportal Administration": Umgebung: ${process.
await login.login(ADMIN, PW);

const idSP = await getSPId(page, 'itslearning');
const userInfo: UserInfo = await createRolleAndPersonWithUserContext(page, 'Testschule Schulportal', 'LERN', 'TAuto-PW-B-JansenSchüler', 'TAuto-PW-B-Helga', idSP, 'TAuto-PW-R-RolleSuS');
const userInfo: UserInfo = await createRolleAndPersonWithUserContext(page, 'Testschule Schulportal', 'LERN', 'TAuto-PW-B-JansenSchüler', 'TAuto-PW-B-Helga', idSP, generateRolleName());
personId.push(userInfo.personId);
rolleId.push(userInfo.rolleId);
await header.logout();
Expand All @@ -114,7 +115,7 @@ test.describe(`Testfälle für Schulportal Administration": Umgebung: ${process.
await login.login(ADMIN, PW);

const idSP = await getSPId(page, 'Schulportal-Administration');
const userInfo: UserInfo = await createRolleAndPersonWithUserContext(page, 'Testschule Schulportal', 'LEIT', 'TAuto-PW-B-MeierAdmin', 'TAuto-PW-B-Peter', idSP, 'TAuto-PW-R-RolleSchuladmin');
const userInfo: UserInfo = await createRolleAndPersonWithUserContext(page, 'Testschule Schulportal', 'LEIT', 'TAuto-PW-B-MeierAdmin', 'TAuto-PW-B-Peter', idSP, generateRolleName());
personId.push(userInfo.personId);
rolleId.push(userInfo.rolleId);

Expand Down
3 changes: 2 additions & 1 deletion tests/login.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ import { faker } from "@faker-js/faker/locale/de";
import { UserInfo } from "../base/api/testHelper.page.ts";
import { deletePersonByUsername, deleteRolleById } from "../base/testHelperDeleteTestdata.ts";
import { getOrganisationId } from "../base/api/testHelperOrganisation.page.ts";
import { generateRolleName } from '../base/testHelperGenerateTestdataNames.ts';

const PW: string | undefined = process.env.PW;
const ADMIN: string | undefined = process.env.USER;
Expand Down Expand Up @@ -108,7 +109,7 @@ test.describe(`Testfälle für die Authentifizierung: Umgebung: ${process.env.UM

const lehrerVorname = "TAuto-PW-V-" + faker.person.firstName();
const lehrerNachname = "TAuto-PW-N-" + faker.person.lastName();
const lehrerRolle = "TAuto-PW-LEHR-" + faker.lorem.word({ length: { min: 8, max: 12 }});
const lehrerRolle = generateRolleName();
const lehrerRollenart = 'LEHR';
const lehrerOrganisation = 'Testschule Schulportal';
let userInfoLehrer: UserInfo;
Expand Down

0 comments on commit 0625c84

Please sign in to comment.