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

[Emmanuel LŽcharny]

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 :)