Skip to content

Refactored Playwright E2E tests#3

Open
deacon-mp wants to merge 3 commits intomasterfrom
test/exhaustive-ui-e2e-tests
Open

Refactored Playwright E2E tests#3
deacon-mp wants to merge 3 commits intomasterfrom
test/exhaustive-ui-e2e-tests

Conversation

@deacon-mp
Copy link
Copy Markdown
Owner

Mirror of mitre/debrief PR mitre#80 for Copilot review

Covers plugin page load, operation selection, report tabs (stats, agents,
steps, tactics, facts), PDF download, JSON export, D3 graph rendering,
graph settings modal, and error states. Tests run against a full Caldera
instance via CALDERA_URL env var.
…e bugs

- Extract mockDebriefRoutes() helper to eliminate ~400 lines of copy-pasted
  route mocking across 15 tests
- Replace all waitForTimeout() calls with deterministic waits
  (waitForResponse, waitFor on locators)
- Replace networkidle with domcontentloaded + explicit heading wait
- Remove unmocked live API test that would fail without running server
- Fix playback button count comment (6 buttons including legend toggle)
- Use baseURL from playwright config instead of duplicating CALDERA_URL
- Fix file handle leak in c_story.py adjust_icon_svgs
- Add viewBox null check in c_story.py
- Fix stale op_id bug in debrief_svc.py build_steps_d3
In CI (process.env.CI), CALDERA_USER and CALDERA_PASS must be set
explicitly via env vars — config throws if missing. Locally, defaults
to admin/admin for developer convenience.
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