Repository navigation
ci(e2e): add narrow Linux workflow for agent-review spec - #763
Conversation
Dedicated narrow workflow that runs only agent-review.spec.ts under tauri-driver + Xvfb and uploads app/test/e2e/artifacts/**. Gates on spec presence so it can land before the spec itself. Intentionally does not re-enable the broader commented E2E matrix in test.yml.
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 26 minutes and 55 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Dedicated narrow workflow that runs only agent-review.spec.ts under tauri-driver + Xvfb and uploads app/test/e2e/artifacts/**. Gates on spec presence so it can land before the spec itself. Intentionally does not re-enable the broader commented E2E matrix in test.yml. Co-authored-by: Jwalin Shah <jshah1331@gmail.com>
…n\nDedicated narrow workflow that runs only agent-review.spec.ts under\ntauri-driver + Xvfb and uploads app/test/e2e/artifacts/**. Gates on\nspec presence so it can land before the spec itself. Intentionally does\nnot re-enable the broader commented E2E matrix in test.yml.\n\n
Summary
app/test/e2e/specs/agent-review.spec.tsunder tauri-driver + Xvfb.app/test/e2e/artifacts/**plus the tauri-driver log as a CI artifact when present.e2e-linuxmatrix intest.yml.Why a new workflow instead of uncommenting the old block
Behavior
app/test/e2e/helpers/**,app/test/wdio.conf.ts,app/scripts/e2e-*.sh, or this workflow itself; plusworkflow_dispatch.app/test/e2e/specs/agent-review.spec.tsexists — every subsequent step is skipped if not, so this PR is green on its own.yarn workspace openhuman-app test:e2e:build+app/scripts/e2e-run-spec.sh, matching the previously-commented Linux E2E recipe.if-no-files-found: ignoreso empty artifact dirs don't fail the job.Remaining risks / follow-ups
agent-review.spec.tslanding; harness issues will only surface then.app/test/e2e/artifacts/is not yet written by any spec — the spec itself will need to emit there.Test plan