You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by se...@apache.org on 2007/08/19 11:41:39 UTC
svn commit: r567376 -
/directory/shared/trunk/ldap/src/test/java/org/apache/directory/shared/ldap/schema/syntax/ACIItemSyntaxCheckerTest.java
Author: seelmann
Date: Sun Aug 19 02:41:38 2007
New Revision: 567376
URL: http://svn.apache.org/viewvc?view=rev&rev=567376
Log:
Fix for DIRSERVER-891, uncommentend a test.
Modified:
directory/shared/trunk/ldap/src/test/java/org/apache/directory/shared/ldap/schema/syntax/ACIItemSyntaxCheckerTest.java
Modified: directory/shared/trunk/ldap/src/test/java/org/apache/directory/shared/ldap/schema/syntax/ACIItemSyntaxCheckerTest.java
URL: http://svn.apache.org/viewvc/directory/shared/trunk/ldap/src/test/java/org/apache/directory/shared/ldap/schema/syntax/ACIItemSyntaxCheckerTest.java?view=diff&rev=567376&r1=567375&r2=567376
==============================================================================
--- directory/shared/trunk/ldap/src/test/java/org/apache/directory/shared/ldap/schema/syntax/ACIItemSyntaxCheckerTest.java (original)
+++ directory/shared/trunk/ldap/src/test/java/org/apache/directory/shared/ldap/schema/syntax/ACIItemSyntaxCheckerTest.java Sun Aug 19 02:41:38 2007
@@ -238,48 +238,48 @@
assertTrue( checker.isValidSyntax( spec ) );
}
-// /**
-// * Test case for DIRSERVER-891
-// */
-// public void testInvalidAttributeValue()
-// {
-// String spec;
-//
-// // no name-value-pair
-// spec = "{ identificationTag \"id2\" , precedence 14, authenticationLevel none , "
-// + "itemOrUserFirst userFirst: { userPermissions { { protectedItems{ entry , attributeType { cn , ou } , attributeValue { must_be_a_name_value_pair } , "
-// + "rangeOfValues (cn=ErsinEr) } , grantsAndDenials { grantBrowse } } }, userClasses { allUsers , name { \"ou=people,cn=ersin\" }, "
-// + "subtree {{ minimum 7, maximum 9, base \"ou=system\" }, { base \"ou=ORGANIZATIONUNIT\","
-// + " maximum 2, minimum 1 } } } } } ";
-// assertFalse( checker.isValidSyntax( spec ) );
-//
-// // no name-value-pair
-// spec = "{ identificationTag \"id2\" , precedence 14, authenticationLevel none , "
-// + "itemOrUserFirst userFirst: { userPermissions { { protectedItems{ entry , attributeType { cn , ou } , attributeValue { x=y,m=n,k=l,x } , "
-// + "rangeOfValues (cn=ErsinEr) } , grantsAndDenials { grantBrowse } } }, userClasses { allUsers , name { \"ou=people,cn=ersin\" }, "
-// + "subtree {{ minimum 7, maximum 9, base \"ou=system\" }, { base \"ou=ORGANIZATIONUNIT\","
-// + " maximum 2, minimum 1 } } } } } ";
-// assertFalse( checker.isValidSyntax( spec ) );
-// }
-//
-// /**
-// * Test case for DIRSERVER-891
-// */
-// public void testIncomplete()
-// {
-// String spec;
-//
-// spec = "{ }";
-// assertFalse( checker.isValidSyntax( spec ) );
-//
-// spec = "{ identificationTag \"id2\" }";
-// assertFalse( checker.isValidSyntax( spec ) );
-//
-// spec = "{ identificationTag \"id2\", precedence 14 } ";
-// assertFalse( checker.isValidSyntax( spec ) );
-//
-// spec = "{ identificationTag \"id2\", precedence 14, authenticationLevel none } ";
-// assertFalse( checker.isValidSyntax( spec ) );
-// }
+ /**
+ * Test case for DIRSERVER-891
+ */
+ public void testInvalidAttributeValue()
+ {
+ String spec;
+
+ // no name-value-pair
+ spec = "{ identificationTag \"id2\" , precedence 14, authenticationLevel none , "
+ + "itemOrUserFirst userFirst: { userPermissions { { protectedItems{ entry , attributeType { cn , ou } , attributeValue { must_be_a_name_value_pair } , "
+ + "rangeOfValues (cn=ErsinEr) } , grantsAndDenials { grantBrowse } } }, userClasses { allUsers , name { \"ou=people,cn=ersin\" }, "
+ + "subtree {{ minimum 7, maximum 9, base \"ou=system\" }, { base \"ou=ORGANIZATIONUNIT\","
+ + " maximum 2, minimum 1 } } } } } ";
+ assertFalse( checker.isValidSyntax( spec ) );
+
+ // no name-value-pair
+ spec = "{ identificationTag \"id2\" , precedence 14, authenticationLevel none , "
+ + "itemOrUserFirst userFirst: { userPermissions { { protectedItems{ entry , attributeType { cn , ou } , attributeValue { x=y,m=n,k=l,x } , "
+ + "rangeOfValues (cn=ErsinEr) } , grantsAndDenials { grantBrowse } } }, userClasses { allUsers , name { \"ou=people,cn=ersin\" }, "
+ + "subtree {{ minimum 7, maximum 9, base \"ou=system\" }, { base \"ou=ORGANIZATIONUNIT\","
+ + " maximum 2, minimum 1 } } } } } ";
+ assertFalse( checker.isValidSyntax( spec ) );
+ }
+
+ /**
+ * Test case for DIRSERVER-891
+ */
+ public void testIncomplete()
+ {
+ String spec;
+
+ spec = "{ }";
+ assertFalse( checker.isValidSyntax( spec ) );
+
+ spec = "{ identificationTag \"id2\" }";
+ assertFalse( checker.isValidSyntax( spec ) );
+
+ spec = "{ identificationTag \"id2\", precedence 14 } ";
+ assertFalse( checker.isValidSyntax( spec ) );
+
+ spec = "{ identificationTag \"id2\", precedence 14, authenticationLevel none } ";
+ assertFalse( checker.isValidSyntax( spec ) );
+ }
}