Make CC_CODE_COVERAGE_SCRIPT failures fail tests

If you're editing, or overriding, this script and you introduce and introduce an error, previously this wouldn't fail the test invocation.
bazelbuild · Jan 10, 2023 · 9a0b847 · 9a0b847
1 parent 88c426e
commit 9a0b847
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/tools/test/collect_coverage.sh b/tools/test/collect_coverage.sh
@@ -180,7 +180,10 @@ fi
 cd $ROOT
 # Call the C++ code coverage collection script.
 if [[ "$CC_CODE_COVERAGE_SCRIPT" ]]; then
-    eval "${CC_CODE_COVERAGE_SCRIPT}"
+    if ! eval "${CC_CODE_COVERAGE_SCRIPT}"; then
+      echo "error: coverage collection script failed" >&2
+      exit 1
+    fi
 fi
 
 if [[ -z "$LCOV_MERGER" ]]; then