ci: vitest tweaks to fix ci brittleness#3902
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ 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 |
|
🚀 Deployed on https://deploy-preview-3902--glific-frontend.netlify.app |
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
TIP This summary will be updated as you push new changes. Give us feedback
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #3902 +/- ##
==========================================
+ Coverage 82.29% 82.43% +0.14%
==========================================
Files 312 312
Lines 13467 13467
Branches 3118 3118
==========================================
+ Hits 11082 11101 +19
+ Misses 1417 1402 -15
+ Partials 968 964 -4 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Glific
|
||||||||||||||||||||||||||||
| Project |
Glific
|
| Branch Review |
rvignesh/limit-ci-test-concurr
|
| Run status |
|
| Run duration | 11m 08s |
| Commit |
|
| Committer | Vignesh Rajasekaran |
| View all properties for this run ↗︎ | |
| Test results | |
|---|---|
Failures
|
0
|
Flaky
|
1
|
Pending
|
0
|
Skipped
|
0
|
Passing
|
74
|
| View all changes introduced in this branch ↗︎ | |
rvignesh89
force-pushed
the
rvignesh/limit-ci-test-concurr
branch
from
93f3a7c to
2f771a3
Compare
rvignesh89
force-pushed
the
rvignesh/limit-ci-test-concurr
branch
from
2f771a3 to
43e2a0b
Compare
- Assistants.test: mock navigator.clipboard in beforeEach so the copyCurrentAssistantId click no longer fires a "Sorry, cannot copy" notification in jsdom, which was poisoning the setNotification spy before the expected Knowledge-base notification could fire. - SpeedSend.test: move STICKER URL type and label setup before the first Marathi language-bar click. onLanguageChange fires a notification via the values.type branch whenever type + (label|body) are set, regardless of validation errors (since `if (errors)` is always truthy for a plain object). Relying on values.body from the Lexical editor was unreliable in CI because fireEvent.input does not synchronously propagate to Formik. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 participant
No description provided.