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 12:35:10 UTC
svn commit: r1469271 -
/jackrabbit/oak/trunk/oak-core/src/test/java/org/apache/jackrabbit/oak/plugins/memory/MemoryNodeBuilderTest.java
Author: mduerig
Date: Thu Apr 18 10:35:09 2013
New Revision: 1469271
URL: http://svn.apache.org/r1469271
Log:
OAK-782: MemoryNodeBuilder.setNode() loses property values
updated test case to demonstrate cases for NodeState instances other then ModifiedNodeState
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=1469271&r1=1469270&r2=1469271&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 10:35:09 2013
@@ -201,20 +201,18 @@ public class MemoryNodeBuilderTest {
@Ignore("OAK-782")
public void setNodeTest() {
MemoryNodeBuilder rootBuilder = new MemoryNodeBuilder(EmptyNodeState.EMPTY_NODE);
- MemoryNodeBuilder aBuilder = new MemoryNodeBuilder(EmptyNodeState.EMPTY_NODE);
- aBuilder.child("b").setProperty("c", "cValue");
- rootBuilder.setNode("a", aBuilder.getNodeState());
+ rootBuilder.setNode("a", createBC(true));
// Note: using the following way to construct the initial state makes the test pass
// rootBuilder = new MemoryNodeBuilder(EmptyNodeState.EMPTY_NODE);
-// rootBuilder.child("a").child("b").setProperty("c", "cValue");
+// rootBuilder.child("a").child("c").setProperty("c", "cValue");
- NodeState b = rootBuilder.getNodeState().getChildNode("a").getChildNode("b");
+ NodeState b = rootBuilder.getNodeState().getChildNode("a").getChildNode("c");
assertTrue(b.hasProperty("c"));
- rootBuilder.child("a").child("b").setProperty("c2", "c2Value");
+ rootBuilder.child("a").child("c").setProperty("c2", "c2Value");
- b = rootBuilder.getNodeState().getChildNode("a").getChildNode("b");
+ b = rootBuilder.getNodeState().getChildNode("a").getChildNode("c");
assertTrue(b.hasProperty("c"));
assertTrue(b.hasProperty("c2"));
}