svn commit: r1485729 - in /jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/index/property: EmptyPropertyTest.java MultipleIndicesTest.java

[mr...@apache.org]

Author: mreutegg
Date: Thu May 23 14:28:54 2013
New Revision: 1485729

URL: http://svn.apache.org/r1485729
Log:
OAK-841: Indexing empty string fails with "there's already a child node with name ':index'"
- Move test to MultipleIndicesTest

Removed:
    jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/index/property/EmptyPropertyTest.java
Modified:
    jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/index/property/MultipleIndicesTest.java

Modified: jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/index/property/MultipleIndicesTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/index/property/MultipleIndicesTest.java?rev=1485729&r1=1485728&r2=1485729&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/index/property/MultipleIndicesTest.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/index/property/MultipleIndicesTest.java Thu May 23 14:28:54 2013
@@ -22,6 +22,7 @@ import static org.apache.jackrabbit.oak.
 import java.util.ArrayList;
 
 import org.apache.jackrabbit.oak.Oak;
+import org.apache.jackrabbit.oak.api.CommitFailedException;
 import org.apache.jackrabbit.oak.api.ContentRepository;
 import org.apache.jackrabbit.oak.api.Tree;
 import org.apache.jackrabbit.oak.plugins.nodetype.write.InitialContent;
@@ -30,6 +31,7 @@ import org.apache.jackrabbit.oak.spi.lif
 import org.apache.jackrabbit.oak.spi.security.OpenSecurityProvider;
 import org.apache.jackrabbit.oak.spi.state.NodeBuilder;
 import org.apache.jackrabbit.oak.spi.state.NodeState;
+import org.junit.Ignore;
 import org.junit.Test;
 
 import com.google.common.collect.ImmutableList;
@@ -88,4 +90,19 @@ public class MultipleIndicesTest extends
                 ImmutableList.of("/content/y"));
         setTravesalFallback(true);
     }
+
+    /**
+     * Test for OAK-841
+     */
+    @Ignore
+    @Test
+    public void emptyStringValue() throws CommitFailedException {
+        Tree t = root.getTree("/");
+        t.addChild("node-1").setProperty("pid", "value");
+        root.commit();
+
+        t = root.getTree("/");
+        t.addChild("node-2").setProperty("pid", "");
+        root.commit();
+    }
 }