test(e2e): event-lifecycle.spec.ts und base.spec.ts auf getByTestId umstellen #261

New issue

Closed

opened 2026-07-06 21:50:42 +02:00 by niboer · 2 comments

niboer commented 2026-07-06 21:50:42 +02:00

Owner

Copy link

Beschreibung

Nachdem in #260 die fehlenden testids im Frontend ergaenzt wurden, sollen die E2E-Tests in event-lifecycle.spec.ts und base.spec.ts von getByLabel/getByRole/getByText auf getByTestId umgestellt werden.

Abhaengigkeit

Blocked by #260 — die testids muessen erst im Frontend vorhanden sein.

Aenderungen

e2e/specs/test-3-workflows/event-lifecycle.spec.ts

3 Stellen mit je 3 Selektoren (Zeilen 30-32, 42-43, 51-53):

- page.getByLabel('Titel')
+ page.getByTestId('input-event-title')
- page.getByLabel('Datum')
+ page.getByTestId('input-event-date')
- page.getByRole('button', { name: 'Event anlegen' })
+ page.getByTestId('btn-event-create')

e2e/specs/test-1-base/base.spec.ts

Sidebar-Links (testids existieren bereits):

- nav.getByRole('link', { name: 'Dashboard' })
+ page.getByTestId('nav-link-dashboard')
- nav.getByRole('link', { name: 'Events' })
+ page.getByTestId('nav-link-events')
- nav.getByRole('link', { name: 'Personen' })
+ page.getByTestId('nav-link-persons')
- nav.getByRole('link', { name: 'Mein Haushalt' })
+ page.getByTestId('nav-link-groups')

Selektoren die testids aus #260 benoetigen:

- nav.getByRole('link', { name: /logout/i })
+ page.getByTestId('btn-logout')
- nav.getByText('Max Mustermann')
+ page.getByTestId('text-sidebar-name')
- nav.getByText('max@example.com')
+ page.getByTestId('text-sidebar-email')
- nav.getByText('App-Admin')
+ page.getByTestId('text-sidebar-role')
- getByRole('heading', { name: 'Naechste Feier' })
+ page.getByTestId('heading-next-event')
- getByRole('heading', { name: 'Offene RSVPs' })
+ page.getByTestId('heading-open-rsvps')
- getByRole('heading', { name: 'Personen' })
+ page.getByTestId('heading-persons')

Test-Hinweise

• npm run test:integration muss gruen sein
• Ergebnis in /output/e2e-results/ kontrollieren — alle Tests passed

Checkliste

• event-lifecycle.spec.ts auf testid umgestellt
• base.spec.ts auf testid umgestellt
• E2E-Testlauf erfolgreich (alle 64+ Tests passed)
• PR erstellt mit Referenz auf #260

