brianium / paratest

src/Console/Testers/PHPUnit.php

Doc Comments +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
      * Require the bootstrap. If the file is specified, but does not exist
      * then an exception will be raised.
      *
-     * @param $file
+     * @param string $file
      * @throws \RuntimeException
      */
     public function requireBootstrap($file)

src/Logging/JUnit/Reader.php

Doc Comments +3 added lines

@@ -37,6 +37,9 @@
                                       'skipped' => 0,
                                       'time' => 0];
 
+    /**
+     * @param string $logFile
+     */
     public function __construct($logFile)
     {
         if (!file_exists($logFile)) {

src/Logging/JUnit/Writer.php

Doc Comments +4 added lines, -4 removed lines

@@ -102,7 +102,7 @@ 
      * Append a testsuite node to the given
      * root element
      *
-     * @param $root
+     * @param \DOMElement $root
      * @param TestSuite $suite
      * @return \DOMElement
      */
@@ -123,7 +123,7 @@ 
      * Append a testcase node to the given testsuite
      * node
      *
-     * @param $suiteNode
+     * @param \DOMElement $suiteNode
      * @param TestCase $case
      * @return \DOMElement
      */
@@ -145,9 +145,9 @@ 
     /**
      * Append error or failure nodes to the given testcase node
      *
-     * @param $caseNode
+     * @param \DOMElement $caseNode
      * @param $defects
-     * @param $type
+     * @param string $type
      */
     protected function appendDefects($caseNode, $defects, $type)
     {

src/Logging/LogInterpreter.php

Doc Comments +2 added lines, -2 removed lines

@@ -155,7 +155,7 @@ 
      * Flatten messages into a single collection
      * based on an accessor method
      *
-     * @param $method
+     * @param string $method
      * @return array
      */
     private function mergeMessages($method)
@@ -171,7 +171,7 @@ 
      * Reduces a collection of readers down to a single
      * result based on an accessor
      *
-     * @param $method
+     * @param string $method
      * @return mixed
      */
     private function accumulate($method)

src/Logging/MetaProvider.php

Doc Comments +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
     /**
      * Return a value as a float or integer
      *
-     * @param $property
+     * @param string $property
      * @return float|int
      */
     protected function getNumericValue($property)
@@ -56,7 +56,7 @@ 
     /**
      * Return messages for a given type
      *
-     * @param $type
+     * @param string $type
      * @return array
      */
     protected function getMessages($type)

src/Parser/Parser.php

Doc Comments +2 added lines, -2 removed lines

@@ -146,8 +146,8 @@
     }
 
     /**
-     * @param $filename
-     * @param $className
+     * @param string $filename
+     * @param string $className
      * @return bool
      */
     private function classNameMatchesFileName($filename, $className)

src/Runners/PHPUnit/ResultPrinter.php

Doc Comments +8 added lines, -1 removed lines

@@ -345,6 +345,7 @@ 
      * @todo Skipped/Incomplete test tracking available only in functional mode for now
      *       or in regular mode but without group/exclude-group filters.
      *
+     * @param Options $options
      * @return boolean
      */
     protected function isSkippedIncompleTestCanBeTracked($options)
@@ -420,7 +421,7 @@ 
      * for a collection of errors or failures
      *
      * @param array $defects
-     * @param $type
+     * @param string $type
      * @return string
      */
     protected function getDefects(array $defects, $type)
@@ -512,6 +513,9 @@ 
         }
     }
 
+    /**
+     * @param string $text
+     */
     private function green($text)
     {
         if ($this->colors) {
@@ -522,6 +526,9 @@ 
         return $text;
     }
 
+    /**
+     * @param string $text
+     */
     private function red($text)
     {
         if ($this->colors) {

src/Runners/PHPUnit/WrapperRunner.php

Doc Comments +1 added lines

@@ -137,6 +137,7 @@
     /**
      * Returns the output streams of a subset of workers.
      * @param array    keys are positions in $this->workers
+     * @param Worker[] $workers
      * @return array
      */
     private function streamsOf($workers)

src/Runners/PHPUnit/SuiteLoader.php

Doc Comments +23 added lines

@@ -27,6 +27,9 @@ 
      */
     public $options;
 
+    /**
+     * @param Options $options
+     */
     public function __construct($options = null)
     {
         if ($options && !$options instanceof Options) {
@@ -126,6 +129,9 @@ 
         }
     }
 
+    /**
+     * @param ParsedClass $class
+     */
     private function executableTests($path, $class)
     {
         $executableTests = [];
@@ -167,6 +173,10 @@ 
         return $batches;
     }
 
+    /**
+     * @param string $dependsOn
+     * @param string[] $tests
+     */
     private function addDependentTestsToBatchSet(&$batches, $dependsOn, $tests)
     {
         foreach ($batches as $key => $batch) {
@@ -179,6 +189,9 @@ 
         }
     }
 
+    /**
+     * @param string[] $tests
+     */
     private function addTestsToBatchSet(&$batches, $tests, $maxBatchSize)
     {
         foreach ($tests as $test) {
@@ -269,6 +282,10 @@ 
         return $result;
     }
 
+    /**
+     * @param string $className
+     * @param string $name
+     */
     private function testMatchOptions($className, $name, $group)
     {
         $result = $this->testMatchGroupOptions($group)
@@ -277,6 +294,9 @@ 
         return $result;
     }
 
+    /**
+     * @param ParsedObject $method
+     */
     private function methodDataProvider($method)
     {
         if (preg_match("/@\bdataProvider\b \b(.*)\b/", $method->getDocBlock(), $matches)) {
@@ -293,6 +313,9 @@ 
         return null;
     }
 
+    /**
+     * @param ParsedObject $method
+     */
     private function methodGroups($method)
     {
         if (preg_match_all("/@\bgroup\b \b(.*)\b/", $method->getDocBlock(), $matches)) {