[15] RFR: 8242614: cleanup duplicated test ldap server in some com/sun/jndi/ldap/ tests

[Daniel Fuchs]

Hi Chris,

Thanks for this cleanup.
The proposed changes look reasonable and worthwhile.

best regards,

-- daniel

On 21/04/2020 08:43, Chris Yin 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
>