test(ui-client): clear up runtime warnings#40772
Conversation
|
Looks like this PR is ready to merge! 🎉 |
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
✅ Files skipped from review due to trivial changes (1)
🚧 Files skipped from review as they are similar to previous changes (4)
📜 Recent review details⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
WalkthroughInfoPanelLabel changes its info Icon color to ChangesIcon Color and Test Infrastructure Updates
🎯 3 (Moderate) | ⏱️ ~20 minutes
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Warning Review ran into problems🔥 ProblemsErrors were encountered while retrieving linked issues. Errors (1)
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 |
InfoPanelLabel component5a05f51 to
7f234be
Compare
|
/jira ARCH-2167 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #40772 +/- ##
===========================================
- Coverage 69.80% 69.75% -0.06%
===========================================
Files 3327 3327
Lines 123135 123135
Branches 21983 21956 -27
===========================================
- Hits 85952 85887 -65
- Misses 33832 33892 +60
- Partials 3351 3356 +5
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
7f234be to
6ea43e9
Compare
Proposed changes (including videos or screenshots)
It fixes some runtime warnings in Jest unit tests over
@rocket.chat/ui-client.Issue(s)
Steps to test or reproduce
Further comments
Task: ARCH-2168
Summary by CodeRabbit
Style
Tests