[directory-server] branch apacheds-value updated: DIRSERVER-2109: Enable tests and assertions

[se...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

seelmann pushed a commit to branch apacheds-value
in repository https://gitbox.apache.org/repos/asf/directory-server.git


The following commit(s) were added to refs/heads/apacheds-value by this push:
     new 7d4eee0  DIRSERVER-2109: Enable tests and assertions
7d4eee0 is described below

commit 7d4eee0b49116ad3fe98d1fdf54f20b56af1a470
Author: Stefan Seelmann <ma...@stefan-seelmann.de>
AuthorDate: Mon Dec 11 22:21:31 2017 +0100

    DIRSERVER-2109: Enable tests and assertions
---
 .../java/org/apache/directory/server/core/operations/add/AddIT.java | 2 --
 .../org/apache/directory/server/core/integ/TestClassServer.java     | 6 +++---
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/core-integ/src/test/java/org/apache/directory/server/core/operations/add/AddIT.java b/core-integ/src/test/java/org/apache/directory/server/core/operations/add/AddIT.java
index 2fa6586..2cadf46 100644
--- a/core-integ/src/test/java/org/apache/directory/server/core/operations/add/AddIT.java
+++ b/core-integ/src/test/java/org/apache/directory/server/core/operations/add/AddIT.java
@@ -143,7 +143,6 @@ public class AddIT extends AbstractLdapTestUnit
      * Test for DIRSERVER-2109.
      */
     @Test
-    @Ignore
     public void test_DIRSERVER_2109_1() throws Exception
     {
         Dn dn = new Dn( "cn=\\#\\\\\\+\\, \\\"\u00F6\u00E9\\\",ou=users,ou=system" );
@@ -166,7 +165,6 @@ public class AddIT extends AbstractLdapTestUnit
      * Test for DIRSERVER-2109.
      */
     @Test
-    @Ignore
     public void test_DIRSERVER_2109_2() throws Exception
     {
         Entry entry = new DefaultEntry( new Dn( "cn=a\\\\b,ou=users,ou=system" ) );
diff --git a/test-framework/src/test/java/org/apache/directory/server/core/integ/TestClassServer.java b/test-framework/src/test/java/org/apache/directory/server/core/integ/TestClassServer.java
index 7c79116..82d1a33 100644
--- a/test-framework/src/test/java/org/apache/directory/server/core/integ/TestClassServer.java
+++ b/test-framework/src/test/java/org/apache/directory/server/core/integ/TestClassServer.java
@@ -19,6 +19,7 @@
 package org.apache.directory.server.core.integ;
 
 
+import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertTrue;
 
 import org.apache.directory.api.ldap.model.entry.Entry;
@@ -41,8 +42,7 @@ public class TestClassServer extends AbstractLdapTestUnit
 
         assertTrue( getService().getAdminSession().exists( new Dn( "cn=\\#\\\\\\+\\, \\\"\u00F6\u00E9\\\",ou=users,ou=system" ) ) );
         Entry entry = getService().getAdminSession().lookup( new Dn("cn=\\#\\\\\\+\\, \\\"\u00F6\u00E9\\\",ou=users,ou=system") );
-        //System.out.println( entry );
-        //assertEquals(1, entry.get( "cn" ).size());
-        //assertEquals("#\\+, \"\u00F6\u00E9\"", entry.get( "cn" ).get().getString());
+        assertEquals(1, entry.get( "cn" ).size());
+        assertEquals("#\\+, \"\u00F6\u00E9\"", entry.get( "cn" ).getString());
     }
 }

-- 
To stop receiving notification emails like this one, please contact
['"commits@directory.apache.org" <co...@directory.apache.org>'].