Code review comment for lp:~thumper/juju-core/fix-trunk-break

Revision history for this message
Tim Penhey (thumper) wrote :

*** Submitted:

Fix trunk so it builds and runs tests

r1247 introduced a regression to MachineSuite.TestServeAPI in
cmd/jujud/machine_test.go It was not obvious to me in five minutes how
to make
the test still test what it should, so I have disabled the test by
commenting
it out, and left a TODO with the original branch author to either fix
the test
or delete it.

R=wallyworld
CC=
https://codereview.appspot.com/10022043

https://codereview.appspot.com/10022043/

« Back to merge proposal