Repository navigation
Conversation
…ighten the admin workflow test
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 22 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1828 +/- ##
==========================================
- Coverage 42.74% 42.73% -0.01%
==========================================
Files 273 273
Lines 6939 6940 +1
Branches 1383 1384 +1
==========================================
Hits 2966 2966
- Misses 3950 3951 +1
Partials 23 23
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Adds an end-to-end test that takes a request through the whole admin workflow, from submitted to closed with a finished video, and checks the result on the public site. It relies on the accessible names from #1827.
The workflow
One spec,
e2e/admin.spec.ts, runs its steps in order on the seeded "Kari gólyabál", which no other spec touches. If a step fails, the later ones are skipped, so the failure shows where the workflow broke.A bug fixed along the way
The staff pickers ("Felelős", "Vágó", "Stábtag", "Felelősök") filter a staff list that loads in the background. A search typed before it arrived found nobody until the user typed again. The picker now stays disabled until the list has loaded.