8224252: [TESTBUG] hotspot/test/serviceability/sa/sadebugd/SADebugDTest.java is timing out again after fix for JDK-8163805
authorysuenaga
Sat, 25 May 2019 20:55:33 +0900
changeset 55043 d871ce8ab96b
parent 55042 6264783438f6
child 55044 d3afe760b392
8224252: [TESTBUG] hotspot/test/serviceability/sa/sadebugd/SADebugDTest.java is timing out again after fix for JDK-8163805 Reviewed-by: cjplummer, sspitsyn
test/hotspot/jtreg/serviceability/sa/sadebugd/SADebugDTest.java
--- 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);
         }
 
     }