I was originally going to try to see if I could use generics to set a context.Context field in the Tester embedded in the struct so that every test case could automatically get our default deadline (and we could even forbid context.Background() from being called from tests). But, in practice we declare test methods without receivers, e.g.:

func (s) TestFoo(t *testing.T)

So the only way to do that would require a change to:

func (s s) TestFoo(t *testing.T) {
	// access s.Ctx here
}

At which point, we might just be better off doing:

func (s) TestFoo(ctx context.Context, t *testing.T)

But regardless of any of that, I found a simple cleanup to use an interface instead of reflection to find the Setup and Teardown methods, which at one point (before we started overriding the logger) were optional.

RELEASE NOTES: n/a