Opened at 2022-08-15T22:58:27Z
Last modified at 2022-08-16T17:24:52Z
#3916 new defect
Refactor tests that can but do not use base test classes.
Reported by: | fenn-cs | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | undecided |
Component: | unknown | Version: | n/a |
Keywords: | Cc: | ||
Launchpad Bug: |
Description (last modified by fenn-cs)
Just like in #3758, update test such as FakeWebTest in src/allmydata/test/test_testing.py to either use SyncTestCase? and AsyncTestCase? and not testtools.TestCase
- FakeWebTest?: src/allmydata/test/test_testing.py
- MemoryConsumerTests?: src/allmydata/test/test_consumer.py
- PythonTwoRegressions?: src/allmydata/test/test_python2_regressions.py
Change History (3)
comment:1 Changed at 2022-08-15T23:12:13Z by fenn-cs
- Description modified (diff)
- Summary changed from Refactor FakeWebTest to Refactor tests that can but do not use base test classes.
comment:2 Changed at 2022-08-15T23:13:02Z by fenn-cs
- Description modified (diff)
comment:3 Changed at 2022-08-16T17:24:52Z by exarkun
Note: See
TracTickets for help on using
tickets.
test_testing.py and test_consumer.py look like good candidates for switching over. I think test_python2_regressions.py might be obsolete at this point - as we have discontinued Python 2 support at this point (ie, that module could just be deleted).