matteosister / GitElephant

tests/GitElephant/Command/ResetCommandTest.php

Unused Use Statements -1 removed lines

@@ -8,7 +8,6 @@
 
 namespace GitElephant\Command;
 
-use GitElephant\Objects\Commit;
 use \GitElephant\TestCase;
 
 class ResetCommandTest extends TestCase

src/GitElephant/Objects/Commit.php

Doc Comments +3 added lines, -3 removed lines

@@ -351,7 +351,7 @@ 
      *
      * @param bool $short short version
      *
-     * @return mixed
+     * @return string
      */
     public function getSha($short = false)
     {
@@ -361,7 +361,7 @@ 
     /**
      * tree getter
      *
-     * @return mixed
+     * @return string
      */
     public function getTree()
     {
@@ -371,7 +371,7 @@ 
     /**
      * datetimeAuthor getter
      *
-     * @return mixed
+     * @return \DateTime
      */
     public function getDatetimeAuthor()
     {

src/GitElephant/Objects/Tree.php

Doc Comments +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
      * @param \GitElephant\Repository $repository  repo
      * @param array                   $outputLines output lines from command.log
      *
-     * @return \GitElephant\Objects\Log
+     * @return Tree
      */
     public static function createFromOutputLines(Repository $repository, $outputLines)
     {

src/GitElephant/Repository.php

Doc Comments +13 added lines, -13 removed lines

@@ -144,7 +144,7 @@ 
      * create a repository from a remote git url, or a local filesystem
      * and save it in a temp folder
      *
-     * @param string|Repository $git            the git remote url, or the filesystem path
+     * @param string $git            the git remote url, or the filesystem path
      * @param null              $repositoryPath path
      * @param GitBinary         $binary         binary
      * @param null              $name           repository name
@@ -192,7 +192,7 @@ 
     /**
      * Stage the working tree content
      *
-     * @param string|NodeObject $path the path to store
+     * @param string $path the path to store
      *
      * @throws \RuntimeException
      * @throws \Symfony\Component\Process\Exception\LogicException
@@ -210,7 +210,7 @@ 
     /**
      * Unstage a tree content
      *
-     * @param string|NodeObject $path the path to unstage
+     * @param string $path the path to unstage
      *
      * @throws \RuntimeException
      * @throws \Symfony\Component\Process\Exception\LogicException
@@ -228,8 +228,8 @@ 
     /**
      * Move a file/directory
      *
-     * @param string|NodeObject $from source path
-     * @param string|NodeObject $to   destination path
+     * @param string $from source path
+     * @param string $to   destination path
      *
      * @throws \RuntimeException
      * @throws \Symfony\Component\Process\Exception\LogicException
@@ -248,7 +248,7 @@ 
     /**
      * Remove a file/directory
      *
-     * @param string|NodeObject $path      the path to remove
+     * @param string $path      the path to remove
      * @param bool              $recursive recurse
      * @param bool              $force     force
      *
@@ -328,7 +328,7 @@ 
     }
 
     /**
-     * @param TreeishInterface|Commit|string $arg
+     * @param Commit $arg
      * @param array $options
      */
     public function reset($arg,$options)
@@ -804,7 +804,7 @@ 
      *
      * @param string|TreeishInterface|array $ref         the treeish to check, as a string, as an object or as an array
      * @param string|NodeObject             $path        the physical path to the tree relative to the repository root
-     * @param int|null                      $limit       limit to n entries
+     * @param integer                      $limit       limit to n entries
      * @param int|null                      $offset      skip n entries
      * @param boolean|false                 $firstParent skip commits brought in to branch by a merge
      *
@@ -821,7 +821,7 @@ 
      * @param string            $refStart
      * @param string            $refEnd
      * @param string|NodeObject $path        the physical path to the tree relative to the repository root
-     * @param int|null          $limit       limit to n entries
+     * @param integer          $limit       limit to n entries
      * @param int|null          $offset      skip n entries
      * @param boolean|false     $firstParent skip commits brought in to branch by a merge
      *
@@ -846,7 +846,7 @@ 
      * Get a log for an object
      *
      * @param \GitElephant\Objects\NodeObject         $obj    The Object instance
-     * @param null|string|\GitElephant\Objects\Branch $branch The branch to read from
+     * @param string|null $branch The branch to read from
      * @param int                                     $limit  Limit to n entries
      * @param int|null                                $offset Skip n entries
      *
@@ -915,7 +915,7 @@ 
      * Get a Diff object for a commit with its parent, by default the diff is between the current head and its parent
      *
      * @param \GitElephant\Objects\Commit|string      $commit1 A TreeishInterface instance
-     * @param \GitElephant\Objects\Commit|string|null $commit2 A TreeishInterface instance
+     * @param Commit $commit2 A TreeishInterface instance
      * @param null|string|NodeObject                  $path    The path to get the diff for or a Object instance
      *
      * @throws \RuntimeException
@@ -931,7 +931,7 @@ 
      * Clone a repository
      *
      * @param string $url the repository url (i.e. git://github.com/matteosister/GitElephant.git)
-     * @param null   $to  where to clone the repo
+     * @param string|null   $to  where to clone the repo
      *
      * @throws \RuntimeException
      * @throws \Symfony\Component\Process\Exception\LogicException
@@ -1067,7 +1067,7 @@ 
      * output a node content as an array of lines
      *
      * @param \GitElephant\Objects\NodeObject              $obj     The Object of type BLOB
-     * @param \GitElephant\Objects\TreeishInterface|string $treeish A treeish object
+     * @param null|Branch $treeish A treeish object
      *
      * @throws \RuntimeException
      * @throws \Symfony\Component\Process\Exception\LogicException

tests/GitElephant/TestCase.php

Doc Comments +17 added lines, -3 removed lines

@@ -50,7 +50,7 @@ 
     protected $finder;
 
     /**
-     * @param null $name
+     * @param integer $name
      *
      * @return \GitElephant\Repository
      */
@@ -121,7 +121,7 @@ 
     /**
      * @param string      $name       file name
      * @param string|null $folder     folder name
-     * @param null        $content    content
+     * @param string|null        $content    content
      * @param Repository  $repository repository to add file to
      *
      * @return void
@@ -202,7 +202,7 @@ 
     }
 
     /**
-     * @param $classname
+     * @param string $classname
      *
      * @return \PHPUnit\Framework\MockObject\MockObject
      */
@@ -251,6 +251,9 @@ 
             ->will($this->returnValue($this->getMockCommand()));
     }
 
+    /**
+     * @param string[] $output
+     */
     protected function addOutputToMockRepo(\PHPUnit\Framework\MockObject\MockObject $repo, $output)
     {
         $repo
@@ -287,6 +290,17 @@ 
         return $this->getMock('GitElephant\GitBinary');
     }
 
+    /**
+     * @param string $sha
+     * @param string $tree
+     * @param string $author
+     * @param string $committer
+     * @param string $emailAuthor
+     * @param string $emailCommitter
+     * @param string $datetimeAuthor
+     * @param string $datetimeCommitter
+     * @param string $message
+     */
     protected function doCommitTest(
         Commit $commit,
         $sha,