Merge pull request #144 from phpcr/rename-not-change-position

lsmith77 · lsmith77 · commit fb0faeb01308 · 2014-12-12T16:57:52.000+01:00
Ensure that rename does not change the position of a node among its siblings
diff --git a/tests/10_Writing/MoveMethodsTest.php b/tests/10_Writing/MoveMethodsTest.php
@@ -43,6 +43,9 @@ public function testNodeRename()
         $session = $this->saveAndRenewSession();
 
         $this->assertTrue($session->nodeExists('/tests_write_manipulation_move/testNodeRename/otherName'));
+
+        $parent = $session->getNode('/tests_write_manipulation_move/testNodeRename');
+        $this->assertEquals(array('otherName', 'secondNode'), (array) $parent->getNodeNames(), 'Rename may not change position of nodes');
     }
 
     public function testSessionMove()

Original file line number	Diff line number	Diff line change
`@@ -43,6 +43,9 @@ public function testNodeRename()`
`43`	`43`	`$session = $this->saveAndRenewSession();`
`44`	`44`
`45`	`45`	`$this->assertTrue($session->nodeExists('/tests_write_manipulation_move/testNodeRename/otherName'));`
	`46`	`+`
	`47`	`+ $parent = $session->getNode('/tests_write_manipulation_move/testNodeRename');`
	`48`	`+ $this->assertEquals(array('otherName', 'secondNode'), (array) $parent->getNodeNames(), 'Rename may not change position of nodes');`
`46`	`49`	`}`
`47`	`50`
`48`	`51`	`public function testSessionMove()`