On Tue, 7 Feb 2023 16:02:30 GMT, Matthew Donovan <d...@openjdk.org> wrote:
>> test/jdk/javax/rmi/ssl/SSLSocketParametersTest.java line 192: >> >>> 190: throw exc; >>> 191: } >>> 192: } >> >> What is the story with the different exception catching here and in case 5? >> It doesn't look like a simple refactoring. Is another bug being fixed here? > > The expected result of cases 4 and 5 is that an IllegalArgumentException is > thrown because of the unsupported ciphersuite. The original code just caught > `Exception` which would hide a problem in the test e.g., if > `SSLContext.getDefault()` throws the NoSuchAlgorithmException. OK - but the run( ) method catches IllegalArgumentException already? ------------- PR: https://git.openjdk.org/jdk/pull/11910
