summaryrefslogtreecommitdiff
path: root/ci/build_test_suites.py
diff options
context:
space:
mode:
author Luca Farsi <lucafarsi@google.com> 2024-12-04 03:47:03 -0800
committer Luca Farsi <lucafarsi@google.com> 2024-12-04 04:07:17 -0800
commit1b9b6edcea340377f0ccce98b5eeb8054803ab2a (patch)
tree3f0aabd90d9b37c8ab936fa74dde8b92b1bac0b9 /ci/build_test_suites.py
parent80b6687e62e785ac35c64b30e418bf5e7e13b09a (diff)
Fix crash when failing to parse test discovery output
Test discovery code should never fail a build as it's for info purposes only. Test: atest build_test_suites_test Bug: 382217072 Change-Id: Ib546ee6209e1c3bb9d249cb1ee1d37454449008d
Diffstat (limited to 'ci/build_test_suites.py')
-rw-r--r--ci/build_test_suites.py13
1 files changed, 8 insertions, 5 deletions
diff --git a/ci/build_test_suites.py b/ci/build_test_suites.py
index 4d3b5467dd..623d256b7b 100644
--- a/ci/build_test_suites.py
+++ b/ci/build_test_suites.py
@@ -87,11 +87,14 @@ class BuildPlanner:
if optimization_rationale:
get_metrics_agent().report_unoptimized_target(target, optimization_rationale)
else:
- regex = r'\b(%s)\b' % re.escape(target)
- if any(re.search(regex, opt) for opt in test_discovery_zip_regexes):
- get_metrics_agent().report_unoptimized_target(target, 'Test artifact used.')
- else:
- get_metrics_agent().report_optimized_target(target)
+ try:
+ regex = r'\b(%s)\b' % re.escape(target)
+ if any(re.search(regex, opt) for opt in test_discovery_zip_regexes):
+ get_metrics_agent().report_unoptimized_target(target, 'Test artifact used.')
+ else:
+ get_metrics_agent().report_optimized_target(target)
+ except Exception as e:
+ logging.error(f'unable to parse test discovery output: {repr(e)}')
if self._unused_target_exclusion_enabled(
target