From ce775edbc1dfa687dbc1a1cc2152791b6f5f3e7f Mon Sep 17 00:00:00 2001 From: Dengke Tang Date: Mon, 9 Oct 2023 20:36:36 -0700 Subject: [PATCH] only try to generate coverage on success --- builder/actions/cmake.py | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/builder/actions/cmake.py b/builder/actions/cmake.py index 49762760e..61d2af6dc 100644 --- a/builder/actions/cmake.py +++ b/builder/actions/cmake.py @@ -246,9 +246,13 @@ def run(self, env): return ctest = toolchain.ctest_binary() - sh.exec(*toolchain.shell_env, ctest, - "--output-on-failure", working_dir=project_build_dir, check=True) - # Try to generate the coverage report. Will be ignored by ctest if no coverage data available. + try: + sh.exec(*toolchain.shell_env, ctest, + "--output-on-failure", working_dir=project_build_dir, check=True) + except: + # Skip generate the coverage report on error + return + # If CTest found no test, generate coverage will hang sh.exec(*toolchain.shell_env, ctest, "-T", "coverage", working_dir=project_build_dir, check=True)