Skip to content

wip#7034

Draft
EvilGenius13 wants to merge 1 commit intomainfrom
theme-e2e-tests
Draft

wip#7034
EvilGenius13 wants to merge 1 commit intomainfrom
theme-e2e-tests

Conversation

@EvilGenius13
Copy link
Contributor

🚧 Testing in progress🚧

WHY are these changes introduced?

Fixes #0000

WHAT is this pull request doing?

How to test your changes?

Post-release steps

Measuring impact

How do we know this change was effective? Please choose one:

  • n/a - this doesn't need measurement, e.g. a linting rule or a bug-fix
  • Existing analytics will cater for this addition
  • PR includes analytics changes to measure impact

Checklist

  • I've considered possible cross-platform impacts (Mac, Linux, Windows)
  • I've considered possible documentation changes

@github-actions
Copy link
Contributor

github-actions bot commented Mar 17, 2026

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 82.8% 14597/17630
🟡 Branches 75.14% 7237/9631
🟢 Functions 82.13% 3708/4515
🟢 Lines 83.24% 13801/16579

Test suite run success

3811 tests passing in 1473 suites.

Report generated by 🧪jest coverage report action from a214539

@EvilGenius13 EvilGenius13 force-pushed the theme-e2e-tests branch 3 times, most recently from b572e73 to 5787117 Compare March 18, 2026 14:24
Add Playwright-based E2E tests for theme commands:
- theme-local.spec.ts: init, check, package (no auth required)
- theme-crud.spec.ts: push, pull, list, delete, rename, duplicate
- theme-dev.spec.ts: dev server lifecycle (skipped pending store setup)
- theme-console.spec.ts: REPL evaluation (skipped pending investigation)

Infrastructure:
- Add theme scaffold fixture with push/pull/delete helpers
- Add minimal Dawn theme fixture for testing
- Add store password support for password-protected stores
- Create separate theme-e2e.yml workflow triggered on theme path changes

The dev and console tests are skipped until a test store without password
protection is configured.
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