juju-core

Code review comment for lp://staging/~jameinel/juju-core/disable-testmanagestateservesapi-1219661

  1. disable-testmanagestateservesapi-1219661
  2. Merge into trunk
Revision history for this message
Andrew Wilkins (axwalk) wrote :

LGTM

https://codereview.appspot.com/13253045/diff/1/cmd/jujud/machine_test.go
File cmd/jujud/machine_test.go (right):

https://codereview.appspot.com/13253045/diff/1/cmd/jujud/machine_test.go#newcode381
cmd/jujud/machine_test.go:381: func (s *MachineSuite)
DONT_TestManageStateServesAPI(c *gc.C) {
I think it would be better to use c.Skip, just so it's in everybody's
faces.

https://codereview.appspot.com/13253045/

« Back to merge proposal