1
51%
master: 51%

Ran 22 Oct 2018 11:52PM UTC

Files 99

Run time 9s

Badge

Embed ▾

Committed 22 Oct 2018 11:45PM UTC coverage: 79.391% (-0.01%) from 79.405%

Job # COVERALLS_TOKEN=MYSkSqcsWXd6DmP6TnSeiDhtvuL4u6ndp

Build Type

push

travis-ci

Committed by

peterbourgon

Commit Message

Fix 'Client.Endpoint' to not 'cancel' when bufferedStream (#776)

* transport/http/client_test: Modify the test to make it fail

With this modifications we can trigger the error that we are searching, 'context canceled'

* transport/http/client: Add the 'bodyWithCancel' to wrap the Response.Body

It adds the context.CancelFunc to the io.ReadCloser.Close function so bouth are called together

* transport/http/client: Add more documentation to clarify the changes

Also abstracted some logic on the test to make it more clear and also added more docuemntation. Added more documentation
on the definition of 'BufferedStream' to clarify that the Body has to be closed manually to properly close the response.

* transport/http/client: Add period at the end of the doc of 'BufferedStream'

Run Details

5293 of 6667 relevant lines covered (79.39%)

74434.8 hits per line

go-kit / kit / 2203 / 1
51%
master: 51%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2203.1 (COVERALLS_TOKEN=MYSkSqcsWXd6DmP6TnSeiDhtvuL4u6ndp)

go-kit / kit / 2203 / 1 51% master: 51%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2203.1 (COVERALLS_TOKEN=MYSkSqcsWXd6DmP6TnSeiDhtvuL4u6ndp)

go-kit / kit / 2203 / 1
51%
master: 51%

README BADGES
x