test update

Author: dakujem

tests/tree.phpt

@@ -212,17 +212,17 @@ require_once __DIR__ . '/setup.php';
     $parent = new Node(null, children: $children);
     Assert::same($children, $parent->children()); // sanity check
 
-    // Attempting to add the same node under an existing (albeit same) key without duplicate node handling mechanism
-    // results in a ChildKeyCollision exception.
-    Assert::throws(function () use ($parent, $node1) {
-        Tree::link($node1, $parent, 'one', handleDuplicateLinking: false);
-    }, ChildKeyCollision::class);
+    // Attempting to add the same node under the same key without the duplicate node handling mechanism still has no effect.
+    // Also, attempting to add the same node under a different key without the duplicate node handling mechanism
+    // still results in a ChildKeyCollision exception.
+    Tree::link($node1, $parent, 'one', handleDuplicateLinking: false);
+    Assert::same($children, $parent->children());
     Assert::throws(function () use ($parent, $node1) {
         Tree::link($node1, $parent, 'two', handleDuplicateLinking: false);
     }, ChildKeyCollision::class);
 
     // Without the duplicate node handling mechanism, this call will allow adding the node as a duplicate child,
-    // possibly causing issues in the tree structure.
+    // possibly causing issues in the tree structure, but improving performance of a large number of subsequent calls.
     Tree::link($node1, $parent, 'three', handleDuplicateLinking: false);
     Assert::same(['one' => $node1, 'two' => $node2, 'three' => $node1], $parent->children());
 })();