-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Use InDelta in TestExternallyControlledRun #1677
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1677 +/- ##
==========================================
+ Coverage 72.10% 72.30% +0.20%
==========================================
Files 167 175 +8
Lines 12882 13443 +561
==========================================
+ Hits 9288 9720 +432
- Misses 3031 3115 +84
- Partials 563 608 +45
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
3784637
to
dd5817c
Compare
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.
We shouldn't have to do this, but OK 🙁
This is actually what should've been to begin with ... as k6 is trying to signal to the goja.VM that it should stop because the context was canceled (or the deadline was reached). Previous to this there was the chance that the goja.VM will be in a call that will throw the context.Canceled on it's own (sleep) if the context was canceled. Fixes #1485
d02f979
to
2d967ab
Compare
2d967ab
to
8ec265e
Compare
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.
LGTM
I added this to the v0.29.0 milestone, since it fixes the |
No description provided.