-
-
Notifications
You must be signed in to change notification settings - Fork 313
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
Add proxy Body::collect_bytes
for easier unit tests
#1445
Conversation
Avoids people having to pull in `http_body_util` for the `BodyExt` Signed-off-by: clux <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1445 +/- ##
=======================================
+ Coverage 72.1% 72.1% +0.1%
=======================================
Files 78 78
Lines 6591 6592 +1
=======================================
+ Hits 4747 4748 +1
Misses 1844 1844
|
Body::collect
for easier unit testsBody::collect
for easier unit tests
Signed-off-by: clux <[email protected]>
Body::collect
for easier unit testsBody::collect_bytes
for easier unit tests
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.
Saw the change is pretty straightforward so I thought I'd jump in to help ship it. LGTM, left two small comments that imo shouldn't block under any circumstance.
Co-authored-by: Matei David <[email protected]> Signed-off-by: Eirik A <[email protected]>
Co-authored-by: Matei David <[email protected]> Signed-off-by: Eirik A <[email protected]>
Avoids people having to pull in
http_body_util
for theBodyExt
Tested with kube-rs/controller-rs#69 locally.