diff options
author | 2024-12-04 03:47:03 -0800 | |
---|---|---|
committer | 2024-12-04 04:07:17 -0800 | |
commit | 1b9b6edcea340377f0ccce98b5eeb8054803ab2a (patch) | |
tree | 3f0aabd90d9b37c8ab936fa74dde8b92b1bac0b9 /ci/build_test_suites.py | |
parent | 80b6687e62e785ac35c64b30e418bf5e7e13b09a (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.py | 13 |
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 |