RFR: 8301367: Add exception handler method to the BaseLdapServer
Daniel Fuchs
dfuchs at openjdk.org
Thu Feb 2 09:56:21 UTC 2023
On Tue, 31 Jan 2023 16:19:34 GMT, Aleksei Efimov <aefimov at openjdk.org> wrote:
> The proposed change adds a new exception handler method to the `test/jdk/com/sun/jndi/ldap/lib/BaseLdapServer.java` LDAP test library class. It will allow LDAP tests to customize the handling of server-side exceptions.
> The current `BaseLdapTestServer` implementation prints an exception and its stack trace to the standard error stream.
>
> Existing tests in `test/jdk/com/sun/jndi/ldap` that use the modified library class are passing with the modified version.
Like Jaikiran I don't like the name of the new method much - but anything I could come up with was not much better. At least the new method gets both the socket and the exception as parameter, so that makes the name more reasonable. So approved as it stands!
-------------
Marked as reviewed by dfuchs (Reviewer).
PR: https://git.openjdk.org/jdk/pull/12347
More information about the core-libs-dev
mailing list