You are viewing a plain text version of this content. The canonical link for it is here.
Posted to oak-commits@jackrabbit.apache.org by an...@apache.org on 2013/01/08 18:20:49 UTC

svn commit: r1430394 - /jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/security/user/CreateGroupTest.java

Author: angela
Date: Tue Jan  8 17:20:49 2013
New Revision: 1430394

URL: http://svn.apache.org/viewvc?rev=1430394&view=rev
Log:
OAK-50 : Implement User Management 

- simplify test teardown

Modified:
    jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/security/user/CreateGroupTest.java

Modified: jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/security/user/CreateGroupTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/security/user/CreateGroupTest.java?rev=1430394&r1=1430393&r2=1430394&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/security/user/CreateGroupTest.java (original)
+++ jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/security/user/CreateGroupTest.java Tue Jan  8 17:20:49 2013
@@ -26,16 +26,12 @@ import org.apache.jackrabbit.api.securit
 import org.apache.jackrabbit.api.security.user.Group;
 import org.apache.jackrabbit.test.NotExecutableException;
 import org.junit.Test;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 /**
  * CreateGroupTest...
  */
 public class CreateGroupTest extends AbstractUserTest {
 
-    private static Logger log = LoggerFactory.getLogger(CreateGroupTest.class);
-
     private List<Authorizable> createdGroups = new ArrayList();
 
     @Override
@@ -45,12 +41,10 @@ public class CreateGroupTest extends Abs
             try {
                 createdGroup.remove();
                 superuser.save();
-            } catch (RepositoryException e) {
-                log.error("Failed to remove Group " + createdGroup.getID() + " during tearDown.");
+            } finally {
+                super.tearDown();
             }
         }
-
-        super.tearDown();
     }
 
     private Group createGroup(Principal p) throws RepositoryException {