-
Notifications
You must be signed in to change notification settings - Fork 919
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
fix(share/getters): set minTimeout #3591
fix(share/getters): set minTimeout #3591
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3591 +/- ##
==========================================
+ Coverage 44.83% 45.62% +0.79%
==========================================
Files 265 280 +15
Lines 14620 15858 +1238
==========================================
+ Hits 6555 7236 +681
- Misses 7313 7794 +481
- Partials 752 828 +76 ☔ View full report in Codecov by Sentry. |
several integration tests failing @vgonkivs |
I'm not sure if my changes caused that. |
Looks like 1min timeout is just too much:
Maybe in tests we should keep 15sec or 30sec timeout, 2c. |
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.
utack
Set minTimeout in case the context hasn't contained it.