removeTest() method patch for unittest.TestSuite

[edgimar]

BPO	1778410
Nosy	@ezio-melotti, @merwok, @voidspace, @iritkatriel
Files	unittest_removetest.patch: patch to add removeTest method to TestSuite

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2021-11-18.09:31:51.090>
created_at = <Date 2007-08-21.09:14:46.000>
labels = ['type-feature', 'library']
title = 'removeTest() method patch for unittest.TestSuite'
updated_at = <Date 2021-11-18.09:31:51.088>
user = 'https://bugs.python.org/edgimar'

bugs.python.org fields:

activity = <Date 2021-11-18.09:31:51.088>
actor = 'iritkatriel'
assignee = 'none'
closed = True
closed_date = <Date 2021-11-18.09:31:51.090>
closer = 'iritkatriel'
components = ['Library (Lib)']
creation = <Date 2007-08-21.09:14:46.000>
creator = 'edgimar'
dependencies = []
files = ['8178']
hgrepos = []
issue_num = 1778410
keywords = ['patch']
message_count = 9.0
messages = ['53044', '108206', '111726', '111732', '111740', '180740', '181857', '181858', '406525']
nosy_count = 5.0
nosy_names = ['edgimar', 'ezio.melotti', 'eric.araujo', 'michael.foord', 'iritkatriel']
pr_nums = []
priority = 'normal'
resolution = 'rejected'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'enhancement'
url = 'https://bugs.python.org/issue1778410'
versions = ['Python 3.4']

[edgimar]

This is a simple patch to add a "removeTest" method to the unittest.TestSuite object. It is called in exactly the same way the "addTest" method is called, except that it removes the provided TestCase from the suite.

This is useful if you want to add a bunch of unittests via TestLoader().loadTestsFromModule(), EXCEPT for a few specific cases, which you would like to subsequently remove.

[BreamoreBoy]

This is almost three years old but strikes me as being useful. Could somebody with more knowledge than myself review the patch and update the relevant Python versions? Failing that, simply close the issue as not being relevant today.

[BreamoreBoy]

As nobody has responded to msg53044 or msg108206 I'll close this unless there are any objections.

[merwok]

Michael, can you say if the feature request is accepted or rejected?

[voidspace]

I'm sympathetic to the feature request. I think that the way tests identify themselves will change (improve) to accommodate the new extension machinery. I would like to leave this feature request open for the moment and revisit it again once the extension machinery is in place.

[ezio-melotti]

Michael, what's the situation now?

[voidspace]

I'm pretty sure the proposed patch doesn't work - but there's no test for it so I can't be sure. I can't think of a better basic strategy, but the strategy here is horrible. This means I'm sympathetic to the desire for a removeTest method but not very enthusiastic about implementing it (and certainly not this implementation I'm afraid).

[voidspace]

I think better filtering at the collection phase - collecting tests by name or filtering out tests by name - could obviate the need for this.

[iritkatriel]

I agree with Michael that it’s better not to include the test in the first place. This issue was abandoned 8 years ago, closing now.