-
-
Notifications
You must be signed in to change notification settings - Fork 229
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
test: Cover func updatestatefulset #863
test: Cover func updatestatefulset #863
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #863 +/- ##
==========================================
+ Coverage 35.20% 39.03% +3.83%
==========================================
Files 19 19
Lines 3213 2631 -582
==========================================
- Hits 1131 1027 -104
+ Misses 2015 1535 -480
- Partials 67 69 +2 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Mahesh Kumar <[email protected]>
5fb8368
to
ea08d1c
Compare
Signed-off-by: Mahesh Kumar <[email protected]>
Signed-off-by: Shubham Gupta <[email protected]>
Signed-off-by: Shubham Gupta <[email protected]>
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
* test case for updatestatefulset functions Signed-off-by: Mahesh Kumar <[email protected]> * tests: updated TestUpdateStatefulSet Signed-off-by: Mahesh Kumar <[email protected]> * fix: failing test Signed-off-by: Shubham Gupta <[email protected]> * test: Add one more case Signed-off-by: Shubham Gupta <[email protected]> --------- Signed-off-by: Mahesh Kumar <[email protected]> Signed-off-by: Shubham Gupta <[email protected]> Co-authored-by: opstree <[email protected]> Co-authored-by: Shubham Gupta <[email protected]> Signed-off-by: Matt Robinson <[email protected]>
Tests for updatestatefulset functions