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 re...@apache.org on 2016/07/14 15:15:19 UTC

svn commit: r1752672 - /jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/BasicDocumentStoreTest.java

Author: reschke
Date: Thu Jul 14 15:15:19 2016
New Revision: 1752672

URL: http://svn.apache.org/viewvc?rev=1752672&view=rev
Log:
OAK-4562: BasicDocumentStore max id test might return misleading results

Modified:
    jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/BasicDocumentStoreTest.java

Modified: jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/BasicDocumentStoreTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/BasicDocumentStoreTest.java?rev=1752672&r1=1752671&r2=1752672&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/BasicDocumentStoreTest.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/document/BasicDocumentStoreTest.java Thu Jul 14 15:15:19 2016
@@ -291,6 +291,11 @@ public class BasicDocumentStoreTest exte
         while (max - min >= 2) {
             test = (max + min) / 2;
             String id = generateId(test, ascii);
+            // make sure it's gone before trying to create it
+            try {
+                super.ds.remove(Collection.NODES, id);
+            } catch (DocumentStoreException ignored) {
+            }
             UpdateOp up = new UpdateOp(id, true);
             up.set("_id", id);
             boolean success = super.ds.create(Collection.NODES, Collections.singletonList(up));