From 1a3308c417e4f8678b89bb1cbbd3e7c5ff6c2b83 Mon Sep 17 00:00:00 2001
From: Willem-Jan Zijderveld <willemjan@beeldspraak.com>
Date: Thu, 19 Dec 2013 14:36:54 +0100
Subject: [PATCH] Added test for reusing a path with a different Identifier

---
 fixtures/10_Writing/delete.xml         | 18 +++++++++++++
 tests/10_Writing/DeleteMethodsTest.php | 37 ++++++++++++++++++++++++++
 2 files changed, 55 insertions(+)

diff --git a/fixtures/10_Writing/delete.xml b/fixtures/10_Writing/delete.xml
index c8afb902..348017bd 100644
--- a/fixtures/10_Writing/delete.xml
+++ b/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>
diff --git a/tests/10_Writing/DeleteMethodsTest.php b/tests/10_Writing/DeleteMethodsTest.php
index 26abfb13..e5c694db 100644
--- a/tests/10_Writing/DeleteMethodsTest.php
+++ b/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