Skip to content
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: replace dial with newclient #7962

Closed
wants to merge 3 commits into from

Conversation

janardhanvissa
Copy link
Contributor

Partially address: #7049

RELEASE NOTES: None

Copy link

codecov bot commented Dec 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.97%. Comparing base (724f450) to head (269155d).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #7962      +/-   ##
==========================================
- Coverage   82.05%   81.97%   -0.09%     
==========================================
  Files         381      381              
  Lines       38539    38539              
==========================================
- Hits        31622    31591      -31     
- Misses       5602     5623      +21     
- Partials     1315     1325      +10     

see 29 files with indirect coverage changes

@eshitachandwani eshitachandwani added the Area: Testing Includes tests and testing utilities that we have for unit and e2e tests within our repo. label Dec 27, 2024
@eshitachandwani eshitachandwani added this to the 1.70 Release milestone Dec 27, 2024
@eshitachandwani eshitachandwani added Type: Internal Cleanup Refactors, etc and removed Area: Testing Includes tests and testing utilities that we have for unit and e2e tests within our repo. labels Dec 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants