jmx-dev RFR: 8316447: 8 sun/management/jmxremote tests ignore VM flags [v2]

Fri Oct 6 23:25:35 UTC 2023

On Fri, 6 Oct 2023 22:34:40 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:

>> Leonid Mesnik has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   simplifed usage, removed classpath adding.
>
> test/jdk/sun/management/jmxremote/bootstrap/CustomLauncherTest.java line 100:
> 
>> 98:             ProcessBuilder client = ProcessTools.createJavaProcessBuilder(
>> 99:                 "-cp",
>> 100:                 Utils.TEST_CLASS_PATH,
> 
> Just a question: Is the `-cp` options added by the `ProcessTools.createTestJvm()`?

It is added even by ProcessTools.createJavaProcessBuilder(..) so it is not needed here.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/16048#discussion_r1349386018