The fix changes test serviceability/jdwp/AllModulesCommandTest.java to accept 
VM flags. 
1) The 'ProcessTools.createTestJvm(JDWP_OPT, DEBUGGEE);'  is used to start 
debugee
2) The stderr is just logging and tests doesn't check if it is empty. stderr 
might contain some VM output which doesn't mean failure
3) Some refactoring is done. Method ' public void onStringRead(StreamHandler 
handler, String line) {' doesn't need to check which handler is used and 
corresponding code and methods are deleted. The field 'inputHandler' and 
'errorHandler' were moved into method.

Tested with tier1-5 to ensure that test correctly work with CI options.

The serviceability/jdwp/AllModulesCommandTest.java is the only one test in 
serviceability/jdwp directory so no other tests should be affected.

-------------

Commit messages:
 - 8315406

Changes: https://git.openjdk.org/jdk/pull/15544/files
 Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=15544&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8315406
  Stats: 63 lines in 3 files changed: 3 ins; 51 del; 9 mod
  Patch: https://git.openjdk.org/jdk/pull/15544.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/15544/head:pull/15544

PR: https://git.openjdk.org/jdk/pull/15544

Reply via email to