1101
89%
master: 91%

Ran 09 Dec 2016 09:08PM UTC

Jobs 1

Files 17

Run time 1s

Badge

Embed ▾

pending completion

Build # 1101

Build Type

push

travis-ci

Committed by

kozlovic

Commit Message

Replace GetListenEndpoint() with ReadyForConnections()

The RunServer() function (and the various variants)
call Server.Start() in a go-routine, but do not return until
it has verified that the server is ready to accept connections.
To do so, it use GetListenEndpoint() to get a suitable connect
address (replacing "0.0.0.0" or "::" with localhost - important
on Windows). It then creates a raw TCP connection to ensure the
server is started, repeating the process in case of failure up
to 10 seconds.

This PR replaces this with a function that checks that client
listener, and route listener if configured, are set. This removes
the need to get a connect address and create test tcp connections.

The reason for this change is that NATS Streaming when starting
the NATS Server (unless configured to connect to a remote one)
calls RunServerWithAuth(), which when getting "localhost" from
GetListenEndpoint(), would fail trying to resolve it. This happened
for the NATS Streaming Docker image built with Go 1.7+.

Run Details

4911 of 5500 relevant lines covered (89.29%)

2636.69 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
2	1101.2	09 Dec 2016 09:08PM UTC	0	89.29	Travis Job 1101.2

Source Files on build 1101

Detailed source file information is not available for this build.

nats-io / gnatsd / 1101
89%
master: 91%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1101

nats-io / gnatsd / 1101 89% master: 91%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 1101

nats-io / gnatsd / 1101
89%
master: 91%

README BADGES
x