Adjust timeouts for waiting GDB to read debug symbols

oracle/graal#6984 reduced the time spent reading debyg symbols but it's still noticeable. This PR re-adjusts the timeouts to a timeout of 60 seconds. Relates to #160
Karm · Jul 24, 2023 · ca35c39 · ca35c39
1 parent d69f85c
commit ca35c39
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 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
@@ -234,7 +234,7 @@ public void debugSymbolsQuarkus(TestInfo testInfo) throws IOException, Interrupt
                 Logs.appendln(report, stringBuffer.toString());
                 assertTrue(waitForBufferToMatch(stringBuffer,
                                 Pattern.compile(".*Reading symbols from.*", Pattern.DOTALL),
-                                3000, 500, TimeUnit.MILLISECONDS),
+                                60000, 500, TimeUnit.MILLISECONDS),
                         "GDB session did not start well. Check the names, paths... Content was: " + stringBuffer.toString());
 
                 writer.write("set confirm off\n");

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
@@ -238,9 +238,7 @@ public CP[] get(boolean inContainer) {
                     new CP("b ConfigTestController.java:33\n",
                             Pattern.compile(".*Breakpoint 1 at .*: file com/example/quarkus/config/ConfigTestController.java, line 33.*",
                                     Pattern.DOTALL),
-                            // FIXME: The timeout should be revisited once we conclude on whether such a delay on loading
-                            // the symbols is acceptable or not. See https://github.com/Karm/mandrel-integration-tests/issues/160
-                            10 * 60),
+                            60),
                     new CP("run&\n",
                             Pattern.compile(".*Installed features:.*", Pattern.DOTALL)),
                     new CP("GOTO URL http://localhost:8080/data/config/lookup",