Thanks for working on this! I've added a couple comments.

 clean up test output

Seems fine to me, the only issue is #1302

 choose a proper log filepath to pass to the testStub?

Is this necessary? The existing code already writes to logfiles when not outputting to the console (e.g. when running tests in parallel).

If a logfile is necessary, though, something which uses some suffix like "test-detailed" alongw tih buildLogPath should do.

 handle unsupported TestType (throw an error?)

Throwing an error seems reasonable.

Thanks!