commit f33ecedcad5a001735fa4ded5d21caa2cbf27911
author Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> Fri Feb 02 21:38:57 2018 -0800
committer Eric Snow <ericsnowcurrently@gmail.com> Fri Feb 02 22:38:57 2018 -0700
tree 56c3d719e16e9106552c31cde310078ebf84ed1a
parent a23d30f64bd9c5655cfae7f359d4279c47f6cab3

bpo-32604: Fix memory leaks in the new _xxsubinterpreters module. (GH-5507)

(cherry picked from commit 4e9da0d163731caa79811c723c703ee416c31826)

Co-authored-by: Eric Snow <ericsnowcurrently@gmail.com>

Lib/test/test__xxsubinterpreters.py

diff --git a/Lib/test/test__xxsubinterpreters.py b/Lib/test/test__xxsubinterpreters.py
index 2b17044..8d72ca2 100644
--- a/Lib/test/test__xxsubinterpreters.py
+++ b/Lib/test/test__xxsubinterpreters.py
@@ -362,13 +362,15 @@
     def test_from_current(self):
         main, = interpreters.list_all()
         id = interpreters.create()
-        script = dedent("""
+        script = dedent(f"""
             import _xxsubinterpreters as _interpreters
-            _interpreters.destroy({})
-            """).format(id)
+            try:
+                _interpreters.destroy({id})
+            except RuntimeError:
+                pass
+            """)
 
-        with self.assertRaises(RuntimeError):
-            interpreters.run_string(id, script)
+        interpreters.run_string(id, script)
         self.assertEqual(set(interpreters.list_all()), {main, id})
 
     def test_from_sibling(self):
@@ -761,12 +763,12 @@
         self.assertEqual(int(cid), 10)
 
     def test_bad_id(self):
-        ids = [-1, 2**64, "spam"]
-        for cid in ids:
+        for cid in [-1, 'spam']:
             with self.subTest(cid):
                 with self.assertRaises(ValueError):
                     interpreters._channel_id(cid)
-
+        with self.assertRaises(OverflowError):
+            interpreters._channel_id(2**64)
         with self.assertRaises(TypeError):
             interpreters._channel_id(object())