RFR: JDK-8296406: ChainingConstructors jtreg test reduce code duplication
Sean Mullan
mullan at openjdk.org
Tue Nov 8 15:56:38 UTC 2022
On Tue, 8 Nov 2022 15:15:38 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:
>> 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" ?
Good point. I think it is ok to make an exception (no pun intended) for `SSLException` for this test. Changing the summary comment to your suggestion is fine with me.
-------------
PR: https://git.openjdk.org/jdk/pull/11016
More information about the security-dev
mailing list