From 6958070eecd9507a28b7c43f7fe572ed1eef9811 Mon Sep 17 00:00:00 2001 From: Jonas L Date: Sat, 22 Jun 2024 13:31:13 +0200 Subject: [PATCH] test: ensure the context cleanup is executed (#3040) See https://pylint.readthedocs.io/en/latest/user_guide/messages/warning/contextmanager-generator-missing-cleanup.html --- playout/tests/liquidsoap/client/conftest.py | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/playout/tests/liquidsoap/client/conftest.py b/playout/tests/liquidsoap/client/conftest.py index 87bbeed00..e44ff837b 100644 --- a/playout/tests/liquidsoap/client/conftest.py +++ b/playout/tests/liquidsoap/client/conftest.py @@ -119,9 +119,10 @@ def run_liq_server( if process.poll() is not None: pytest.fail(process.stdout.read()) - yield manager - - process.terminate() + try: + yield manager + finally: + process.terminate() @pytest.fixture(