Merge pull request #122 from wjzijderveld/test-for-dbal-pr-158

Added test for reusing a path with a different Identifier

Author: dbu

fixtures/10_Writing/delete.xml

@@ -230,6 +230,24 @@
     </sv:node>
   </sv:node>
 
+    <sv:node sv:name="testDeleteNodeAndReusePathWithReference">
+        <sv:property sv:name="jcr:primaryType" sv:type="Name">
+            <sv:value>nt:unstructured</sv:value>
+        </sv:property>
+
+        <sv:node sv:name="idExample">
+            <sv:property sv:name="jcr:primaryType" sv:type="Name">
+                <sv:value>nt:unstructured</sv:value>
+            </sv:property>
+            <sv:property sv:name="jcr:mixinTypes" sv:type="Name">
+                <sv:value>mix:referenceable</sv:value>
+            </sv:property>
+            <sv:property sv:name="jcr:uuid" sv:type="String">
+                <sv:value>cbc172b2-c317-44ac-a73b-1df61c35fb1a</sv:value>
+            </sv:property>
+        </sv:node>
+    </sv:node>
+
     <sv:node sv:name="testWorkspaceDelete">
         <sv:property sv:name="jcr:primaryType" sv:type="Name">
             <sv:value>nt:unstructured</sv:value>

tests/10_Writing/DeleteMethodsTest.php

@@ -450,6 +450,43 @@ public function testDeleteWeakReferencedNode()
         $this->assertFalse($this->node->hasNode('idExample'));
     }
 
+    /**
+     * test if deleting a node and creating a node at the same path with a new UUID
+     * won't cause trouble with internally cached UUID's
+     */
+    public function testDeleteNodeAndReusePathWithReference()
+    {
+        //relies on the base class setup trick to have the node populated from the fixtures
+        $this->assertInstanceOf('PHPCR\NodeInterface', $this->node);
+
+        // 1. remove the idExample node with UUID cbc172b2-c317-44ac-a73b-1df61c35fb1a
+        $referencedNode = $this->node->getNode('idExample');
+        $referencedNode->remove();
+
+        // 2. Save the session (without reloading)
+        $this->session->save();
+
+        // 3. Recreate the node with a specific UUID
+        $referencedNode = $this->node->addNode('idExample');
+        // Add mixin to make it referenceable
+        $referencedNode->addMixin('mix:referenceable');
+        // Change Identifier from the one in the fixtures
+        $referencedNode->setProperty('jcr:uuid', '54378257-ca4d-4b9f-9383-f30dfb280977');
+
+        // Node should be persisted before using it as a reference
+        $this->session->save();
+
+        // 4. Give the testNode a reference to the new idExample node
+        $this->node->setProperty('reference', '54378257-ca4d-4b9f-9383-f30dfb280977', \PHPCR\PropertyType::REFERENCE);
+
+        // 5. Throws an PHPCR\ReferentialIntegrityException when above UUID is not a valid reference
+        $this->saveAndRenewSession();
+
+        $this->assertInstanceOf('PHPCR\NodeInterface', $this->node);
+        $this->assertEquals('54378257-ca4d-4b9f-9383-f30dfb280977', $this->node->getProperty('reference')->getString(), 'Reference property should contain "54378257-ca4d-4b9f-9383-f30dfb280977" as string value');
+        $this->assertEquals('54378257-ca4d-4b9f-9383-f30dfb280977', $this->node->getNode('idExample')->getIdentifier(), 'idExample node should have "54378257-ca4d-4b9f-9383-f30dfb280977" as UUID');
+    }
+
     public function testWorkspaceDelete()
     {
         //relies on the base class setup trick to have the node populated from the fixtures