Increase timeout for DebugSymbolsTest#debugSymbolsQuarkus

Karm · Jun 6, 2023 · 36d4f62 · 36d4f62
1 parent 2f86a18
commit 36d4f62
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/testsuite/src/it/java/org/graalvm/tests/integration/DebugSymbolsTest.java b/testsuite/src/it/java/org/graalvm/tests/integration/DebugSymbolsTest.java
@@ -406,7 +406,7 @@ public static void carryOutGDBSession(StringBuffer stringBuffer, GDBSession gdbS
                         } else {
                             writer.write(cp.c);
                             writer.flush();
-                            boolean m = waitForBufferToMatch(stringBuffer, cp.p, 10, 1, TimeUnit.SECONDS);
+                            boolean m = waitForBufferToMatch(stringBuffer, cp.p, cp.timeoutSeconds, 1, TimeUnit.SECONDS);
                             Logs.appendlnSection(report, cp.c);
                             Logs.appendln(report, stringBuffer.toString());
                             if (!m) {

diff --git a/testsuite/src/it/java/org/graalvm/tests/integration/utils/CP.java b/testsuite/src/it/java/org/graalvm/tests/integration/utils/CP.java
@@ -29,9 +29,17 @@
 public class CP {
     public final String c;
     public final Pattern p;
+    public final int timeoutSeconds;
 
     public CP(String c, Pattern p) {
         this.c = c;
         this.p = p;
+        this.timeoutSeconds = 10;
+    }
+
+    public CP(String c, Pattern p, int timeoutSeconds) {
+        this.c = c;
+        this.p = p;
+        this.timeoutSeconds = timeoutSeconds;
     }
 }
diff --git a/testsuite/src/it/java/org/graalvm/tests/integration/utils/GDBSession.java b/testsuite/src/it/java/org/graalvm/tests/integration/utils/GDBSession.java
@@ -237,7 +237,8 @@ public CP[] get(boolean inContainer) {
                     SHOW_VERSION,
                     new CP("b ConfigTestController.java:33\n",
                             Pattern.compile(".*Breakpoint 1 at .*: file com/example/quarkus/config/ConfigTestController.java, line 33.*",
-                                    Pattern.DOTALL)),
+                                    Pattern.DOTALL),
+                            5 * 60), //FIXME: The timeout should be revisited once we conclude on whether such a delay on loading the symbols is acceptable or not.
                     new CP("run&\n",
                             Pattern.compile(".*Installed features:.*", Pattern.DOTALL)),
                     new CP("GOTO URL http://localhost:8080/data/config/lookup",