juju-core

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

  1. client-test-no-fixed-port
  2. Merge into trunk
Revision history for this message
Andrew Wilkins (axwalk) wrote :

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#newcode69
state/api/client_test.go:69: lis, err := net.Listen("tcp", ":0")
Sorry, just realised: please defer lis.Close() after checking the error.

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

« Back to merge proposal