phpDocumentor / phpDocumentor

tests/features/bootstrap/Ast/ApiContext.php

Doc Comments +3 added lines

@@ -189,6 +189,7 @@ 
      * @param string $classFqsen
      * @param string $tagName
      * @param string $methodName
+     * @param integer $expectedCount
      * @Then class ":classFqsen" has a method named :method with exactly :expected tag :tagName
      */
     public function classHasMethodWithExpectedCountTag($classFqsen, $tagName, $methodName, $expectedCount)
@@ -439,6 +440,8 @@ 
 
     /**
      * @throws Exception
+     * @param string $classFqsen
+     * @param string $methodName
      */
     private function findMagicMethodResponse($classFqsen, $methodName): ReturnDescriptor
     {

Unused Use Statements -2 removed lines

@@ -13,7 +13,6 @@ 
 namespace phpDocumentor\Behat\Contexts\Ast;
 
 use Behat\Behat\Context\Context;
-use Behat\Behat\Tester\Exception\PendingException;
 use Behat\Gherkin\Node\PyStringNode;
 use phpDocumentor\Descriptor\ArgumentDescriptor;
 use phpDocumentor\Descriptor\ClassDescriptor;
@@ -28,7 +27,6 @@ 
 use phpDocumentor\Descriptor\Tag\ReturnDescriptor;
 use phpDocumentor\Descriptor\Tag\VersionDescriptor;
 use phpDocumentor\Descriptor\TraitDescriptor;
-use phpDocumentor\Reflection\Php\File;
 use Webmozart\Assert\Assert;
 
 class ApiContext extends BaseContext implements Context

tests/ReferenceImplementation.php

Doc Comments +7 added lines

@@ -163,6 +163,9 @@ 
     {
         const INTERFACE_CONSTANT = 'test';
 
+        /**
+         * @return SubInterface|null
+         */
         public function publicMethod();
     }
 
@@ -172,6 +175,10 @@ 
      */
     interface AnotherSuperInterface
     {
+
+        /**
+         * @return void
+         */
         public static function staticMethod();
     }
 }

tests/unit/phpDocumentor/Descriptor/PropertyDescriptorTest.php

Doc Comments +2 added lines, -2 removed lines

@@ -391,7 +391,7 @@ 
     /**
      * Sets up mocks as such that the fixture has a file.
      *
-     * @return m\MockInterface|FileDescriptor
+     * @return m\LegacyMockInterface
      */
     protected function whenFixtureIsDirectlyRelatedToAFile()
     {
@@ -403,7 +403,7 @@ 
     /**
      * Sets up mocks as such that the fixture has a parent class, with a file.
      *
-     * @return m\MockInterface|FileDescriptor
+     * @return m\LegacyMockInterface
      */
     protected function whenFixtureIsRelatedToAClassWithFile()
     {

tests/unit/phpDocumentor/Descriptor/Builder/Reflector/Tags/UsesAssemblerTest.php

Doc Comments +4 added lines

@@ -63,6 +63,10 @@
         $this->assertSame([], $descriptor->getErrors()->getAll());
     }
 
+    /**
+     * @param string $description
+     * @param string $reference
+     */
     protected function givenAUsesTag($description, $reference) : Uses
     {
         return new Uses(

tests/features/bootstrap/EnvironmentContext.php

Unused Use Statements -1 removed lines

@@ -13,7 +13,6 @@
 namespace phpDocumentor\Behat\Contexts;
 
 use Behat\Behat\Context;
-use Behat\Behat\Tester\Exception\PendingException;
 use FilesystemIterator;
 use RecursiveDirectoryIterator;
 use RecursiveIteratorIterator;

src/phpDocumentor/Descriptor/Builder/Reflector/FileAssembler.php

Doc Comments +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     /**
      * Creates a Descriptor from the provided data.
      *
-     * @param mixed $data
+     * @param File $data
      */
     public function create($data) : FileDescriptor
     {

src/phpDocumentor/Descriptor/Interfaces/ArgumentInterface.php

Doc Comments +4 added lines

@@ -36,6 +36,7 @@ 
      * @param ?Type $type Type of this agument represented as a reflection type.
      *
      * @todo update link to point to the final destination for the PHPDoc Standard.
+     * @return void
      */
     public function setType(?Type $type) : void;
 
@@ -43,6 +44,7 @@ 
      * Returns a normalized Types.
      *
      * @see self::setTypes() for details on what types represent.
+     * @return Type|null
      */
     public function getType() : ?Type;
 
@@ -50,6 +52,7 @@ 
      * Sets the default value for an argument expressed as a string.
      *
      * @param string $value A textual representation of the default value.
+     * @return void
      */
     public function setDefault(string $value) : void;
 
@@ -64,6 +67,7 @@ 
      * Sets whether this argument passes its parameter by reference or by value.
      *
      * @param bool $byReference True if the parameter is passed by reference, otherwise it is by value.
+     * @return void
      */
     public function setByReference(bool $byReference) : void;
 

src/phpDocumentor/Descriptor/Interfaces/ChildInterface.php

Doc Comments +1 added lines

@@ -32,6 +32,7 @@
      * Sets the parent for this Descriptor.
      *
      * @param ?DescriptorAbstract $parent
+     * @return void
      */
     public function setParent(?DescriptorAbstract $parent) : void;
 }

tests/unit/phpDocumentor/Descriptor/Builder/Reflector/MethodAssemblerTest.php

Doc Comments +2 added lines

@@ -169,6 +169,7 @@ 
 
     /**
      * Creates a sample method reflector for the tests with the given data.
+     * @param DocBlock $docBlockMock
      */
     protected function givenAMethodReflector(
         string $namespace,
@@ -189,6 +190,7 @@ 
 
     /**
      * Generates a DocBlock object with applicable defaults for these tests.
+     * @param boolean $withTags
      */
     protected function givenADocBlockObject($withTags) : DocBlock
     {