3
87%
dev: 88%

Ran 17 Jun 2016 04:12PM UTC

Files 42

Run time 8s

Badge

Embed ▾

Committed 17 Jun 2016 04:00PM UTC coverage: 85.613%. First build

Job # TEST_TIMEOUT_SCALE=20

Build Type

push

travis-ci

Committed by

prashantv

Commit Message

Add call validation, and fix flaky call validation tests

Previously, we only validated that a timeout was set, not that a
servicename was specified when making an outbound call.

Add validation, and do it before we get a connection. This avoids
creating a connection that will not be used.

Previously, we would create a connection, do validation, and close the
connection immediately. On the inbound side, this led to a new
connection that was closed almost immediately, and so sometimes the test
would cause a:
"Failed to add connection to peer" "connection is in an invalid state"

By doing validation before we create a connection we make TestNoTimeout
less flaky, while also returning errors earlier and faster.

Run Details

3761 of 4393 relevant lines covered (85.61%)

3160.23 hits per line

uber / tchannel-go / 2070 / 3
87%
dev: 88%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2070.3 (TEST_TIMEOUT_SCALE=20)

uber / tchannel-go / 2070 / 3 87% dev: 88%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2070.3 (TEST_TIMEOUT_SCALE=20)

uber / tchannel-go / 2070 / 3
87%
dev: 88%

README BADGES
x