@@ -186,7 +186,7 @@ |
||
| 186 | 186 | } |
| 187 | 187 | |
| 188 | 188 | /** |
| 189 | - * @param mixed $subject |
|
| 189 | + * @param \PhpCollection\Sequence $subject |
|
| 190 | 190 | */ |
| 191 | 191 | private function assertInterfaces($subject) |
| 192 | 192 | { |
@@ -8,7 +8,6 @@ |
||
| 8 | 8 | |
| 9 | 9 | namespace GitElephant\Command; |
| 10 | 10 | |
| 11 | -use GitElephant\Objects\Commit; |
|
| 12 | 11 | use \GitElephant\TestCase; |
| 13 | 12 | |
| 14 | 13 | class ResetCommandTest extends TestCase |
@@ -351,7 +351,7 @@ discard block |
||
| 351 | 351 | * |
| 352 | 352 | * @param bool $short short version |
| 353 | 353 | * |
| 354 | - * @return mixed |
|
| 354 | + * @return string |
|
| 355 | 355 | */ |
| 356 | 356 | public function getSha($short = false) |
| 357 | 357 | { |
@@ -361,7 +361,7 @@ discard block |
||
| 361 | 361 | /** |
| 362 | 362 | * tree getter |
| 363 | 363 | * |
| 364 | - * @return mixed |
|
| 364 | + * @return string |
|
| 365 | 365 | */ |
| 366 | 366 | public function getTree() |
| 367 | 367 | { |
@@ -371,7 +371,7 @@ discard block |
||
| 371 | 371 | /** |
| 372 | 372 | * datetimeAuthor getter |
| 373 | 373 | * |
| 374 | - * @return mixed |
|
| 374 | + * @return \DateTime |
|
| 375 | 375 | */ |
| 376 | 376 | public function getDatetimeAuthor() |
| 377 | 377 | { |
@@ -90,7 +90,7 @@ |
||
| 90 | 90 | * |
| 91 | 91 | * @param null $commit1 commit 1 |
| 92 | 92 | * @param null $commit2 commit 2 |
| 93 | - * @param null $path path |
|
| 93 | + * @param null|string $path path |
|
| 94 | 94 | * |
| 95 | 95 | * @throws \RuntimeException |
| 96 | 96 | * @throws \Symfony\Component\Process\Exception\InvalidArgumentException |
@@ -54,7 +54,7 @@ |
||
| 54 | 54 | * @param string $refEnd ending reference |
| 55 | 55 | * @param null $path path |
| 56 | 56 | * @param int $limit limit |
| 57 | - * @param null $offset offset |
|
| 57 | + * @param integer|null $offset offset |
|
| 58 | 58 | * @param boolean $firstParent first parent |
| 59 | 59 | * |
| 60 | 60 | * @throws \RuntimeException |
@@ -79,7 +79,7 @@ |
||
| 79 | 79 | * @param \GitElephant\Repository $repository repo |
| 80 | 80 | * @param array $outputLines output lines from command.log |
| 81 | 81 | * |
| 82 | - * @return \GitElephant\Objects\Log |
|
| 82 | + * @return Tree |
|
| 83 | 83 | */ |
| 84 | 84 | public static function createFromOutputLines(Repository $repository, $outputLines) |
| 85 | 85 | { |
@@ -50,7 +50,7 @@ discard block |
||
| 50 | 50 | protected $finder; |
| 51 | 51 | |
| 52 | 52 | /** |
| 53 | - * @param null $name |
|
| 53 | + * @param integer $name |
|
| 54 | 54 | * |
| 55 | 55 | * @return \GitElephant\Repository |
| 56 | 56 | */ |
@@ -121,7 +121,7 @@ discard block |
||
| 121 | 121 | /** |
| 122 | 122 | * @param string $name file name |
| 123 | 123 | * @param string|null $folder folder name |
| 124 | - * @param null $content content |
|
| 124 | + * @param string|null $content content |
|
| 125 | 125 | * @param Repository $repository repository to add file to |
| 126 | 126 | * |
| 127 | 127 | * @return void |
@@ -202,7 +202,7 @@ discard block |
||
| 202 | 202 | } |
| 203 | 203 | |
| 204 | 204 | /** |
| 205 | - * @param $classname |
|
| 205 | + * @param string $classname |
|
| 206 | 206 | * |
| 207 | 207 | * @return \PHPUnit\Framework\MockObject\MockObject |
| 208 | 208 | */ |
@@ -251,6 +251,9 @@ discard block |
||
| 251 | 251 | ->will($this->returnValue($this->getMockCommand())); |
| 252 | 252 | } |
| 253 | 253 | |
| 254 | + /** |
|
| 255 | + * @param string[] $output |
|
| 256 | + */ |
|
| 254 | 257 | protected function addOutputToMockRepo(\PHPUnit\Framework\MockObject\MockObject $repo, $output) |
| 255 | 258 | { |
| 256 | 259 | $repo |
@@ -287,6 +290,17 @@ discard block |
||
| 287 | 290 | return $this->getMock('GitElephant\GitBinary'); |
| 288 | 291 | } |
| 289 | 292 | |
| 293 | + /** |
|
| 294 | + * @param string $sha |
|
| 295 | + * @param string $tree |
|
| 296 | + * @param string $author |
|
| 297 | + * @param string $committer |
|
| 298 | + * @param string $emailAuthor |
|
| 299 | + * @param string $emailCommitter |
|
| 300 | + * @param string $datetimeAuthor |
|
| 301 | + * @param string $datetimeCommitter |
|
| 302 | + * @param string $message |
|
| 303 | + */ |
|
| 290 | 304 | protected function doCommitTest( |
| 291 | 305 | Commit $commit, |
| 292 | 306 | $sha, |
@@ -147,7 +147,7 @@ discard block |
||
| 147 | 147 | * create a repository from a remote git url, or a local filesystem |
| 148 | 148 | * and save it in a temp folder |
| 149 | 149 | * |
| 150 | - * @param string|Repository $git the git remote url, or the filesystem path |
|
| 150 | + * @param string $git the git remote url, or the filesystem path |
|
| 151 | 151 | * @param null $repositoryPath path |
| 152 | 152 | * @param GitBinary $binary binary |
| 153 | 153 | * @param null $name repository name |
@@ -195,7 +195,7 @@ discard block |
||
| 195 | 195 | /** |
| 196 | 196 | * Stage the working tree content |
| 197 | 197 | * |
| 198 | - * @param string|NodeObject $path the path to store |
|
| 198 | + * @param string $path the path to store |
|
| 199 | 199 | * |
| 200 | 200 | * @throws \RuntimeException |
| 201 | 201 | * @throws \Symfony\Component\Process\Exception\LogicException |
@@ -213,7 +213,7 @@ discard block |
||
| 213 | 213 | /** |
| 214 | 214 | * Unstage a tree content |
| 215 | 215 | * |
| 216 | - * @param string|NodeObject $path the path to unstage |
|
| 216 | + * @param string $path the path to unstage |
|
| 217 | 217 | * |
| 218 | 218 | * @throws \RuntimeException |
| 219 | 219 | * @throws \Symfony\Component\Process\Exception\LogicException |
@@ -231,8 +231,8 @@ discard block |
||
| 231 | 231 | /** |
| 232 | 232 | * Move a file/directory |
| 233 | 233 | * |
| 234 | - * @param string|NodeObject $from source path |
|
| 235 | - * @param string|NodeObject $to destination path |
|
| 234 | + * @param string $from source path |
|
| 235 | + * @param string $to destination path |
|
| 236 | 236 | * |
| 237 | 237 | * @throws \RuntimeException |
| 238 | 238 | * @throws \Symfony\Component\Process\Exception\LogicException |
@@ -251,7 +251,7 @@ discard block |
||
| 251 | 251 | /** |
| 252 | 252 | * Remove a file/directory |
| 253 | 253 | * |
| 254 | - * @param string|NodeObject $path the path to remove |
|
| 254 | + * @param string $path the path to remove |
|
| 255 | 255 | * @param bool $recursive recurse |
| 256 | 256 | * @param bool $force force |
| 257 | 257 | * |
@@ -332,7 +332,7 @@ discard block |
||
| 332 | 332 | } |
| 333 | 333 | |
| 334 | 334 | /** |
| 335 | - * @param TreeishInterface|Commit|string $arg |
|
| 335 | + * @param Commit $arg |
|
| 336 | 336 | * @param array $options |
| 337 | 337 | */ |
| 338 | 338 | public function reset($arg, $options) |
@@ -808,7 +808,7 @@ discard block |
||
| 808 | 808 | * |
| 809 | 809 | * @param string|TreeishInterface|array $ref the treeish to check, as a string, as an object or as an array |
| 810 | 810 | * @param string|NodeObject $path the physical path to the tree relative to the repository root |
| 811 | - * @param int|null $limit limit to n entries |
|
| 811 | + * @param integer $limit limit to n entries |
|
| 812 | 812 | * @param int|null $offset skip n entries |
| 813 | 813 | * @param boolean|false $firstParent skip commits brought in to branch by a merge |
| 814 | 814 | * |
@@ -825,7 +825,7 @@ discard block |
||
| 825 | 825 | * @param string $refStart |
| 826 | 826 | * @param string $refEnd |
| 827 | 827 | * @param string|NodeObject $path the physical path to the tree relative to the repository root |
| 828 | - * @param int|null $limit limit to n entries |
|
| 828 | + * @param integer $limit limit to n entries |
|
| 829 | 829 | * @param int|null $offset skip n entries |
| 830 | 830 | * @param boolean|false $firstParent skip commits brought in to branch by a merge |
| 831 | 831 | * |
@@ -850,7 +850,7 @@ discard block |
||
| 850 | 850 | * Get a log for an object |
| 851 | 851 | * |
| 852 | 852 | * @param \GitElephant\Objects\NodeObject $obj The Object instance |
| 853 | - * @param null|string|\GitElephant\Objects\Branch $branch The branch to read from |
|
| 853 | + * @param string|null $branch The branch to read from |
|
| 854 | 854 | * @param int $limit Limit to n entries |
| 855 | 855 | * @param int|null $offset Skip n entries |
| 856 | 856 | * |
@@ -919,7 +919,7 @@ discard block |
||
| 919 | 919 | * Get a Diff object for a commit with its parent, by default the diff is between the current head and its parent |
| 920 | 920 | * |
| 921 | 921 | * @param \GitElephant\Objects\Commit|string $commit1 A TreeishInterface instance |
| 922 | - * @param \GitElephant\Objects\Commit|string|null $commit2 A TreeishInterface instance |
|
| 922 | + * @param Commit $commit2 A TreeishInterface instance |
|
| 923 | 923 | * @param null|string|NodeObject $path The path to get the diff for or a Object instance |
| 924 | 924 | * |
| 925 | 925 | * @throws \RuntimeException |
@@ -935,7 +935,7 @@ discard block |
||
| 935 | 935 | * Clone a repository |
| 936 | 936 | * |
| 937 | 937 | * @param string $url the repository url (i.e. git://github.com/matteosister/GitElephant.git) |
| 938 | - * @param null $to where to clone the repo |
|
| 938 | + * @param string|null $to where to clone the repo |
|
| 939 | 939 | * |
| 940 | 940 | * @throws \RuntimeException |
| 941 | 941 | * @throws \Symfony\Component\Process\Exception\LogicException |
@@ -1072,7 +1072,7 @@ discard block |
||
| 1072 | 1072 | * output a node content as an array of lines |
| 1073 | 1073 | * |
| 1074 | 1074 | * @param \GitElephant\Objects\NodeObject $obj The Object of type BLOB |
| 1075 | - * @param \GitElephant\Objects\TreeishInterface|string $treeish A treeish object |
|
| 1075 | + * @param null|Branch $treeish A treeish object |
|
| 1076 | 1076 | * |
| 1077 | 1077 | * @throws \RuntimeException |
| 1078 | 1078 | * @throws \Symfony\Component\Process\Exception\LogicException |