On Fri, 17 Feb 2023 15:27:04 GMT, Roger Riggs <rri...@openjdk.org> wrote:

>> Matthew Donovan has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   added exceptions for cases 4 and 5
>
> test/jdk/javax/rmi/ssl/SSLSocketParametersTest.java line 34:
> 
>> 32:  * @run main/othervm SSLSocketParametersTest 5
>> 33:  * @run main/othervm SSLSocketParametersTest 6
>> 34:  * @run main/othervm SSLSocketParametersTest 7
> 
> Is a fresh VM needed for each of these tests?
> From a naive point of view it appears that 1, 2, 3, 4, 6, 7 use the same 
> system properties.
> The tests would complete more quickly if they could run the compatible tests 
> in a single VM.
> Perhaps main() could iterate over the args[n] and run the corresponding test.

I tried this suggestion but test case 6 failed. From the logs, it looked like 
an object was being re-used and an exception wasn't being thrown when it should 
have been.

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

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

Reply via email to