8224252: [TESTBUG] hotspot/test/serviceability/sa/sadebugd/SADebugDTest.java is timing out again after fix for JDK-8163805
Reviewed-by: cjplummer, sspitsyn
--- a/test/hotspot/jtreg/serviceability/sa/sadebugd/SADebugDTest.java Sat May 25 10:54:10 2019 +0100
+++ b/test/hotspot/jtreg/serviceability/sa/sadebugd/SADebugDTest.java Sat May 25 20:55:33 2019 +0900
@@ -23,9 +23,11 @@
/**
* @test
+ * @bug 8163805 8224252
* @summary Checks that the jshdb debugd utility sucessfully starts
* and tries to attach to a running process
* @requires vm.hasSAandCanAttach
+ * @requires os.family != "windows"
* @modules java.base/jdk.internal.misc
* @library /test/lib
*
@@ -40,7 +42,7 @@
public class SADebugDTest {
- private static final String GOLDEN = "Attaching to process";
+ private static final String GOLDEN = "Debugger attached";
public static void main(String[] args) throws Exception {
LingeredApp app = null;
@@ -62,10 +64,9 @@
// If we are here, this means we have received the golden line and the test has passed
// The debugd remains running, we have to kill it
debugd.destroy();
+ debugd.waitFor();
} finally {
- if (app != null) {
- LingeredApp.stopApp(app);
- }
+ LingeredApp.stopApp(app);
}
}