Commit da4f1d0759ee5993b8ed36742d938ee78e18b9ff - golang-github-go-kit-kit

test(transport/nats): remove nats server address log line The running bit is set way earlier than the listener that s.Addr returns. This lead to panics on several occasions when the listener was not yet set. Moving after ReadyForConnections could be a solution, but logging the nats server address does not add much to the test anyway: the address is logged if there is an error. Signed-off-by: Mark Sagi-Kazar <mark.sagikazar@gmail.com> Mark Sagi-Kazar 2 years ago

1 changed file(s) with 0 addition(s) and 2 deletion(s). Raw diff Collapse all Expand all

-2

transport/nats/subscriber_test.go less more

41	41	t.Fatal("not yet running")
42	42	}
43	43
44		t.Log(s.Addr().String())
45
46	44	if ok := s.ReadyForConnections(5 * time.Second); !ok {
47	45	t.Fatal("not ready for connections")
48	46	}