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 md...@apache.org on 2013/04/18 15:19:58 UTC

svn commit: r1469309 - /jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/memory/MemoryNodeBuilderTest.java

Author: mduerig
Date: Thu Apr 18 13:19:58 2013
New Revision: 1469309

URL: http://svn.apache.org/r1469309
Log:
OAK-781: Clarify / fix effects of MISSING_NODE as base state of NodeBuilder
Update test case according to the latest discussion on OAK-781

Modified:
    jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/memory/MemoryNodeBuilderTest.java

Modified: jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/memory/MemoryNodeBuilderTest.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/memory/MemoryNodeBuilderTest.java?rev=1469309&r1=1469308&r2=1469309&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/memory/MemoryNodeBuilderTest.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/memory/MemoryNodeBuilderTest.java Thu Apr 18 13:19:58 2013
@@ -261,10 +261,10 @@ public class MemoryNodeBuilderTest {
         c = b.getChildNode("c");
 
         assertTrue(a.exists());
-        assertFalse(b.exists());  // FIXME fails
+        assertTrue(b.exists());  // shadowed by above child("b")
         assertTrue(c.exists());
 
-        assertTrue(c.hasProperty("c"));
+        assertFalse(c.hasProperty("c"));  // shadowed by subtree b
         assertTrue(c.hasProperty("c2"));
     }