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 ju...@apache.org on 2013/07/16 11:08:58 UTC

svn commit: r1503621 - /jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/ConcurrentAddIT.java

Author: jukka
Date: Tue Jul 16 09:08:58 2013
New Revision: 1503621

URL: http://svn.apache.org/r1503621
Log:
OAK-786: Fall back to pessimism

Enable ConcurrentAddIT also for the SegmentMK

Modified:
    jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/ConcurrentAddIT.java

Modified: jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/ConcurrentAddIT.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/ConcurrentAddIT.java?rev=1503621&r1=1503620&r2=1503621&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/ConcurrentAddIT.java (original)
+++ jackrabbit/oak/trunk/oak-jcr/src/test/java/org/apache/jackrabbit/oak/jcr/ConcurrentAddIT.java Tue Jul 16 09:08:58 2013
@@ -18,20 +18,20 @@ package org.apache.jackrabbit.oak.jcr;
 
 import java.util.ArrayList;
 import java.util.Collections;
+import java.util.Iterator;
 import java.util.List;
 
 import javax.jcr.Node;
 import javax.jcr.RepositoryException;
 import javax.jcr.Session;
 
-import org.apache.jackrabbit.commons.iterator.NodeIterable;
 import org.junit.Test;
 
-import com.google.common.collect.Iterables;
+import com.google.common.collect.Iterators;
 
+import static org.apache.jackrabbit.commons.JcrUtils.in;
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.fail;
-import static org.junit.Assume.assumeTrue;
 
 /**
  * <code>ConcurrentAddIT</code> adds nodes with multiple sessions in separate
@@ -47,10 +47,8 @@ public class ConcurrentAddIT extends Abs
         super(fixture);
     }
 
-    @Test
+    @Test @SuppressWarnings("unchecked")
     public void addNodes() throws Exception {
-        // OAK-786
-        assumeTrue(fixture != NodeStoreFixture.SEGMENT_MK);
         List<Exception> exceptions = Collections.synchronizedList(
                 new ArrayList<Exception>());
         Node test = getAdminSession().getRootNode().addNode("test");
@@ -71,8 +69,8 @@ public class ConcurrentAddIT extends Abs
             fail(e.toString());
         }
         getAdminSession().refresh(false);
-        for (Node n : new NodeIterable(test.getNodes())) {
-            assertEquals(NODES_PER_WORKER, Iterables.size(new NodeIterable(n.getNodes())));
+        for (Node n : in((Iterator<Node>) test.getNodes())) {
+            assertEquals(NODES_PER_WORKER, Iterators.size(n.getNodes()));
         }
     }