On Wed, 16 Sep 2026 11:28:07 GMT, Timofei Fedotov <[email protected]> wrote:

>> test/jdk/javax/naming/ldap/Rdn/EmptyRdnTest.java line 1:
>> 
>>> 1: /*
>> 
>> I believe the test should also check the other methods by which an empty RDN 
>> might be created, even if those currently appropriately throw an 
>> InvalidNameException.
>> (e.g. `new Rdn(new BasicAttributes())`)
>> 
>> We also need a scenario that deserializes an empty RDN() (bytes that were 
>> formed on a previous version where new Rdn("") didn't throw) and verify that 
>> StreamCorruptedException is raised.
>
> done

Thanks!

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/32648#discussion_r4025991566

Reply via email to