@@ -79,6 +79,11 @@ discard block |
||
79 | 79 | $this->navigator->addChild($allPluginsPresenter); |
80 | 80 | } |
81 | 81 | |
82 | + /** |
|
83 | + * @param DirectoryIterator $file |
|
84 | + * @param TestsPluginSuitePresenter $allPluginsPresenter |
|
85 | + * @param TestsPluginOrderedSuite $allPluginsSuite |
|
86 | + */ |
|
82 | 87 | private function addPluginSuite($file, $allPluginsPresenter, $allPluginsSuite) { |
83 | 88 | $pluginName = basename($file->getPathname()); |
84 | 89 | $testsPath = $file->getPathname() . '/tests'; |
@@ -93,6 +98,11 @@ discard block |
||
93 | 98 | } |
94 | 99 | } |
95 | 100 | |
101 | + /** |
|
102 | + * @param TestsPluginOrderedSuite $parentSuite |
|
103 | + * @param TestsPluginSuitePresenter $presenter |
|
104 | + * @param string $path |
|
105 | + */ |
|
96 | 106 | public function addSuite($parentSuite, $presenter, $name, $path) { |
97 | 107 | |
98 | 108 | foreach ($this->getTestsIterator($path) as $file) { |
@@ -125,6 +135,9 @@ discard block |
||
125 | 135 | return new TestsPluginOrderedSuite($title); |
126 | 136 | } |
127 | 137 | |
138 | + /** |
|
139 | + * @param DirectoryIterator $test |
|
140 | + */ |
|
128 | 141 | public function isSuite($test, $append = '') { |
129 | 142 | $pathName = $test->getPathname(); |
130 | 143 | $baseName = baseName($pathName); |
@@ -133,12 +146,19 @@ discard block |
||
133 | 146 | $this->isNotATestsRestDirectory($pathName); |
134 | 147 | } |
135 | 148 | |
149 | + /** |
|
150 | + * @param DirectoryIterator $test |
|
151 | + */ |
|
136 | 152 | public function isTest($test) { |
137 | 153 | $baseName = basename($test->getPathname()); |
138 | 154 | return !in_array($baseName[0], array('_', '.')) && |
139 | 155 | (substr($baseName, -8) === 'Test.php' || (version_compare(phpversion(), '5.3', '>=') && substr($baseName, -13) === 'TestPHP53.php')); |
140 | 156 | } |
141 | 157 | |
158 | + /** |
|
159 | + * @param string $name |
|
160 | + * @param string $value |
|
161 | + */ |
|
142 | 162 | public function getPresenter($prefix, $name, $value) { |
143 | 163 | return new TestsPluginSuitePresenter($prefix, $name, $value, $this->isSelected($value)); |
144 | 164 | } |
@@ -169,6 +189,9 @@ discard block |
||
169 | 189 | return $this->navigator; |
170 | 190 | } |
171 | 191 | |
192 | + /** |
|
193 | + * @param string $order |
|
194 | + */ |
|
172 | 195 | public function getTitleByOrder($order) { |
173 | 196 | return $this->titles[$order]; |
174 | 197 | } |
@@ -181,6 +204,10 @@ discard block |
||
181 | 204 | } |
182 | 205 | } |
183 | 206 | |
207 | + /** |
|
208 | + * @param TestsPluginSuitePresenter $navigator |
|
209 | + * @param string $results |
|
210 | + */ |
|
184 | 211 | protected function render($navigator, $results) { |
185 | 212 | $presenter = new TestsPluginRunnerPresenter($this->request, $navigator, $results); |
186 | 213 | $template = 'testsPluginRunner' . strtoupper($this->request->getDisplay()); |
@@ -125,6 +125,9 @@ |
||
125 | 125 | } |
126 | 126 | } |
127 | 127 | |
128 | + /** |
|
129 | + * @param TestsPluginFilterIterator $iterator |
|
130 | + */ |
|
128 | 131 | public static function cleanIteratorToArray($iterator) { |
129 | 132 | $array = array(); |
130 | 133 | foreach($iterator as $testFile) { |
@@ -28,6 +28,11 @@ |
||
28 | 28 | $this->assertTrue($this->assertUriHasArgument('/foo?bar=baz&truc=muche', 'bar', 'baz')); |
29 | 29 | } |
30 | 30 | |
31 | + /** |
|
32 | + * @param string $uri |
|
33 | + * @param string $param |
|
34 | + * @param string $value |
|
35 | + */ |
|
31 | 36 | public function assertUriHasArgument($uri, $param, $value) { |
32 | 37 | //disable the current reporter when we test assertions |
33 | 38 | $reporter = $this->_reporter; |
@@ -18,6 +18,9 @@ discard block |
||
18 | 18 | class CodeCoverageInvokerDecorator extends SimpleInvokerDecorator { |
19 | 19 | protected $coverage; |
20 | 20 | |
21 | + /** |
|
22 | + * @param SimpleInvoker $invoker |
|
23 | + */ |
|
21 | 24 | function __construct($coverage, $invoker) { |
22 | 25 | $this->SimpleInvokerDecorator($invoker); |
23 | 26 | $this->coverage = $coverage; |
@@ -57,6 +60,9 @@ discard block |
||
57 | 60 | */ |
58 | 61 | public function createInvoker($invoker); |
59 | 62 | |
63 | + /** |
|
64 | + * @return boolean |
|
65 | + */ |
|
60 | 66 | public function generateCoverage($path); |
61 | 67 | } |
62 | 68 | |
@@ -65,6 +71,9 @@ discard block |
||
65 | 71 | protected $_timer; |
66 | 72 | protected $coverage; |
67 | 73 | |
74 | + /** |
|
75 | + * @param PHP_CodeCoverage|null $coverage |
|
76 | + */ |
|
68 | 77 | function __construct($coverage) { |
69 | 78 | parent::__construct(); |
70 | 79 | $this->coverage = $coverage; |
@@ -177,11 +186,17 @@ discard block |
||
177 | 186 | class CodendiJUnitXMLReporter extends JUnitXMLReporter implements iCodeCoverageReporter { |
178 | 187 | protected $coverage; |
179 | 188 | |
189 | + /** |
|
190 | + * @param PHP_CodeCoverage|null $coverage |
|
191 | + */ |
|
180 | 192 | function __construct($coverage) { |
181 | 193 | $this->JUnitXMLReporter(); |
182 | 194 | $this->coverage = $coverage; |
183 | 195 | } |
184 | 196 | |
197 | + /** |
|
198 | + * @return string |
|
199 | + */ |
|
185 | 200 | public function getXML() { |
186 | 201 | return $this->doc->saveXML(); |
187 | 202 | } |
@@ -278,9 +293,6 @@ discard block |
||
278 | 293 | |
279 | 294 | /** |
280 | 295 | * Paints a PHP error or exception. |
281 | - * @param string $message Message to be shown. |
|
282 | - * @access public |
|
283 | - * @abstract |
|
284 | 296 | */ |
285 | 297 | function paintException($exception) { |
286 | 298 | parent::paintException($exception); |
@@ -25,6 +25,9 @@ |
||
25 | 25 | function clean_plugins_root($entry) { |
26 | 26 | return substr($entry, strlen($GLOBALS['config']['plugins_root']), -strlen($GLOBALS['config']['tests_root'])); |
27 | 27 | } |
28 | +/** |
|
29 | + * @param string $dir |
|
30 | + */ |
|
28 | 31 | function search_tests_rec($dir, &$tab, $entry) { |
29 | 32 | if (is_dir($dir)) { |
30 | 33 | if ($dh = opendir($dir)) { |
@@ -45,6 +45,10 @@ |
||
45 | 45 | $this->executeQuery($sql, $msg); |
46 | 46 | } |
47 | 47 | |
48 | + /** |
|
49 | + * @param string $sql |
|
50 | + * @param string $msg |
|
51 | + */ |
|
48 | 52 | private function executeQuery($sql, $msg) { |
49 | 53 | $res = $this->db->dbh->exec($sql); |
50 | 54 | if ($res === false) { |
@@ -123,12 +123,19 @@ |
||
123 | 123 | } |
124 | 124 | } |
125 | 125 | |
126 | + /** |
|
127 | + * @param string $table |
|
128 | + */ |
|
126 | 129 | private function isTableInnoDB($table) { |
127 | 130 | $sql = "SHOW TABLE STATUS WHERE Name = '$table' AND Engine = 'InnoDB'"; |
128 | 131 | $result = $this->db->dbh->query($sql); |
129 | 132 | return ($result->fetch() !== false); |
130 | 133 | } |
131 | 134 | |
135 | + /** |
|
136 | + * @param string $tableName |
|
137 | + * @param string $index |
|
138 | + */ |
|
132 | 139 | private function indexNameExists($tableName, $index) { |
133 | 140 | $sql = 'SHOW INDEX FROM '.$tableName.' WHERE Key_name LIKE '.$this->db->dbh->quote($index); |
134 | 141 | $res = $this->db->dbh->query($sql); |
@@ -40,6 +40,10 @@ |
||
40 | 40 | $this->createTable('tracker_rule_date', $sql); |
41 | 41 | } |
42 | 42 | |
43 | + /** |
|
44 | + * @param string $name |
|
45 | + * @param string $sql |
|
46 | + */ |
|
43 | 47 | private function createTable($name, $sql) { |
44 | 48 | $result = $this->db->createTable($name, $sql); |
45 | 49 |
@@ -75,6 +75,10 @@ |
||
75 | 75 | return $local_inc; |
76 | 76 | } |
77 | 77 | |
78 | + /** |
|
79 | + * @param string $sql |
|
80 | + * @param string $error_message |
|
81 | + */ |
|
78 | 82 | private function exec($sql, $error_message) { |
79 | 83 | $res = $this->db->dbh->exec($sql); |
80 | 84 | if ($res === false) { |