Code review comment for lp:~themue/juju-core/go-state-unit-status

Revision history for this message
William Reade (fwereade) wrote :

LGTM with test fix

https://codereview.appspot.com/6454113/diff/9001/state/unit_test.go
File state/unit_test.go (right):

https://codereview.appspot.com/6454113/diff/9001/state/unit_test.go#newcode52
state/unit_test.go:52: c.Assert(err, IsNil)
p, err := s.unit.SetAgentAlive()
c.Assert(err, IsNil)
defer func(){
     c.Assert(p.Kill(), IsNil)
}()

https://codereview.appspot.com/6454113/

« Back to merge proposal