pitfriedrich/chart-app
1
1
Fork 0
Code Issues 6 Pull Requests Actions Packages Projects 1 Releases Wiki Activity

Add Playwright tests to this project #16

New Issue
Closed
opened 2026-07-26 18:00:55 +00:00 by pitfriedrich · 1 comment
pitfriedrich commented 2026-07-26 18:00:55 +00:00
Owner
Copy Link
Copy Source

There should at least be an example test for this project. An excellent start would be the form view.

There should at least be an example test for this project. An excellent start would be the form view.
pitfriedrich added the enhancementai-ready labels 2026-07-26 18:00:55 +00:00
claude-bot added ai-wip and removed ai-ready labels 2026-07-26 18:23:32 +00:00
claude-bot commented 2026-07-26 18:38:56 +00:00
Collaborator
Copy Link
Copy Source

PR: #18

Playwright-E2E-Beispiel für die FormView: e2e/PlaywrightTestBase startet die App auf einem zufälligen Port und fährt ein headless Chromium dagegen, e2e/FormViewPlaywrightTest deckt Rendering, Binder-Validierung sowie Speichern/Zurücksetzen ab (5 Tests).

Zwei Punkte zum Nachlesen:

  • Die Assertions prüfen die englischen Captions — Deutsch ist das Fallback-Bundle ohne Suffix, de ist damit keine provided locale und Vaadin liefert auch einem deutschen Browser translations_en.properties. Die Basisklasse pinnt die Locale deshalb explizit auf en-US.
  • Ohne startbaren Browser (offline Runner, fehlende System-Libs) werden die Tests per JUnit-Assumption übersprungen statt rot — CI bleibt grün.

./mvnw test: 10 Tests, 0 Failures, BUILD SUCCESS.

PR: https://gitea.pitfriedrich.net/pitfriedrich/chart-app/pulls/18 Playwright-E2E-Beispiel für die `FormView`: `e2e/PlaywrightTestBase` startet die App auf einem zufälligen Port und fährt ein headless Chromium dagegen, `e2e/FormViewPlaywrightTest` deckt Rendering, Binder-Validierung sowie Speichern/Zurücksetzen ab (5 Tests). Zwei Punkte zum Nachlesen: - Die Assertions prüfen die **englischen** Captions — Deutsch ist das Fallback-Bundle ohne Suffix, `de` ist damit keine *provided locale* und Vaadin liefert auch einem deutschen Browser `translations_en.properties`. Die Basisklasse pinnt die Locale deshalb explizit auf `en-US`. - Ohne startbaren Browser (offline Runner, fehlende System-Libs) werden die Tests per JUnit-Assumption **übersprungen** statt rot — CI bleibt grün. `./mvnw test`: 10 Tests, 0 Failures, BUILD SUCCESS.
claude-bot added ai-review and removed ai-wip labels 2026-07-26 18:39:02 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
ai-blocked
AI Autofix is blocked - issue needs human
 ai-ready
May be solved by the Autofix loop
 ai-review
Autofix is done, merge ready
 ai-wip
AI Autofix is work-in-progress
 bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No Label ai-review enhancement
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claude-bot pitfriedrich
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: pitfriedrich/chart-app#16
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?