Ticket #5859 enhancement closed fixed

Opened 20 months ago

Last modified 20 months ago

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

Reported by: exarkun Owned by: exarkun
Priority: normal Milestone:
Component: trial Keywords:
Cc: jml Branch: branches/extra-reporter-assertions-5859
(diff, github, buildbot, log)
Author: exarkun Launchpad Bug:

Description

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.

Change History

1

Changed 20 months ago by DefaultCC Plugin

  • cc jml added

2

Changed 20 months ago by exarkun

  • branch set to branches/extra-reporter-assertions-5859
  • branch_author set to exarkun

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

3

Changed 20 months ago by 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

4

Changed 20 months ago by exarkun

  • keywords review added
  • owner exarkun deleted

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

5

Changed 20 months ago by Julian

  • keywords review removed
  • owner set to Julian

6

Changed 20 months ago by Julian

  • owner changed from Julian to exarkun

Hello :).

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

7

Changed 20 months ago by exarkun

  • status changed from new to closed
  • resolution set to fixed

(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.

Note: See TracTickets for help on using tickets.