brianium / paratest

src/ParaTest/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/ParaTest/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/ParaTest/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/ParaTest/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/ParaTest/Runners/PHPUnit/Options.php

Doc Comments +1 added lines, -1 removed lines

@@ -260,7 +260,7 @@
     }
 
     /**
-     * @param $file
+     * @param string $file
      * @return bool
      */
     private function isFile($file)

src/ParaTest/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/ParaTest/Runners/PHPUnit/TestFileLoader.php

Doc Comments +2 added lines, -2 removed lines

@@ -86,8 +86,8 @@
      * Loads suites based on a specific path.
      * A valid path can be a directory or file
      *
-     * @param $path
-     * @param $pattern
+     * @param string $path
+     * @param string $pattern
      * @throws \InvalidArgumentException
      * @return string[]
      */

src/ParaTest/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/ParaTest/Runners/PHPUnit/SuiteLoader.php

Doc Comments +23 added lines

@@ -22,6 +22,9 @@ 
      */
     protected $loadedSuites = array();
 
+    /**
+     * @param Options $options
+     */
     public function __construct($options = null)
     {
         if ($options && !$options instanceof Options) {
@@ -121,6 +124,9 @@ 
         }
     }
 
+    /**
+     * @param ParsedClass $class
+     */
     private function executableTests($path, $class)
     {
         $executableTests = array();
@@ -162,6 +168,10 @@ 
         return $batches;
     }
 
+    /**
+     * @param string $dependsOn
+     * @param string[] $tests
+     */
     private function addDependentTestsToBatchSet(&$batches, $dependsOn, $tests)
     {
         foreach ($batches as $key => $batch) {
@@ -174,6 +184,9 @@ 
         }
     }
 
+    /**
+     * @param string[] $tests
+     */
     private function addTestsToBatchSet(&$batches, $tests, $maxBatchSize)
     {
         foreach ($tests as $test) {
@@ -264,6 +277,10 @@ 
         return $result;
     }
 
+    /**
+     * @param string $className
+     * @param string $name
+     */
     private function testMatchOptions($className, $name, $group)
     {
         $result = $this->testMatchGroupOptions($group)
@@ -272,6 +289,9 @@ 
         return $result;
     }
 
+    /**
+     * @param ParsedObject $method
+     */
     private function methodDataProvider($method)
     {
         if (preg_match("/@\bdataProvider\b \b(.*)\b/", $method->getDocBlock(), $matches)) {
@@ -288,6 +308,9 @@ 
         return null;
     }
 
+    /**
+     * @param ParsedObject $method
+     */
     private function methodGroups($method)
     {
         if (preg_match_all("/@\bgroup\b \b(.*)\b/", $method->getDocBlock(), $matches)) {