diff options
author | Serhiy Storchaka <storchaka@gmail.com> | 2019-04-01 07:59:24 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-04-01 07:59:24 (GMT) |
commit | a37f356de19828241bf19129f804369794c72ed3 (patch) | |
tree | 8d91421e049ae0a2781ff6c5b3a2352c3ec89071 /Lib/unittest/case.py | |
parent | 5e233951d931acc0e927100c51e9a27a2791b6a5 (diff) | |
download | cpython-a37f356de19828241bf19129f804369794c72ed3.zip cpython-a37f356de19828241bf19129f804369794c72ed3.tar.gz cpython-a37f356de19828241bf19129f804369794c72ed3.tar.bz2 |
[3.7] bpo-36492: Fix passing special keyword arguments to some functions. (GH-12637) (GH-12645)
The following arguments can be passed as keyword arguments for passing
to other function if the corresponding required argument is passed as
positional:
- "func" in functools.partialmethod(), weakref.finalize(),
profile.Profile.runcall(), cProfile.Profile.runcall(),
bdb.Bdb.runcall(), trace.Trace.runfunc() and
curses.wrapper().
- "function" in unittest.addModuleCleanup() and
unittest.TestCase.addCleanup().
- "fn" in the submit() method of concurrent.futures.ThreadPoolExecutor
and concurrent.futures.ProcessPoolExecutor.
- "callback" in contextlib.ExitStack.callback(),
contextlib.AsyncExitStack.callback() and
contextlib.AsyncExitStack.push_async_callback().
- "c" and "typeid" in multiprocessing.managers.Server.create().
- "obj" in weakref.finalize().
(cherry picked from commit 42a139ed88c487f325a241c6ee8b308b3c045975)
Diffstat (limited to 'Lib/unittest/case.py')
-rw-r--r-- | Lib/unittest/case.py | 15 |
1 files changed, 14 insertions, 1 deletions
diff --git a/Lib/unittest/case.py b/Lib/unittest/case.py index 758924d..811f5df 100644 --- a/Lib/unittest/case.py +++ b/Lib/unittest/case.py @@ -442,12 +442,25 @@ class TestCase(object): """ self._type_equality_funcs[typeobj] = function - def addCleanup(self, function, *args, **kwargs): + def addCleanup(*args, **kwargs): """Add a function, with arguments, to be called when the test is completed. Functions added are called on a LIFO basis and are called after tearDown on test failure or success. Cleanup items are called even if setUp fails (unlike tearDown).""" + if len(args) >= 2: + self, function, *args = args + elif not args: + raise TypeError("descriptor 'addCleanup' of 'TestCase' object " + "needs an argument") + elif 'function' in kwargs: + function = kwargs.pop('function') + self, *args = args + else: + raise TypeError('addCleanup expected at least 1 positional ' + 'argument, got %d' % (len(args)-1)) + args = tuple(args) + self._cleanups.append((function, args, kwargs)) def setUp(self): |