-
Notifications
You must be signed in to change notification settings - Fork 0
Implement parallel CI workflows for backend and frontend testing #16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
- Fixed E2E test hangs by racing navigation + API calls with Promise.all() - Replaced 3-test suite with single deterministic run-validation test - Added data-testid attributes to run-events and discovered-screens - Updated frontend development + debugging skills with E2E patterns - Integrated spec-kit as submodule for spec-driven development - Added Appium startup script for automated mobile testing Test result: ✅ 1 passed (5.7s) Resolves: BUG-015, BUG-014 validation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
- Created backend-test.yml for Encore test execution - Created frontend-e2e.yml for Playwright E2E tests with backend service - Updated workflows README with new architecture documentation - Workflows run in parallel with path-based filtering for speed - Added dependency caching and concurrency cancellation - Included comprehensive troubleshooting guide Co-authored-by: nirukk52 <28961063+nirukk52@users.noreply.github.com>
- Quote variables in shell scripts to prevent word splitting - Fixes actionlint validation errors - All workflows now pass actionlint validation Co-authored-by: nirukk52 <28961063+nirukk52@users.noreply.github.com>
- Add explanation of why encore test is used over vitest directly - Document E2E workflow's dependency on backend service - Clarify integration testing approach - Improve troubleshooting guidance Co-authored-by: nirukk52 <28961063+nirukk52@users.noreply.github.com>
| sleep 2 | ||
| done | ||
| env: | ||
| BACKEND_PORT: 4000 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bug: Backend Health Check: Silent Failure, Delayed Errors.
The backend health check loop exits without verifying success. If the backend fails to start within 60 seconds, the workflow silently continues to build and test, causing cryptic failures later instead of failing fast with a clear error about backend unavailability.
Replaces monolithic CI scaffold with two parallel workflows optimized for speed and resource efficiency per qa_vibe.json requirements.
Changes
New Workflows
backend-test.yml- Runsencore teston backend changes only (15m timeout)frontend-e2e.yml- Runs Playwright E2E with backend service on frontend/backend changes (20m timeout)Architecture
Optimizations
bun.lock)Documentation
.github/workflows/README.mdencore testvs direct vitest invocationKey Design Decisions
Frontend E2E depends on backend: E2E tests validate complete user workflows including API calls and event streaming, requiring backend service availability.
Separate workflows over monolithic: Path-based filtering prevents unnecessary runs. Backend tests skip when only frontend changes, saving ~10 minutes per PR.
All validation passing: actionlint, yamllint, shellcheck
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.
Note
Introduces two parallel GitHub Actions for backend Encore tests and frontend Playwright E2E (with backend), plus comprehensive README updates for architecture, usage, and troubleshooting.
backend-test.yml:backend/**changes (push/PR tomain/develop).encore test; uploads artifacts.frontend-e2e.yml:frontend/**andbackend/**changes (push/PR tomain/develop).encore runwith health polling; builds frontend; runs Playwright E2E; uploads logs/reports/results./.github/workflows/README.mdto mark CI active and document parallel architecture, triggers, caching, timeouts, artifacts, troubleshooting, and planned workflows.Written by Cursor Bugbot for commit 0d31386. This will update automatically on new commits. Configure here.