From c6538e91b81c820580e6f403a49b0d4d81523472 Mon Sep 17 00:00:00 2001 From: grekiki Date: Sun, 9 May 2021 11:54:22 +0200 Subject: [PATCH] Valgrind test: fix output parsing when no leaks present (#20862) * should work * spacing and try to get CI to run --- selfdrive/test/test_valgrind_replay.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/selfdrive/test/test_valgrind_replay.py b/selfdrive/test/test_valgrind_replay.py index 7a8529db16..7a78bb7329 100644 --- a/selfdrive/test/test_valgrind_replay.py +++ b/selfdrive/test/test_valgrind_replay.py @@ -36,6 +36,9 @@ CONFIGS = [ class TestValgrind(unittest.TestCase): def extract_leak_sizes(self, log): + if "All heap blocks were freed -- no leaks are possible" in log: + return (0,0,0) + log = log.replace(",","") # fixes casting to int issue with large leaks err_lost1 = log.split("definitely lost: ")[1] err_lost2 = log.split("indirectly lost: ")[1] @@ -89,7 +92,7 @@ class TestValgrind(unittest.TestCase): self.done = True def test_config(self): - open(os.path.join(BASEDIR, "selfdrive/test/valgrind_logs.txt"), "w") + open(os.path.join(BASEDIR, "selfdrive/test/valgrind_logs.txt"), "w").close() for cfg in CONFIGS: self.done = False