Skip to content

Expand and enforce frontend E2E coverage#492

Merged
MaxEriksson2000 merged 1 commit into
developfrom
test/frontend-e2e-coverage
Jun 12, 2026
Merged

Expand and enforce frontend E2E coverage#492
MaxEriksson2000 merged 1 commit into
developfrom
test/frontend-e2e-coverage

Conversation

@MaxEriksson2000

Copy link
Copy Markdown
Collaborator

Summary

  • make the Frontend E2E job part of the aggregate required CI result
  • cover rejected login, primary navigation, account data, logout, chat persistence, history reopening, and reload
  • add an accessible alert role to the login failure message
  • document the E2E selection strategy and prioritized product flows
  • ignore generated test artifacts in frontend linting

Why

The E2E job ran on pull requests but did not affect the aggregate CI status, and the suite covered only a narrow smoke path. Important browser/backend contracts could regress without blocking a merge.

Developer impact

The deterministic E2E suite now blocks the aggregate CI check. The flow plan separates high-value cross-boundary journeys from behavior better covered by unit or component tests.

Validation

  • Svelte check passed with 0 errors and 0 warnings
  • Playwright discovered 7 tests across 6 files
  • frontend format and lint hooks passed
  • full Docker-backed Playwright execution is delegated to CI

@github-actions

Copy link
Copy Markdown

🧹 Dead-code & unused-dependency report

Advisory — never gates the PR. Whole-repo scan, so some findings may be false positives (dynamic dispatch, framework hooks, runtime-resolved imports). Triage before removing.

No dead code or unused dependencies detected.

@MaxEriksson2000 MaxEriksson2000 marked this pull request as ready for review June 12, 2026 19:25
@github-actions

Copy link
Copy Markdown

📊 Patch coverage

Share of this PR's new/changed lines exercised by tests. Report-only — never gates the PR.

Area Changed Uncovered Coverage
Frontend 1 1 0%
Uncovered lines — Frontend (1 file)
  • …/(public)/login/+page.svelte — 690

@MaxEriksson2000 MaxEriksson2000 merged commit fe7e846 into develop Jun 12, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant