fix: continue benchmark looping when some tests fail but timing markers exist

Previously, the benchmark loop stopped immediately when Maven returned
non-zero (any test failure). This was too aggressive because:
- Generated tests may have some failures
- Passing tests still produce valid timing markers
- We need multiple loops for accurate measurements

Now the loop continues if timing markers are present, only stopping when:
- No timing markers are found (all tests failed)
- Target duration is reached
- Max loops is reached

This allows proper multi-loop benchmarking even when some generated
tests fail, improving measurement accuracy.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: misrasaurabh1

codeflash/languages/java/test_runner.py

@@ -640,9 +640,20 @@ def _run_benchmarking_tests_maven(
             )
             break
 
+        # Check if we have timing markers even if some tests failed
+        # We should continue looping if we're getting valid timing data
         if result.returncode != 0:
-            logger.warning("Tests failed in Maven loop %d, stopping", loop_idx)
-            break
+            import re
+            timing_pattern = re.compile(r"!######[^:]*:[^:]*:[^:]*:[^:]*:[^:]+:[^:]+######!")
+            has_timing_markers = bool(timing_pattern.search(result.stdout or ""))
+            if not has_timing_markers:
+                logger.warning("Tests failed in Maven loop %d with no timing markers, stopping", loop_idx)
+                break
+            else:
+                logger.debug(
+                    "Some tests failed in Maven loop %d but timing markers present, continuing",
+                    loop_idx,
+                )
 
     combined_stdout = "\n".join(all_stdout)
     combined_stderr = "\n".join(all_stderr)
@@ -840,10 +851,19 @@ def run_benchmarking_tests(
             )
             break
 
-        # Check if tests failed - don't continue looping
+        # Check if tests failed - continue looping if we have timing markers
         if result.returncode != 0:
-            logger.warning("Tests failed in loop %d, stopping benchmark", loop_idx)
-            break
+            import re
+            timing_pattern = re.compile(r"!######[^:]*:[^:]*:[^:]*:[^:]*:[^:]+:[^:]+######!")
+            has_timing_markers = bool(timing_pattern.search(result.stdout or ""))
+            if not has_timing_markers:
+                logger.warning("Tests failed in loop %d with no timing markers, stopping benchmark", loop_idx)
+                break
+            else:
+                logger.debug(
+                    "Some tests failed in loop %d but timing markers present, continuing",
+                    loop_idx,
+                )
 
     # Create a combined result with all stdout
     combined_stdout = "\n".join(all_stdout)