Skip to content

Commit

Permalink
[3.12] gh-111644: Fix asyncio test_unhandled_exceptions() (GH-111713) (
Browse files Browse the repository at this point in the history
…#111718)

gh-111644: Fix asyncio test_unhandled_exceptions() (GH-111713)

Fix test_unhandled_exceptions() of test_asyncio.test_streams: break
explicitly a reference cycle.

Fix also StreamTests.tearDown(): the loop must not be closed
explicitly, but using set_event_loop() which takes care of shutting
down the executor with executor.shutdown(wait=True).
BaseEventLoop.close() calls executor.shutdown(wait=False).
(cherry picked from commit ac01e22)

Co-authored-by: Victor Stinner <vstinner@python.org>
  • Loading branch information
miss-islington and vstinner committed Nov 4, 2023
1 parent 02f8f78 commit d15fac2
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions Lib/test/test_asyncio/test_streams.py
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,7 @@ def tearDown(self):
# just in case if we have transport close callbacks
test_utils.run_briefly(self.loop)

self.loop.close()
gc.collect()
# set_event_loop() takes care of closing self.loop in a safe way
super().tearDown()

def _basetest_open_connection(self, open_connection_fut):
Expand Down Expand Up @@ -1102,6 +1101,8 @@ async def handle_echo(reader, writer):

self.assertEqual(messages[0]['message'],
'Unhandled exception in client_connected_cb')
# Break explicitly reference cycle
messages = None


if __name__ == '__main__':
Expand Down

0 comments on commit d15fac2

Please sign in to comment.