Merge pull request #74 from jaapio/feature/interfaceExtends

add interface parents

Author: mvriel

src/phpDocumentor/Reflection/Php/Factory/Interface_.php

@@ -14,6 +14,7 @@
 namespace phpDocumentor\Reflection\Php\Factory;
 use InvalidArgumentException;
 use phpDocumentor\Reflection\Element;
+use phpDocumentor\Reflection\Fqsen;
 use phpDocumentor\Reflection\Php\ProjectFactoryStrategy;
 use phpDocumentor\Reflection\Php\StrategyContainer;
 use phpDocumentor\Reflection\Types\Context;
@@ -63,8 +64,12 @@ public function create($object, StrategyContainer $strategies, Context $context
         }
 
         $docBlock = $this->createDocBlock($object->getDocComment(), $strategies, $context);
+        $parents = array();
+        foreach ($object->extends as $extend) {
+            $parents['\\' . (string)$extend] = new Fqsen('\\' . (string)$extend);
+        }
 
-        $interface = new InterfaceElement($object->fqsen, $docBlock);
+        $interface = new InterfaceElement($object->fqsen, $parents, $docBlock);
 
         if (isset($object->stmts)) {
             foreach ($object->stmts as $stmt) {

src/phpDocumentor/Reflection/Php/Interface_.php

@@ -36,16 +36,21 @@ final class Interface_ implements Element
     /** @var Method[] $methods */
     protected $methods = array();
 
+    /** @var Fqsen[] $parents */
+    protected $parents = array();
+
     /**
      * Initializes the object.
      *
      * @param Fqsen $fqsen
+     * @param Fqsen[] $parents
      * @param DocBlock $docBlock
      */
-    public function __construct(Fqsen $fqsen, DocBlock $docBlock = null)
+    public function __construct(Fqsen $fqsen, array $parents = array(), DocBlock $docBlock = null)
     {
         $this->fqsen = $fqsen;
         $this->docBlock = $docBlock;
+        $this->parents = $parents;
     }
 
     /**
@@ -119,4 +124,14 @@ public function getDocBlock()
     {
         return $this->docBlock;
     }
+
+    /**
+     * Returns the Fqsen of the interfaces this interface is extending.
+     *
+     * @return Fqsen[]
+     */
+    public function getParents()
+    {
+        return $this->parents;
+    }
 }

tests/component/ProjectCreationTest.php

@@ -165,4 +165,18 @@ public function testWithTrait()
 
         $this->assertArrayHasKey('\\Luigi\\ExampleNestedTrait', $project->getFiles()[$fileName]->getTraits());
     }
+
+    public function testInterfaceExtends()
+    {
+        $fileName = __DIR__ . '/project/Luigi/Packing.php';
+        $project = $this->fixture->create([
+            $fileName
+        ]);
+
+        $this->assertArrayHasKey('\\Luigi\\Packing', $project->getFiles()[$fileName]->getInterfaces());
+        $interface = current($project->getFiles()[$fileName]->getInterfaces());
+
+        $this->assertEquals(['\\Packing' => new Fqsen('\\Packing')], $interface->getParents());
+
+    }
 }

tests/component/project/Luigi/Packing.php

@@ -0,0 +1,18 @@
+<?php
+/**
+ * This file is part of phpDocumentor.
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ *
+ * @copyright 2015-2015 Mike van Riel<[email protected]>
+ * @license   http://www.opensource.org/licenses/mit-license.php MIT
+ * @link      http://phpdoc.org
+ */
+
+namespace Luigi;
+
+interface Packing extends \Packing
+{
+
+}

tests/component/project/Packing.php

@@ -0,0 +1,16 @@
+<?php
+/**
+ * This file is part of phpDocumentor.
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ *
+ * @copyright 2015-2015 Mike van Riel<[email protected]>
+ * @license   http://www.opensource.org/licenses/mit-license.php MIT
+ * @link      http://phpdoc.org
+ */
+
+interface Packing
+{
+    public function getName();
+}

tests/unit/phpDocumentor/Reflection/Php/Factory/Interface_Test.php

@@ -156,6 +156,7 @@ private function buildClassMock()
     {
         $interfaceMock = m::mock(InterfaceNode::class);
         $interfaceMock->fqsen = new Fqsen('\Space\MyInterface');
+        $interfaceMock->extends = [];
         return $interfaceMock;
     }
 }

tests/unit/phpDocumentor/Reflection/Php/FileTest.php

@@ -99,7 +99,7 @@ public function testAddAndGetInterfaces()
     {
         $this->assertEmpty($this->fixture->getInterfaces());
 
-        $interface = new Interface_(new Fqsen('\MySpace\MyInterface'));
+        $interface = new Interface_(new Fqsen('\MySpace\MyInterface'), array());
         $this->fixture->addInterface($interface);
 
         $this->assertEquals(array('\MySpace\MyInterface' => $interface), $this->fixture->getInterfaces());

tests/unit/phpDocumentor/Reflection/Php/Interface_Test.php

@@ -13,11 +13,8 @@
 namespace phpDocumentor\Reflection\Php;
 
 use Mockery as m;
-use phpDocumentor\Reflection\Php\Tag\AuthorDescriptor;
-use phpDocumentor\Reflection\Php\Tag\VersionDescriptor;
 use phpDocumentor\Reflection\DocBlock;
 use phpDocumentor\Reflection\Fqsen;
-use PhpParser\Comment\Doc;
 
 /**
  * Tests the functionality for the Interface_ class.
@@ -45,7 +42,7 @@ protected function setUp()
     {
         $this->fqsen = new Fqsen('\MySpace\MyInterface');
         $this->docBlock = new DocBlock('');
-        $this->fixture = new Interface_($this->fqsen, $this->docBlock);
+        $this->fixture = new Interface_($this->fqsen, array(), $this->docBlock);
     }
 
     /**