On Tue, 8 Nov 2022 14:01:29 GMT, Sean Mullan <mul...@openjdk.org> wrote:
>> The java/security/Exceptions/ChainingConstructors jdk jtreg test shows quite
>> a lot of code duplication. The test code can be simplified e.g. by using
>> generics. Additionally the test failure output was improved and a test for
>> the constructors of CertPathBuilderException was added.
>
> test/jdk/java/security/Exceptions/ChainingConstructors.java line 208:
>
>> 206: throw new SecurityException("Test 17 failed");
>> 207: }
>> 208: */
>
> I don't know why this is commented out, but when I uncomment it, the test
> still passes, so I would include a test for `SSLException` below.
Hi Sean, the summary says that the test is about "java.security Exceptions"
so javax.net.ssl.SSLException.java from another package might not fit in,
maybe that's why it was commented out.
Should I still add it ? If so, the summary comment might need a little bit
adjustment, what about this "Test constructors for exception chaining of
security-related exceptions" ?
-------------
PR: https://git.openjdk.org/jdk/pull/11016
