Here's a question...

At the bottom of this method we have ...

```objc
[_testSuiteState publishEvents];
```

But, now with your changes... should that just be ...

```objc
[_testSuiteState publishEventsForFinishedTests];
```

?  With your stuff, I'm wondering if we'll ever publish _unfinished_ test results.