juju-core

Code review comment for lp:~themue/juju-core/024-cleanup-watcher

  1. 024-cleanup-watcher
  2. Merge into trunk
Revision history for this message
William Reade (fwereade) wrote :

just another comment on the tests

https://codereview.appspot.com/10078043/diff/6001/state/state_test.go
File state/state_test.go (right):

https://codereview.appspot.com/10078043/diff/6001/state/state_test.go#newcode1357
state/state_test.go:1357: defer cw.Stop()
I think `defer stop(c, cw)` would be slightly more correct.

https://codereview.appspot.com/10078043/

« Back to merge proposal