Code review comment for lp://staging/~gary/juju-core/megawatcher_api

Revision history for this message
Gary Poster (gary) wrote :

I did this one too. :-)

https://codereview.appspot.com/7563046/diff/1/state/api/params/params_test.go
File state/api/params/params_test.go (right):

https://codereview.appspot.com/7563046/diff/1/state/api/params/params_test.go#newcode105
state/api/params/params_test.go:105: c.Check(
On 2013/03/07 18:05:24, dimitern wrote:
> it'll be nicer to read the code if you assign the json.Unmarshal() to
a var to
> check (all 3 cases below).

Done.

https://codereview.appspot.com/7563046/

« Back to merge proposal