patch up a bit of reporting metric logic

Ensure we stop updating the optimized status for the metrics
once we come of a decision.

Change-Id: I14d74ea71cc79f22170d2cf10ca299f5dd2f3c1d
Test: presubmit
Bug: 383118767

1 files changed, 3 insertions, 1 deletions

diff --git a/ci/build_test_suites.py b/ci/build_test_suites.py
index addad15d0b..7602c64fe0 100644
--- a/ci/build_test_suites.py
+++ b/ci/build_test_suites.py
@@ -166,13 +166,15 @@ class BuildPlanner:
           if re.search(regex, opt):
             get_metrics_agent().report_unoptimized_target(target, 'Test artifact used.')
             build_targets.add(target)
-            continue
+            # proceed to next target evaluation
+            break
           get_metrics_agent().report_optimized_target(target)
         except Exception as e:
           # In case of exception report as unoptimized
           build_targets.add(target)
           get_metrics_agent().report_unoptimized_target(target, f'Error in parsing test discovery output for {target}: {repr(e)}')
           logging.error(f'unable to parse test discovery output: {repr(e)}')
+          break
 
     return build_targets