Skip to content

Test the full admin workflow end to end - #1828

Merged
KOliver94 merged 2 commits into
mainfrom
e2e-admin
Oct 13, 2026
Merged

KOliver94 merged 2 commits into
mainfrom
e2e-admin

Conversation

@KOliver94

Copy link
Copy Markdown
Member

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.

  1. An admin accepts the request in the decision dialog. Only admins can.
  2. A crew member sets himself as responsible, moves the event into the past (only a finished event can be recorded), adds a crew member and enters the recording path.
  3. He creates the video with an editor and takes it through edited, coded and published (which adds the "Megosztás közösségi platformokon" todo) to archived.
  4. He marks the recording as copied to Google Drive and deleted, which closes the request.
  5. The requester sees the finished request and the published video with its link on the site.

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.

@coderabbitai

coderabbitai Bot commented Oct 13, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You'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.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f8304ee5-4e19-41b6-a1df-60160577cce8
📥 Commits

Reviewing files that changed from the base of the PR and between a9922a3 and c47cb6c.

📒 Files selected for processing (2)
  • frontend/e2e/admin.spec.ts
  • frontend/src/admin/components/AutoCompleteStaff/AutoCompleteStaff.tsx
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 42.73%. Comparing base (a9922a3) to head (c47cb6c).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...components/AutoCompleteStaff/AutoCompleteStaff.tsx 0.00% 2 Missing ⚠️
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              
Flag Coverage Δ
backend 97.33% <ø> (ø)
frontend 4.57% <0.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@KOliver94
KOliver94 merged commit 059c3aa into main Oct 13, 2026
16 checks passed
@KOliver94
KOliver94 deleted the e2e-admin branch October 13, 2026 00:52
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