Check for invalid node values on delete (#9)

Author: kimpepper

src/Storage/DbalNestedSet.php

@@ -230,6 +230,9 @@ public function getTree() {
    * {@inheritdoc}
    */
   public function deleteNode(Node $node) {
+    if ($node->getLeft() < 1 || $node->getRight() < 1) {
+      throw new \InvalidArgumentException("Left and right values must be > 0");
+    }
     $left = $node->getLeft();
     $right = $node->getRight();
     $width = $right - $left + 1;

tests/Functional/DbalNestedSetTest.php

@@ -281,6 +281,16 @@ public function testDeleteNode() {
     $tree = $this->nestedSet->getTree();
   }
 
+  /**
+   * Tests deleting a node with missing values.
+   *
+   * @expectedException \InvalidArgumentException
+   */
+  public function testDeleteNodeInvalid() {
+    $node = new Node(1, 1);
+    $this->nestedSet->deleteNode($node);
+  }
+
   /**
    * Tests deleting a node and its sub-tree.
    */