## Beschreibung Nachdem in #260 die fehlenden testids im Frontend ergaenzt wurden, sollen die E2E-Tests in `event-lifecycle.spec.ts` und `base.spec.ts` von `getByLabel`/`getByRole`/`getByText` auf `getByTestId` umgestellt werden. ## Abhaengigkeit **Blocked by #260** — die testids muessen erst im Frontend vorhanden sein. ## Aenderungen ### `e2e/specs/test-3-workflows/event-lifecycle.spec.ts` 3 Stellen mit je 3 Selektoren (Zeilen 30-32, 42-43, 51-53): ```diff - page.getByLabel('Titel') + page.getByTestId('input-event-title') - page.getByLabel('Datum') + page.getByTestId('input-event-date') - page.getByRole('button', { name: 'Event anlegen' }) + page.getByTestId('btn-event-create') ``` ### `e2e/specs/test-1-base/base.spec.ts` Sidebar-Links (testids existieren bereits): ```diff - nav.getByRole('link', { name: 'Dashboard' }) + page.getByTestId('nav-link-dashboard') - nav.getByRole('link', { name: 'Events' }) + page.getByTestId('nav-link-events') - nav.getByRole('link', { name: 'Personen' }) + page.getByTestId('nav-link-persons') - nav.getByRole('link', { name: 'Mein Haushalt' }) + page.getByTestId('nav-link-groups') ``` Selektoren die testids aus #260 benoetigen: ```diff - nav.getByRole('link', { name: /logout/i }) + page.getByTestId('btn-logout') - nav.getByText('Max Mustermann') + page.getByTestId('text-sidebar-name') - nav.getByText('max@example.com') + page.getByTestId('text-sidebar-email') - nav.getByText('App-Admin') + page.getByTestId('text-sidebar-role') - getByRole('heading', { name: 'Naechste Feier' }) + page.getByTestId('heading-next-event') - getByRole('heading', { name: 'Offene RSVPs' }) + page.getByTestId('heading-open-rsvps') - getByRole('heading', { name: 'Personen' }) + page.getByTestId('heading-persons') ``` ## Test-Hinweise - `npm run test:integration` muss gruen sein - Ergebnis in `/output/e2e-results/` kontrollieren — alle Tests `passed` ## Checkliste - [ ] `event-lifecycle.spec.ts` auf testid umgestellt - [ ] `base.spec.ts` auf testid umgestellt - [ ] E2E-Testlauf erfolgreich (alle 64+ Tests passed) - [ ] PR erstellt mit Referenz auf #260

niboer added this to the MS0: Projekt-Konventionen & Tooling milestone 2026-07-06 21:50:42 +02:00

niboer added the

prio/medium

type/test

component/e2e

labels 2026-07-06 21:50:42 +02:00

niboer referenced this issue 2026-07-06 21:50:43 +02:00

chore(frontend): fehlende testids an Dashboard, Sidebar, Toast und Personenliste ergaenzen #260

niboer commented 2026-07-06 21:50:48 +02:00

Author

Owner

Copy link

Blocked by #260 — testids muessen erst im Frontend vorhanden sein.

Blocked by #260 — testids muessen erst im Frontend vorhanden sein.

niboer referenced this issue 2026-07-08 17:31:07 +02:00

chore(frontend+e2e): fehlende testids an Dashboard, Sidebar, Toast und Personenliste ergaenzt #264

niboer referenced this issue from a commit 2026-07-08 17:40:18 +02:00

