Merge pull request #45960 from brunobat/fix-test-stability

Improve MetricsTest reliability
quarkusio · Jan 30, 2025 · 3e7485f · 3e7485f
2 parents 7901fe1 + c34a2de
commit 3e7485f
Showing 1 changed file with 14 additions and 2 deletions.
diff --git a/integration-tests/opentelemetry/src/test/java/io/quarkus/it/opentelemetry/MetricsTest.java b/integration-tests/opentelemetry/src/test/java/io/quarkus/it/opentelemetry/MetricsTest.java
@@ -25,7 +25,15 @@ public class MetricsTest {
     @BeforeEach
     @AfterEach
     void reset() {
-        given().get("/reset").then().statusCode(HTTP_OK);
+        await().atMost(5, SECONDS).until(() -> {
+            List<Map<String, Object>> spans = getSpans();
+            if (spans.size() == 0) {
+                return true;
+            } else {
+                given().get("/reset").then().statusCode(HTTP_OK);
+                return false;
+            }
+        });
     }
 
     private List<Map<String, Object>> getSpans() {
@@ -54,7 +62,11 @@ public void directCounterTest() {
                 .then()
                 .statusCode(200);
 
-        await().atMost(5, SECONDS).until(() -> getSpans().size() == 2);
+        await().atMost(10, SECONDS).until(() -> {
+            List<Map<String, Object>> spans = getSpans();
+            System.out.println("spans size " + spans.size());
+            return spans.size() == 2;
+        });
         await().atMost(10, SECONDS).until(() -> getMetrics("direct-trace-counter").size() > 2);
 
         List<Map<String, Object>> metrics = getMetrics("direct-trace-counter");