Clean up tests to use assert.New(t)
Review Request #10002 — Created June 6, 2018 and submitted — Latest diff uploaded
The documentation for
assert
mentions 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.T
and mark itself as a helper so any
crash or error in it will be reported in the correct test.
Ran
go test ./...
Rango fmt ./...