Code review comment for lp:~thumper/juju-core/client-test-no-fixed-port

Revision history for this message
Andrew Wilkins (axwalk) wrote :

LGTM

https://codereview.appspot.com/82940044/diff/1/state/api/client_test.go
File state/api/client_test.go (right):

https://codereview.appspot.com/82940044/diff/1/state/api/client_test.go#newcode72
state/api/client_test.go:72: url := fmt.Sprintf("http://localhost:%d",
port)
This can be simplified to:
     fmt.Sprintf("http://%s", lis.Addr())

https://codereview.appspot.com/82940044/

« Back to merge proposal