From dd93a7e0036233295cc68919f06e904127e478f7 Mon Sep 17 00:00:00 2001 From: Daniel Rainer Date: Wed, 17 Dec 2025 16:15:21 +0100 Subject: [PATCH] test_driver: stop printing tmpdir on failure We delete the tmpdir unconditionally once all tests are completed, so there is no point in printing a path which will no longer exist when analyzing test failures. The paths don't contain any useful information, so let's delete them to avoid confusion and useless output. --- tests/test_driver.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/tests/test_driver.py b/tests/test_driver.py index 6d07ae48e..fb96b1e37 100755 --- a/tests/test_driver.py +++ b/tests/test_driver.py @@ -334,7 +334,7 @@ async def run_test( elif ret: return TestPass(arg, duration_ms) else: - return TestFail(arg, duration_ms, f"{error_message}\nTmpdir is {home}") + return TestFail(arg, duration_ms, error_message) elif test_file_path.endswith(".py"): test_env.update( { @@ -371,7 +371,6 @@ async def run_test( error_message += stdout.decode("utf-8") if stderr: error_message += stderr.decode("utf-8") - error_message += f"Tmpdir is {home}" return TestFail(arg, duration_ms, error_message) else: return TestFail(arg, None, "Error in test driver. This should be unreachable.")