Have all tests use `t *testing.T` instead of `tt *testing.T`

[andresuribe87]

The benefits include:

• Preventing the subtest from accessing the upper level by hiding it.
• More maintainable, as it's standard go practice.
• Easier to copy/paste and move around tests.