chore(frontend+e2e): fehlende testids an Dashboard, Sidebar, Toast und Personenliste ergaenzt (#264)

niboer modified the milestone from MS0: Projekt-Konventionen & Tooling to MS-REST: Restliche Aufgaben 2026-07-08 17:42:17 +02:00

niboer commented 2026-07-09 16:37:22 +02:00

Author

Owner

Copy link

Plan aktualisiert (09.07.2026)

Nach Analyse des aktuellen Code-Stands und Ruecksprache:

Scope

Datei	Aenderungen
e2e/fixtures/test-data.ts	SEED_EVENT_ID-Konstante fuer hartcodierte Seed-Event-UUID 9675d3e8-b3cd-4a38-aafe-d031fb6fd3b5
event-lifecycle.spec.ts	3 Formular-Selektoren: getByLabel('Titel') → getByTestId('input-event-title'), getByLabel('Datum') → getByTestId('input-event-date'), getByRole('button', ...) → getByTestId('btn-event-create')
event-lifecycle.spec.ts	7 Tabs/Back-Link: page.locator('[data-testid="..."]') → page.getByTestId('...')
event-lifecycle.spec.ts	2 Row-Selektoren: page.locator('[data-testid^="row-"]').first() → page.getByTestId('row-9675d3e8-b3cd-4a38-aafe-d031fb6fd3b5')
base.spec.ts	6 Nav-Selektoren: nav.getByRole('link', ...) → page.getByTestId('nav-link-*') / page.getByTestId('btn-logout') / page.getByTestId('link-login')
docs/design/07-e2e-tests.md	Beispiel-Code (Z. 169-174) von page.locator('[data-testid="..."]') auf page.getByTestId('...') aktualisieren

Ausgelassen

• page.locator('h1') — keine testids, bleiben
• page.getByText('Letzte Feiern') — kein testid vorhanden, nicht im Scope

**Plan aktualisiert (09.07.2026)** Nach Analyse des aktuellen Code-Stands und Ruecksprache: ### Scope | Datei | Aenderungen | |-------|------------| | `e2e/fixtures/test-data.ts` | `SEED_EVENT_ID`-Konstante fuer hartcodierte Seed-Event-UUID `9675d3e8-b3cd-4a38-aafe-d031fb6fd3b5` | | `event-lifecycle.spec.ts` | **3 Formular-Selektoren**: `getByLabel('Titel')` → `getByTestId('input-event-title')`, `getByLabel('Datum')` → `getByTestId('input-event-date')`, `getByRole('button', ...)` → `getByTestId('btn-event-create')` | | `event-lifecycle.spec.ts` | **7 Tabs/Back-Link**: `page.locator('[data-testid="..."]')` → `page.getByTestId('...')` | | `event-lifecycle.spec.ts` | **2 Row-Selektoren**: `page.locator('[data-testid^="row-"]').first()` → `page.getByTestId('row-9675d3e8-b3cd-4a38-aafe-d031fb6fd3b5')` | | `base.spec.ts` | **6 Nav-Selektoren**: `nav.getByRole('link', ...)` → `page.getByTestId('nav-link-*')` / `page.getByTestId('btn-logout')` / `page.getByTestId('link-login')` | | `docs/design/07-e2e-tests.md` | Beispiel-Code (Z. 169-174) von `page.locator('[data-testid="..."]')` auf `page.getByTestId('...')` aktualisieren | ### Ausgelassen - `page.locator('h1')` — keine testids, bleiben - `page.getByText('Letzte Feiern')` — kein testid vorhanden, nicht im Scope

niboer referenced this issue from a commit 2026-07-09 16:39:37 +02:00

test(e2e): Event-Lifecycle und Base auf getByTestId umgestellt

niboer referenced this issue from a pull request that will close it, 2026-07-09 16:39:43 +02:00

test(e2e): Event-Lifecycle und Base auf getByTestId umgestellt #279

niboer closed this issue 2026-07-09 16:56:12 +02:00

niboer referenced this issue from a commit 2026-07-09 16:56:12 +02:00

test(e2e): Event-Lifecycle und Base auf getByTestId umgestellt (#279)

Sign in to join this conversation.

No Branch/Tag specified

Branches Tags

main

v0.5.0

v0.4.0

v0.3.0

v0.2.2

v0.2.1

v0.2.0

v0.1.0

Labels

Clear labels   

component/backend

Backend-Komponente

  

component/docs

Dokumentation (README, design.md, AGENTS.md)

  

component/e2e

E2E-Testinfrastruktur (Playwright, Docker-E2E-Runner)

  

component/frontend

Frontend (Cloudflare Worker, BFF, UI)

  

component/infra

Infrastruktur (Docker, CI/CD, Build)

  

component/keycloak

Keycloak (Realm, Theme, Admin API)

  

prio/high

Hohe Prioritaet (blockierend/kritisch)

  

prio/low

Niedrige Prioritaet

  

prio/medium

Mittlere Prioritaet

  

type/bug

Fehler oder unerwartetes Verhalten

  

type/chore

Wartungs- und Konfigurationsaufgaben

  

type/feature

Neues Feature

  

type/refactor

Refactoring

  

type/test

Tests

No labels

component/backend

component/docs

component/e2e

component/frontend

component/infra

component/keycloak

prio/high

prio/low

prio/medium

type/bug

type/chore

type/feature

type/refactor

type/test

Milestone

Clear milestone

No items

No milestone

MS-REST: Restliche Aufgaben

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

WompSchmiede/FamilienFeierPlaner#261

No description provided.