Clean up tests to use assert.New(t)
Review Request #10002 — Created June 6, 2018 and submitted — Latest diff uploaded
The documentation for
assertmentions that you should use
assert.New(t)when doing more than one assertion, so this patch
updates all our unit tests to this format. Additionally, a helper method
was updated to accepttesting.Tand mark itself as a helper so any
crash or error in it will be reported in the correct test.
Ran
go test ./...
Rango fmt ./...