nats-io / gnatsd / 1357

91%

master: 91%

push

travis-ci

[FIXED] Shutdown stops http server when started manually

In case one creates a server instance with New() and then starts
the http server manually (s.StartHTTPMonitoring()), calling
s.Shutdown() would not stop the http server because Shutdown()
would return without doing anything if `running` was not true.
This boolean was set to true only in `s.Start()`.

Also added StartMonitoring() to perform the options check and
selectively start http or https server to replace individual calls.
This is useful for NATS Streaming server that will now be able
to call s.StartMonitoring() without having to duplicate code
about options checks and http server code.

This is related to PR #481

30 of 30 new or added lines in 1 file covered. (100.0%)

4931 of 5425 relevant lines covered (90.89%)

2820.61 hits per line