]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
bpo-44515: handle non-refcounted GC in contextlib tests (GH-26910) (GH-27379)
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>
Mon, 26 Jul 2021 21:21:36 +0000 (14:21 -0700)
committerGitHub <noreply@github.com>
Mon, 26 Jul 2021 21:21:36 +0000 (23:21 +0200)
Co-authored-by: Ɓukasz Langa <lukasz@langa.pl>
(cherry picked from commit a2c45e5bcf9d3dfff9f2699dbc161489897616b5)

Co-authored-by: Nick Coghlan <ncoghlan@gmail.com>
Lib/test/test_contextlib.py
Lib/test/test_contextlib_async.py
Misc/NEWS.d/next/Tests/2021-06-26-18-37-36.bpo-44515.e9fO6f.rst [new file with mode: 0644]

index e6500da0ba54a6d531a8a9e2200465477eb07f38..5a080654acb6c78ee258453bc28c20921e5ac8d1 100644 (file)
@@ -231,6 +231,8 @@ def woohoo():
         def woohoo(a, b):
             a = weakref.ref(a)
             b = weakref.ref(b)
+            # Allow test to work with a non-refcounted GC
+            support.gc_collect()
             self.assertIsNone(a())
             self.assertIsNone(b())
             yield
index 9cb40d1d60664c212057529e61aa75995d355f53..603162eaeaa7bfe3ae640f91679a8ffd471958aa 100644 (file)
@@ -1,7 +1,7 @@
 import asyncio
 from contextlib import (
     asynccontextmanager, AbstractAsyncContextManager,
-    AsyncExitStack, nullcontext, aclosing)
+    AsyncExitStack, nullcontext, aclosing, contextmanager)
 import functools
 from test import support
 import unittest
@@ -357,14 +357,17 @@ class AclosingTestCase(unittest.TestCase):
     async def test_aclosing_bpo41229(self):
         state = []
 
-        class Resource:
-            def __del__(self):
+        @contextmanager
+        def sync_resource():
+            try:
+                yield
+            finally:
                 state.append(1)
 
         async def agenfunc():
-            r = Resource()
-            yield -1
-            yield -2
+            with sync_resource():
+                yield -1
+                yield -2
 
         x = agenfunc()
         self.assertEqual(state, [])
diff --git a/Misc/NEWS.d/next/Tests/2021-06-26-18-37-36.bpo-44515.e9fO6f.rst b/Misc/NEWS.d/next/Tests/2021-06-26-18-37-36.bpo-44515.e9fO6f.rst
new file mode 100644 (file)
index 0000000..d2867b6
--- /dev/null
@@ -0,0 +1,2 @@
+Adjust recently added contextlib tests to avoid assuming the use of a
+refcounted GC