adding tests for Node::getDefinition

Author: dbu

tests/NodeTypeDiscovery/NodeDefinitionTest.php

@@ -11,6 +11,10 @@
 
 namespace PHPCR\Tests\NodeTypeDiscovery;
 
+use PHPCR\NodeType\NodeDefinitionInterface;
+use PHPCR\NodeType\NodeTypeInterface;
+use PHPCR\NodeType\NodeTypeManagerInterface;
+
 /**
  * Test the NoteDefinition §8.
  *
@@ -19,16 +23,40 @@
 class NodeDefinitionTest extends \PHPCR\Test\BaseCase
 {
     private static $base;
+
+    /**
+     * @var NodeTypeInterface
+     */
     private static $file;
+
+    /**
+     * @var NodeTypeInterface
+     */
     private static $folder;
+
+    /**
+     * @var NodeTypeInterface
+     */
     private static $hierarchyNodeType;
-    /** jcr:content of nt:file */
+
+    /**
+     * Node definition of the jcr:content in an nt:file type.
+     *
+     * @var NodeDefinitionInterface
+     */
     private $content;
+
+    /**
+     * Node definition of a hierarchy node.
+     *
+     * @var NodeDefinitionInterface
+     */
     private $hierarchyNodeDef;
 
     public static function setupBeforeClass($fixtures = false)
     {
         parent::setupBeforeClass($fixtures);
+        /** @var NodeTypeManagerInterface $ntm */
         $ntm = self::$staticSharedFixture['session']->getWorkspace()->getNodeTypeManager();
         self::$file = $ntm->getNodeType('nt:file');
         self::$folder = $ntm->getNodeType('nt:folder');
@@ -37,6 +65,7 @@ public static function setupBeforeClass($fixtures = false)
 
     public function setUp()
     {
+        parent::setUp();
         try {
             $defs = self::$file->getChildNodeDefinitions();
             $this->assertInternalType('array', $defs);
@@ -127,4 +156,25 @@ public function testIsProtected()
         $this->assertFalse($this->content->isProtected());
         $this->assertFalse($this->hierarchyNodeDef->isProtected());
     }
+
+    // some tests about Node::getDefinition()
+
+    public function testGetNodeDefinitionExact()
+    {
+        // an nt:file must have a jcr:content property
+        $node = $this->rootNode->getNode('tests_general_base/numberPropertyNode/jcr:content');
+        $nodeDef = $node->getDefinition();
+        $this->assertInstanceOf('PHPCR\\NodeType\\NodeDefinitionInterface', $nodeDef);
+        $this->assertEquals('jcr:content', $nodeDef->getName());
+        $this->assertTrue($nodeDef->isMandatory());
+    }
+
+    public function testGetNodeDefinitionWildcard()
+    {
+        // defines a child of nt:folder
+        $node = $this->rootNode->getNode('tests_general_base/index.txt');
+        $nodeDef = $node->getDefinition();
+        $this->assertInstanceOf('PHPCR\\NodeType\\NodeDefinitionInterface', $nodeDef);
+        $this->assertEquals('*', $nodeDef->getName());
+    }
 }

tests/NodeTypeDiscovery/PropertyDefinitionTest.php

@@ -11,6 +11,9 @@
 
 namespace PHPCR\Tests\NodeTypeDiscovery;
 
+use PHPCR\NodeType\NodeTypeInterface;
+use PHPCR\NodeType\NodeTypeManagerInterface;
+use PHPCR\NodeType\PropertyDefinitionInterface;
 use PHPCR\Query\QOM\QueryObjectModelConstantsInterface;
 
 /**
@@ -20,26 +23,77 @@
  */
 class PropertyDefinitionTest extends \PHPCR\Test\BaseCase
 {
+    /**
+     * @var NodeTypeInterface
+     */
     private static $base;
+
+    /**
+     * @var NodeTypeInterface
+     */
     private static $address;
+
+    /**
+     * @var NodeTypeInterface
+     */
     private static $mix_created;
+
+    /**
+     * @var NodeTypeInterface
+     */
     private static $resource;
 
-    /** nt:base property */
-    private $primaryType; // (NAME) mandatory autocreated protected COMPUTE
-    private $mixinTypes; // (NAME) protected multiple COMPUTE
+    // properties of nt:base
+    /**
+     * (NAME) mandatory autocreated protected COMPUTE
+     * @var PropertyDefinitionInterface
+     */
+    private $primaryType;
+
+    /**
+     * (NAME) protected multiple COMPUTE
+     * @var PropertyDefinitionInterface
+     */
+    private $mixinTypes;
+
     /** properties of nt:address */
-    private $workspace; // (STRING)
-    private $pathprop; // (PATH)
-    private $id; // (WEAKREFERENCE)
-    /** property of mix:created */
-    private $created; // (DATE) autocreated protected
-    /** property of nt:resource */
-    private $data; // (BINARY) mandatory
+
+    /**
+     * (STRING)
+     * @var PropertyDefinitionInterface
+     */
+    private $workspace;
+
+    /**
+     * (PATH)
+     * @var PropertyDefinitionInterface
+     */
+    private $pathprop;
+
+    /**
+     * (WEAKREFERENCE)
+     * @var PropertyDefinitionInterface
+     */
+    private $id;
+
+    /**
+     * (DATE) autocreated protected
+     * property of mix:created
+     * @var PropertyDefinitionInterface
+     */
+    private $created; //
+
+    /**
+     * (BINARY) mandatory
+     * property of nt:resource
+     * @var PropertyDefinitionInterface
+     */
+    private $data; //
 
     public static function setupBeforeClass($fixtures = false)
     {
         parent::setupBeforeClass(); // load default fixtures
+        /** @var NodeTypeManagerInterface $ntm */
         $ntm = self::$staticSharedFixture['session']->getWorkspace()->getNodeTypeManager();
         self::$base = $ntm->getNodeType('nt:base');
         self::$address = $ntm->getNodeType('nt:address');
@@ -238,7 +292,7 @@ public function testGetPropertyDefinitionExact()
         $this->assertEquals('jcr:created', $propDef->getName());
     }
 
-    public function estGetPropertyDefinitionWildcard()
+    public function testGetPropertyDefinitionWildcard()
     {
         $node = $this->rootNode->getNode('tests_general_base/numberPropertyNode/jcr:content');
         $valProperty = $node->getProperty('foo');