Clean up tests to use assert.New(t)

Review Request #10002 — Created June 6, 2018 and submitted

Information

rb-gateway
master
e1a9bc6...

Reviewers

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 accept testing.T and mark itself as a helper so any
crash or error in it will be reported in the correct test.

Ran go test ./...
Ran go fmt ./...

brennie
brennie
brennie
brennie
brennie
brennie
david
  1. Ship It!
  2. 
      
brennie
Review request changed
Status:
Completed
Change Summary:
Pushed to master (6991583)