From b1db308c6114bb18c06656c94c0ffa73ba954977 Mon Sep 17 00:00:00 2001 From: andrei kulakov Date: Sun, 22 Aug 2021 14:32:45 -0400 Subject: bpo-41322: Add unit tests for deprecation of test return values (GH-27846) Also fix the traceback of warnings. --- Doc/library/unittest.rst | 4 ++++ Doc/whatsnew/3.11.rst | 4 ++++ Lib/unittest/async_case.py | 2 +- Lib/unittest/case.py | 2 +- Lib/unittest/test/test_async_case.py | 20 ++++++++++++++++++++ Lib/unittest/test/test_case.py | 20 ++++++++++++++++++++ 6 files changed, 50 insertions(+), 2 deletions(-) diff --git a/Doc/library/unittest.rst b/Doc/library/unittest.rst index 99c2f6e..f0fba94 100644 --- a/Doc/library/unittest.rst +++ b/Doc/library/unittest.rst @@ -151,6 +151,10 @@ The above examples show the most commonly used :mod:`unittest` features which are sufficient to meet many everyday testing needs. The remainder of the documentation explores the full feature set from first principles. +.. versionchanged:: 3.11 + The behavior of returning a value from a test method (other than the default + ``None`` value), is now deprecated. + .. _unittest-command-line-interface: diff --git a/Doc/whatsnew/3.11.rst b/Doc/whatsnew/3.11.rst index 49b4364..cb8f84c 100644 --- a/Doc/whatsnew/3.11.rst +++ b/Doc/whatsnew/3.11.rst @@ -395,3 +395,7 @@ Removed :func:`~gettext.install` are also removed, since they are only used for the ``l*gettext()`` functions. (Contributed by Dong-hee Na and Serhiy Storchaka in :issue:`44235`.) + +* The behavior of returning a value from a :class:`~unittest.TestCase` and + :class:`~unittest.IsolatedAsyncioTestCase` test methods (other than the default ``None`` + value), is now deprecated. diff --git a/Lib/unittest/async_case.py b/Lib/unittest/async_case.py index cc404cc..bfc68a7 100644 --- a/Lib/unittest/async_case.py +++ b/Lib/unittest/async_case.py @@ -65,7 +65,7 @@ class IsolatedAsyncioTestCase(TestCase): def _callTestMethod(self, method): if self._callMaybeAsync(method) is not None: warnings.warn(f'It is deprecated to return a value!=None from a ' - f'test case ({method})', DeprecationWarning) + f'test case ({method})', DeprecationWarning, stacklevel=4) def _callTearDown(self): self._callAsync(self.asyncTearDown) diff --git a/Lib/unittest/case.py b/Lib/unittest/case.py index 625d27e..8775ba9 100644 --- a/Lib/unittest/case.py +++ b/Lib/unittest/case.py @@ -548,7 +548,7 @@ class TestCase(object): def _callTestMethod(self, method): if method() is not None: warnings.warn(f'It is deprecated to return a value!=None from a ' - f'test case ({method})', DeprecationWarning) + f'test case ({method})', DeprecationWarning, stacklevel=3) def _callTearDown(self): self.tearDown() diff --git a/Lib/unittest/test/test_async_case.py b/Lib/unittest/test/test_async_case.py index 6e48b9e..93ef199 100644 --- a/Lib/unittest/test/test_async_case.py +++ b/Lib/unittest/test/test_async_case.py @@ -167,6 +167,26 @@ class TestAsyncCase(unittest.TestCase): test.run() self.assertEqual(events, ['asyncSetUp', 'test', 'asyncTearDown', 'cleanup']) + def test_deprecation_of_return_val_from_test(self): + # Issue 41322 - deprecate return of value!=None from a test + class Test(unittest.IsolatedAsyncioTestCase): + async def test1(self): + return 1 + async def test2(self): + yield 1 + + with self.assertWarns(DeprecationWarning) as w: + Test('test1').run() + self.assertIn('It is deprecated to return a value!=None', str(w.warnings[0].message)) + self.assertIn('test1', str(w.warnings[0].message)) + self.assertEqual(w.warnings[0].filename, __file__) + + with self.assertWarns(DeprecationWarning) as w: + Test('test2').run() + self.assertIn('It is deprecated to return a value!=None', str(w.warnings[0].message)) + self.assertIn('test2', str(w.warnings[0].message)) + self.assertEqual(w.warnings[0].filename, __file__) + def test_cleanups_interleave_order(self): events = [] diff --git a/Lib/unittest/test/test_case.py b/Lib/unittest/test/test_case.py index b8aca92..f3cabe4 100644 --- a/Lib/unittest/test/test_case.py +++ b/Lib/unittest/test/test_case.py @@ -306,6 +306,26 @@ class Test_TestCase(unittest.TestCase, TestEquality, TestHashing): Foo('test').run() + def test_deprecation_of_return_val_from_test(self): + # Issue 41322 - deprecate return of value!=None from a test + class Foo(unittest.TestCase): + def test1(self): + return 1 + def test2(self): + yield 1 + + with self.assertWarns(DeprecationWarning) as w: + Foo('test1').run() + self.assertIn('It is deprecated to return a value!=None', str(w.warnings[0].message)) + self.assertIn('test1', str(w.warnings[0].message)) + self.assertEqual(w.warnings[0].filename, __file__) + + with self.assertWarns(DeprecationWarning) as w: + Foo('test2').run() + self.assertIn('It is deprecated to return a value!=None', str(w.warnings[0].message)) + self.assertIn('test2', str(w.warnings[0].message)) + self.assertEqual(w.warnings[0].filename, __file__) + def _check_call_order__subtests(self, result, events, expected_events): class Foo(Test.LoggingTestCase): def test(self): -- cgit v0.12