Add more important assertions to twisted/trial/test/test_tests.py

[exarkun]

Many of the tests in test_tests.py operate by running a suite and then making assertions on the results/reporter object. Some of these tests skip checking certain parts of the reporter's state, though, which potentially allows a broken implementation to pass them.

The tests should have a few more assertions added to make sure a properly working implementation is really in use.

This is split off of #5853.

[exarkun]

(In [35170]) Branching to 'extra-reporter-assertions-5859'

[exarkun]

(In [35171]) Add a few extra assertions to some existing test methods. Also add missing docstrings, cleanup whitespace, and fix test names.

refs #5859

[exarkun]

Okay, made those improvements. There's surely more that could be done here, like unifying the logic for asserting a particular result, rather than repeating the same (or, more often than not, not quite the same) group of 5 or 8 individual assertions. But I added the extra coverage I wanted, at least, and made some other minor cleanups as well.

 Build results

[Julian]

Hello :).

There's a typo in TestSkipClasses's test_results test method (a doubled period). Other than that, what's here looks fine :).

[exarkun]

(In [35176]) Merge extra-reporter-assertions-5859

Author: exarkun Reviewer: Julian Fixes: #5859

Expand the test coverage for what is done to the test results object passed in to a trial test suite. Also, clean up a few other minor issues such as whitespace and docstrings.