directory-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Emmanuel LŽcharny <elecha...@gmail.com>
Subject Re: svn commit: r891807 - in /directory/shared/branches/shared-schema/ldap/src: main/java/org/apache/directory/shared/ldap/name/Rdn.java test/java/org/apache/directory/shared/ldap/name/RdnTest.java
Date Thu, 17 Dec 2009 19:23:07 GMT
Stefan Seelmann a écrit :
> Hi,
>> +    @Test
>> +    public void testRDNCreation() throws InvalidNameException
>> +    {
>> +        Rdn rdn = new Rdn( "A", "  b  " );
>> +        assertEquals( "a=\\  b \\ ", rdn.getNormName() );
>> +        assertEquals( "A=  b  ", rdn.getUpName() );
>> +    }
>>   
> IMO this test is wrong. The UpName should be "A=\\ b \\ ". Leading and 
> trailing spaces in the AttributeValue must be escaped when converting 
> it to a String, see RFC 4514 Section 2.4.

Sadly, this test is not wrong :/ It's the RDN rendering of UP values 
which is wrong ... As soon as we fix it, then the test will be wrong, I 
agree :)

>


Mime
View raw message