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
I did this one too. :-)
https:/ /codereview. appspot. com/7563046/ diff/1/ state/api/ params/ params_ test.go params/ params_ test.go (right):
File state/api/
https:/ /codereview. appspot. com/7563046/ diff/1/ state/api/ params/ params_ test.go# newcode105 params/ params_ test.go: 105: c.Check(
state/api/
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/