Hi Chris, Changes looks good to me.
Thanks, Vyom On Tue, Apr 21, 2020 at 1:14 PM Chris Yin <xu.y....@oracle.com> wrote: > Hello > > Please review following changes to cleanup duplicated test ldap server in > some com/sun/jndi/ldap/ tests, thanks > > Bug: https://bugs.openjdk.java.net/browse/JDK-8242614 > Webrev: http://cr.openjdk.java.net/~xyin/8242614/webrev.00/ > > This is the first part to cleanup straightforward duplicated test ldap > server which was embedded in test itself. In the past, ldap tests may copy > or implemented a dummy server in test itself to support specific test > logic, so we have many duplicated dummy server logic across tests. Now > there is already a common test ldap server skeleton available, it’s time to > cleanup those duplicated logic to reduce possible future maintenance works. > Run modified tests on 4 platforms for total 200 times, all green. > > Thanks, > Chris -- Thanks, Vyom