Completed
Pull Request — master (#256)
by Filippo
03:10
created
src/Console/Testers/PHPUnit.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -157,7 +157,7 @@
 block discarded – undo
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)
Please login to merge, or discard this patch.
src/Logging/JUnit/Reader.php 1 patch
Doc Comments   +3 added lines patch added patch discarded remove patch
@@ -37,6 +37,9 @@
 block discarded – undo
37 37
                                       'skipped' => 0,
38 38
                                       'time' => 0];
39 39
 
40
+    /**
41
+     * @param string $logFile
42
+     */
40 43
     public function __construct($logFile)
41 44
     {
42 45
         if (!file_exists($logFile)) {
Please login to merge, or discard this patch.
src/Logging/JUnit/Writer.php 1 patch
Doc Comments   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
     {
Please login to merge, or discard this patch.
src/Logging/LogInterpreter.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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)
Please login to merge, or discard this patch.
src/Logging/MetaProvider.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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)
Please login to merge, or discard this patch.
src/Parser/Parser.php 1 patch
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -146,8 +146,8 @@
 block discarded – undo
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)
Please login to merge, or discard this patch.
src/Runners/PHPUnit/ResultPrinter.php 1 patch
Doc Comments   +8 added lines, -1 removed lines patch added patch discarded remove patch
@@ -345,6 +345,7 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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) {
Please login to merge, or discard this patch.
src/Runners/PHPUnit/WrapperRunner.php 1 patch
Doc Comments   +1 added lines patch added patch discarded remove patch
@@ -137,6 +137,7 @@
 block discarded – undo
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)
Please login to merge, or discard this patch.
src/Runners/PHPUnit/TestFileLoader.php 1 patch
Doc Comments   +5 added lines, -2 removed lines patch added patch discarded remove patch
@@ -46,6 +46,9 @@  discard block
 block discarded – undo
46 46
     protected $excludingFiles = false;
47 47
 
48 48
 
49
+    /**
50
+     * @param Options $options
51
+     */
49 52
     public function __construct($options = null)
50 53
     {
51 54
         if ($options && !$options instanceof Options) {
@@ -86,8 +89,8 @@  discard block
 block discarded – undo
86 89
      * Loads suites based on a specific path.
87 90
      * A valid path can be a directory or file
88 91
      *
89
-     * @param $path
90
-     * @param $pattern
92
+     * @param string $path
93
+     * @param string $pattern
91 94
      * @throws \InvalidArgumentException
92 95
      * @return string[]
93 96
      */
Please login to merge, or discard this patch.