From 2da7a80bcbee0a9bb1b712b0ba7398e63499d542 Mon Sep 17 00:00:00 2001 From: Adeeb Shihadeh Date: Thu, 8 Apr 2021 23:46:20 -0700 Subject: [PATCH] manager test: check procs are alive (#20632) Co-authored-by: Comma Device old-commit-hash: 90271c245dfd8cf92b63366541bb376c1cb9c170 --- selfdrive/manager/test/test_manager.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/selfdrive/manager/test/test_manager.py b/selfdrive/manager/test/test_manager.py index e19311f0b7..fd8934850f 100644 --- a/selfdrive/manager/test/test_manager.py +++ b/selfdrive/manager/test/test_manager.py @@ -13,7 +13,7 @@ os.environ['FAKEUPLOAD'] = "1" # TODO: make eon fast MAX_STARTUP_TIME = 30 if EON else 15 -ALL_PROCESSES = [p.name for p in managed_processes.values() if (type(p) is not DaemonProcess) and (p.name not in ['updated', 'pandad'])] +ALL_PROCESSES = [p.name for p in managed_processes.values() if (type(p) is not DaemonProcess) and p.enabled and (p.name not in ['updated', 'pandad'])] class TestManager(unittest.TestCase): @@ -45,9 +45,12 @@ class TestManager(unittest.TestCase): time.sleep(30) for p in reversed(ALL_PROCESSES): + state = managed_processes[p].get_process_state_msg() + self.assertTrue(state.running, f"{p} not running") + exit_code = managed_processes[p].stop(retry=False) - if (not EON and p == 'ui') or (EON and p == 'logcatd'): - # TODO: make Qt UI exit gracefully and fix OMX encoder exiting + if (p == 'ui') or (EON and p == 'logcatd'): + # TODO: make Qt UI exit gracefully continue # Make sure the process is actually dead