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/03/26 14:39:27 UTC

svn commit: r758635 - /jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/api/jsr283/lock/AbstractLockTest.java

Author: angela
Date: Thu Mar 26 13:39:18 2009
New Revision: 758635

URL: http://svn.apache.org/viewvc?rev=758635&view=rev
Log:
test: check for supported option before general test setup (incl. session creation) is executed

Modified:
    jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/api/jsr283/lock/AbstractLockTest.java

Modified: jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/api/jsr283/lock/AbstractLockTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/api/jsr283/lock/AbstractLockTest.java?rev=758635&r1=758634&r2=758635&view=diff
==============================================================================
--- jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/api/jsr283/lock/AbstractLockTest.java (original)
+++ jackrabbit/trunk/jackrabbit-core/src/test/java/org/apache/jackrabbit/api/jsr283/lock/AbstractLockTest.java Thu Mar 26 13:39:18 2009
@@ -45,13 +45,11 @@
     protected Lock lock;
 
     protected void setUp() throws Exception {
+        // check for lock support before creating the session in the super.setup
+        checkSupportedOption(Repository.OPTION_LOCKING_SUPPORTED);
+        
         super.setUp();
-
-        // check for lock support
-        if (Boolean.FALSE.toString().equals(superuser.getRepository().getDescriptor(Repository.OPTION_LOCKING_SUPPORTED))) {
-            throw new NotExecutableException();
-        }
-
+        
         lockedNode = testRootNode.addNode(nodeName1, testNodeType);
         lockedNode.addMixin(mixLockable);
         childNode = lockedNode.addNode(nodeName2, testNodeType);