-
Notifications
You must be signed in to change notification settings - Fork 41
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
Node recovery conductor testing #2797
Conversation
…e-recovery-conductor-testing
Codecov Report
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. @@ Coverage Diff @@
## sprint-1.11 #2797 +/- ##
===============================================
- Coverage 28.80% 28.76% -0.05%
===============================================
Files 395 397 +2
Lines 65842 66009 +167
===============================================
+ Hits 18967 18985 +18
- Misses 44581 44726 +145
- Partials 2294 2298 +4
Flags with carried forward coverage won't be shown. Click here to find out more.
|
…chain into node-recovery-conductor-testing
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
Fixes
Add conductor tests for the cases when providers crash then come back up clean.
Changes
Need to be mentioned in CHANGELOG.md?
Tests
Tasks to complete before merging PR:
Associated PRs (Link as appropriate):