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