Actually track total class test times, not the time a thread is alive

[johnmlee101]

cc @tylerkaraszewski
I realized that the test times at the end are practically the full test time run, not the actual time we spent in a certain class. This fixes that.

Problem in auth:

  Slowest Test Classes: 
  235791ms: xxx
  234934ms: xxx
  230713ms: xxx
  228530ms: xxx
  227949ms: xx
  227831ms: xx
  227541ms: xx
  227226ms: xx
  227170ms: xx
  226947ms: xx

you can see some of these tests if you sum up their time actually don't count, so I realized this was a bug

result with 16 threads.

Slowest Test Classes:
28651ms: InfiniteRetryAfter : 28.5613% of total test time
23967ms: GetJob : 23.892% of total test time
12892ms: CreateJob : 12.8516% of total test time
10254ms: GetJobs : 10.2219% of total test time
8275ms: FailedJobReply : 8.2491% of total test time
4212ms: RequeueJobs : 4.19882% of total test time
2544ms: ChainedHTTP : 2.53604% of total test time
1341ms: Write : 1.3368% of total test time
1160ms: Query : 1.15637% of total test time
1156ms: CommandPort : 1.15238% of total test time
Total test time across threads: 100314ms

Details

Fixed Issues

Found this

Tests

See tests

---

Internal Testing Reminder: when changing bedrock, please compile auth against your new changes

[johnmlee101]

hide whitespace since I fixed it off by one