1
100%
main: 88%

Ran 29 May 2019 11:37AM UTC

Files 9

Run time 1s

Badge

Embed ▾

Committed 29 May 2019 11:32AM UTC coverage: 100.0%. Remained the same

Job # 61.1

Build Type

push

travis-ci

Committed by

bassosimone

Commit Message

Avoid publicly exposing an interface

Long time ago @evfirerob mentioned to me that in Go it's more
idiomatic for the consumer to enforce interfaces if needed.

However, my initial stab at this branch was exposing the
HTTPRequestor interface that I actually initially created
to simplify testing and was not meant to be shared.

This knowledge about interfaces resurfaced today when I was
re-reading my diff, and I realized it's probably better to
expose an `*http.Client` rather than an interface.

This simplification does not seem to complicate much more
testing, and rather allows to hide some details of the way
in which we do testing, so I think it's an improvement.

Run Details

401 of 401 relevant lines covered (100.0%)

1.09 hits per line

m-lab / ndt7-client-go / 61 / 1
100%
main: 88%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 61.1

m-lab / ndt7-client-go / 61 / 1 100% main: 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 61.1

m-lab / ndt7-client-go / 61 / 1
100%
main: 88%

README BADGES
x