summaryrefslogtreecommitdiffstats
path: root/Lib/test/test_coroutines.py
diff options
context:
space:
mode:
authorSerhiy Storchaka <storchaka@gmail.com>2021-06-21 07:21:59 (GMT)
committerGitHub <noreply@github.com>2021-06-21 07:21:59 (GMT)
commit5d2b3a0d688cf8a33db3d266c9e7049c13766a4c (patch)
tree8a6fdb152c663c76248f62d4311cfcdb32a3656e /Lib/test/test_coroutines.py
parenta317778fd58b1c6b250feffbdb4ecf15e293ef48 (diff)
downloadcpython-5d2b3a0d688cf8a33db3d266c9e7049c13766a4c.zip
cpython-5d2b3a0d688cf8a33db3d266c9e7049c13766a4c.tar.gz
cpython-5d2b3a0d688cf8a33db3d266c9e7049c13766a4c.tar.bz2
bpo-44469: Fix tests for "async with" with bad object (GH-26817)
Test for execution of the body was null. It would pass even if the code which should be skipped was executed.
Diffstat (limited to 'Lib/test/test_coroutines.py')
-rw-r--r--Lib/test/test_coroutines.py18
1 files changed, 12 insertions, 6 deletions
diff --git a/Lib/test/test_coroutines.py b/Lib/test/test_coroutines.py
index 145adb6..a6a199e 100644
--- a/Lib/test/test_coroutines.py
+++ b/Lib/test/test_coroutines.py
@@ -1205,41 +1205,47 @@ class CoroutineTest(unittest.TestCase):
def __aenter__(self):
pass
- body_executed = False
+ body_executed = None
async def foo():
+ nonlocal body_executed
+ body_executed = False
async with CM():
body_executed = True
with self.assertRaisesRegex(AttributeError, '__aexit__'):
run_async(foo())
- self.assertFalse(body_executed)
+ self.assertIs(body_executed, False)
def test_with_3(self):
class CM:
def __aexit__(self):
pass
- body_executed = False
+ body_executed = None
async def foo():
+ nonlocal body_executed
+ body_executed = False
async with CM():
body_executed = True
with self.assertRaisesRegex(AttributeError, '__aenter__'):
run_async(foo())
- self.assertFalse(body_executed)
+ self.assertIs(body_executed, False)
def test_with_4(self):
class CM:
pass
- body_executed = False
+ body_executed = None
async def foo():
+ nonlocal body_executed
+ body_executed = False
async with CM():
body_executed = True
with self.assertRaisesRegex(AttributeError, '__aenter__'):
run_async(foo())
- self.assertFalse(body_executed)
+ self.assertIs(body_executed, False)
def test_with_5(self):
# While this test doesn't make a lot of sense,