grpctest: add test coverages of `ExitIdle`
Fixes: https://github.com/grpc/grpc-go/issues/8118
- Add test cases for
- [X] ExitIdle()
- [X] ExitIdleOne()
- [X] UpdateClientConnState
- [x] ResolverError
RELEASE NOTES: N/A
Codecov Report
:white_check_mark: All modified and coverable lines are covered by tests.
:white_check_mark: Project coverage is 81.82%. Comparing base (a5e7cd6) to head (705625a).
:warning: Report is 22 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #8375 +/- ##
==========================================
- Coverage 82.48% 81.82% -0.67%
==========================================
Files 414 413 -1
Lines 40464 40518 +54
==========================================
- Hits 33376 33152 -224
- Misses 5736 5990 +254
- Partials 1352 1376 +24
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
Hey @hugehoo , is this ready for review or is it still a work in progress? If it is ready for review , please request a review and unassign yourself so that we can review it.
Hey @hugehoo , is this ready for review or is it still a work in progress? If it is ready for review , please request a review and unassign yourself so that we can review it.
hi, i'm still working on this pr, testing is still broken. i think i can request a review within this weekend.
Hey @hugehoo are you still working on this PR?
This PR is labeled as requiring an update from the reporter, and no update has been received after 6 days. If no update is provided in the next 7 days, this issue will be automatically closed.
Hey @hugehoo are you still working on this PR?
sorry for late, i requested for review now
Hey @hugehoo , The name of this test was recently changed : tests (tests, 1.24, arm64, ubuntu-24.04-arm) , and that is why the test is stuck. Can you merge master so that the test can run again?
Hey @hugehoo , The name of this test was recently changed : tests (tests, 1.24, arm64, ubuntu-24.04-arm) , and that is why the test is stuck. Can you merge master so that the test can run again?
@eshitachandwani thx for the info, i just merged it and run the test again.
This PR is labeled as requiring an update from the reporter, and no update has been received after 6 days. If no update is provided in the next 7 days, this issue will be automatically closed.
@eshitachandwani : Maybe this is good for another round of review. Can you please take another pass? Thanks
Thank you, @hugehoo, for your contribution!