You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@jackrabbit.apache.org by an...@apache.org on 2009/10/29 19:11:59 UTC

svn commit: r831052 - /jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/security/user/AuthorizableImplTest.java

Author: angela
Date: Thu Oct 29 18:11:59 2009
New Revision: 831052

URL: http://svn.apache.org/viewvc?rev=831052&view=rev
Log:
JCR-2313 - Improvements to user management (2) [work in progress] 

- tests

Modified:
    jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/security/user/AuthorizableImplTest.java

Modified: jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/security/user/AuthorizableImplTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/security/user/AuthorizableImplTest.java?rev=831052&r1=831051&r2=831052&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/security/user/AuthorizableImplTest.java (original)
+++ jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/core/security/user/AuthorizableImplTest.java Thu Oct 29 18:11:59 2009
@@ -154,8 +154,13 @@
         GroupImpl gr = (GroupImpl) getTestGroup(superuser);
         NodeImpl n = gr.getNode();
 
-        if (n.hasProperty(UserConstants.P_MEMBERS)) {
-            assertEquals(PropertyType.WEAKREFERENCE, n.getProperty(UserConstants.P_MEMBERS).getType());
+        if (!n.hasProperty(UserConstants.P_MEMBERS)) {
+            gr.addMember(getTestUser(superuser));
+        }
+
+        Property p = n.getProperty(UserConstants.P_MEMBERS);
+        for (Value v : p.getValues()) {
+            assertEquals(PropertyType.WEAKREFERENCE, v.getType());
         }
     }