| @@ -157,7 +157,7 @@ | ||
| 157 | 157 | * Require the bootstrap. If the file is specified, but does not exist | 
| 158 | 158 | * then an exception will be raised. | 
| 159 | 159 | * | 
| 160 | - * @param $file | |
| 160 | + * @param string $file | |
| 161 | 161 | * @throws \RuntimeException | 
| 162 | 162 | */ | 
| 163 | 163 | public function requireBootstrap($file) | 
| @@ -102,7 +102,7 @@ discard block | ||
| 102 | 102 | * Append a testsuite node to the given | 
| 103 | 103 | * root element | 
| 104 | 104 | * | 
| 105 | - * @param $root | |
| 105 | + * @param \DOMElement $root | |
| 106 | 106 | * @param TestSuite $suite | 
| 107 | 107 | * @return \DOMElement | 
| 108 | 108 | */ | 
| @@ -123,7 +123,7 @@ discard block | ||
| 123 | 123 | * Append a testcase node to the given testsuite | 
| 124 | 124 | * node | 
| 125 | 125 | * | 
| 126 | - * @param $suiteNode | |
| 126 | + * @param \DOMElement $suiteNode | |
| 127 | 127 | * @param TestCase $case | 
| 128 | 128 | * @return \DOMElement | 
| 129 | 129 | */ | 
| @@ -145,9 +145,9 @@ discard block | ||
| 145 | 145 | /** | 
| 146 | 146 | * Append error or failure nodes to the given testcase node | 
| 147 | 147 | * | 
| 148 | - * @param $caseNode | |
| 148 | + * @param \DOMElement $caseNode | |
| 149 | 149 | * @param $defects | 
| 150 | - * @param $type | |
| 150 | + * @param string $type | |
| 151 | 151 | */ | 
| 152 | 152 | protected function appendDefects($caseNode, $defects, $type) | 
| 153 | 153 |      { | 
| @@ -155,7 +155,7 @@ discard block | ||
| 155 | 155 | * Flatten messages into a single collection | 
| 156 | 156 | * based on an accessor method | 
| 157 | 157 | * | 
| 158 | - * @param $method | |
| 158 | + * @param string $method | |
| 159 | 159 | * @return array | 
| 160 | 160 | */ | 
| 161 | 161 | private function mergeMessages($method) | 
| @@ -171,7 +171,7 @@ discard block | ||
| 171 | 171 | * Reduces a collection of readers down to a single | 
| 172 | 172 | * result based on an accessor | 
| 173 | 173 | * | 
| 174 | - * @param $method | |
| 174 | + * @param string $method | |
| 175 | 175 | * @return mixed | 
| 176 | 176 | */ | 
| 177 | 177 | private function accumulate($method) | 
| @@ -43,7 +43,7 @@ discard block | ||
| 43 | 43 | /** | 
| 44 | 44 | * Return a value as a float or integer | 
| 45 | 45 | * | 
| 46 | - * @param $property | |
| 46 | + * @param string $property | |
| 47 | 47 | * @return float|int | 
| 48 | 48 | */ | 
| 49 | 49 | protected function getNumericValue($property) | 
| @@ -56,7 +56,7 @@ discard block | ||
| 56 | 56 | /** | 
| 57 | 57 | * Return messages for a given type | 
| 58 | 58 | * | 
| 59 | - * @param $type | |
| 59 | + * @param string $type | |
| 60 | 60 | * @return array | 
| 61 | 61 | */ | 
| 62 | 62 | protected function getMessages($type) | 
| @@ -260,7 +260,7 @@ | ||
| 260 | 260 | } | 
| 261 | 261 | |
| 262 | 262 | /** | 
| 263 | - * @param $file | |
| 263 | + * @param string $file | |
| 264 | 264 | * @return bool | 
| 265 | 265 | */ | 
| 266 | 266 | private function isFile($file) | 
| @@ -345,6 +345,7 @@ discard block | ||
| 345 | 345 | * @todo Skipped/Incomplete test tracking available only in functional mode for now | 
| 346 | 346 | * or in regular mode but without group/exclude-group filters. | 
| 347 | 347 | * | 
| 348 | + * @param Options $options | |
| 348 | 349 | * @return boolean | 
| 349 | 350 | */ | 
| 350 | 351 | protected function isSkippedIncompleTestCanBeTracked($options) | 
| @@ -420,7 +421,7 @@ discard block | ||
| 420 | 421 | * for a collection of errors or failures | 
| 421 | 422 | * | 
| 422 | 423 | * @param array $defects | 
| 423 | - * @param $type | |
| 424 | + * @param string $type | |
| 424 | 425 | * @return string | 
| 425 | 426 | */ | 
| 426 | 427 | protected function getDefects(array $defects, $type) | 
| @@ -512,6 +513,9 @@ discard block | ||
| 512 | 513 | } | 
| 513 | 514 | } | 
| 514 | 515 | |
| 516 | + /** | |
| 517 | + * @param string $text | |
| 518 | + */ | |
| 515 | 519 | private function green($text) | 
| 516 | 520 |      { | 
| 517 | 521 |          if ($this->colors) { | 
| @@ -522,6 +526,9 @@ discard block | ||
| 522 | 526 | return $text; | 
| 523 | 527 | } | 
| 524 | 528 | |
| 529 | + /** | |
| 530 | + * @param string $text | |
| 531 | + */ | |
| 525 | 532 | private function red($text) | 
| 526 | 533 |      { | 
| 527 | 534 |          if ($this->colors) { | 
| @@ -86,8 +86,8 @@ | ||
| 86 | 86 | * Loads suites based on a specific path. | 
| 87 | 87 | * A valid path can be a directory or file | 
| 88 | 88 | * | 
| 89 | - * @param $path | |
| 90 | - * @param $pattern | |
| 89 | + * @param string $path | |
| 90 | + * @param string $pattern | |
| 91 | 91 | * @throws \InvalidArgumentException | 
| 92 | 92 | * @return string[] | 
| 93 | 93 | */ | 
| @@ -137,6 +137,7 @@ | ||
| 137 | 137 | /** | 
| 138 | 138 | * Returns the output streams of a subset of workers. | 
| 139 | 139 | * @param array keys are positions in $this->workers | 
| 140 | + * @param Worker[] $workers | |
| 140 | 141 | * @return array | 
| 141 | 142 | */ | 
| 142 | 143 | private function streamsOf($workers) | 
| @@ -146,8 +146,8 @@ | ||
| 146 | 146 | } | 
| 147 | 147 | |
| 148 | 148 | /** | 
| 149 | - * @param $filename | |
| 150 | - * @param $className | |
| 149 | + * @param string $filename | |
| 150 | + * @param string $className | |
| 151 | 151 | * @return bool | 
| 152 | 152 | */ | 
| 153 | 153 | private function classNameMatchesFileName($filename, $className) |