This fix looks good to me. We've plugged a number of edge cases in the tests where the timing of debuggee and debugger processes was not guaranteed during shutdown of the test.
The only improvement we could add here is to test if the vm is suspended, but an extra resume does no harm to ensure the debuggee is running to receive the quit message. On 6/5/19, 10:36 PM, Daniil Titov wrote:
Please review a change that fixes the intermittent failure of the test. The problem here that there is a chance that a single step event had been posted after the step request was created and before it was deleted. The fix solves this by ensuring that vm.resume() is called at the end of the test. Webrev: http://cr.openjdk.java.net/~dtitov/8206074/webrev.01 Bug: https://bugs.openjdk.java.net/browse/JDK-8206074 Thanks! --Daniil