Enalean / tuleap

plugins/git/tests/Git/Driver/GerritTest/REST/GerritREST_Base.php

Doc Comments +3 added lines

@@ -79,6 +79,9 @@
         $this->driver = new Git_Driver_GerritREST($this->guzzle_client, $this->logger);
     }
 
+    /**
+     * @param string $text
+     */
     protected function getGuzzleRequestWithTextResponse($text) {
         $response     = stub('Guzzle\Http\Message\Response')->getBody(true)->returns($text);
         return stub('Guzzle\Http\Message\EntityEnclosingRequest')->send()->returns($response);

plugins/git/tests/Git/Gitolite/ProjectSerializerTest.php

Doc Comments +4 added lines

@@ -277,6 +277,10 @@
         $this->assertEqual('project1/repo', $this->project_serializer->repoFullName($repo, $unix_name));
     }
 
+    /**
+     * @param string $name
+     * @param string $namespace
+     */
     private function _GivenARepositoryWithNameAndNamespace($name, $namespace) {
         $repo = new GitRepository();
         $repo->setName($name);

plugins/git/tests/Git/Mirror/ManifestFileGeneratorTest.php

Doc Comments +9 added lines

@@ -73,12 +73,18 @@ 
         parent::tearDown();
     }
 
+    /**
+     * @param string $path
+     */
     protected function getManifestContent($path) {
         $content = file_get_contents("compress.zlib://$path");
 
         return json_decode($content, true);
     }
 
+    /**
+     * @param string $path
+     */
     protected function forgeExistingManifestFile($path) {
         file_put_contents(
             "compress.zlib://$path",
@@ -86,6 +92,9 @@ 
         );
     }
 
+    /**
+     * @param string $path
+     */
     protected function forgeExistingManifestFileWithGitoliteAdmin($path) {
         file_put_contents(
             "compress.zlib://$path",

plugins/git/tests/Git/URLTest.php

Doc Comments +4 added lines, -2 removed lines

@@ -164,7 +164,8 @@ 
         $this->assertNull($url->getProject());
     }
 
-    /** @return Git_URL */
+    /** @param string $uri
+/** @return Git_URL */
     private function getGitURL($uri = null) {
         return new Git_URL(
             $this->project_manager,
@@ -337,7 +338,8 @@ 
         $this->assertEqual($url->getQueryString(), 'service=git-upload-pack');
     }
 
-    /** @return Git_URL */
+    /** @param string $url
+/** @return Git_URL */
     private function getUrl($url) {
         return new Git_URL(
             $this->project_manager,

plugins/git/tests/Git_Backend_GitoliteTest.php

Doc Comments +8 added lines

@@ -34,6 +34,10 @@ 
 
 abstract class Git_Backend_GitoliteCommonTest extends TuleapTestCase {
 
+    /**
+     * @param string $name
+     * @param string $namespace
+     */
     protected function _GivenAGitRepoWithNameAndNamespace($name, $namespace) {
         $repository = new GitRepository();
         $repository->setName($name);
@@ -92,6 +96,10 @@ 
         unlink(dirname(__FILE__).'/_fixtures/tmp/perms');
     }
     
+    /**
+     * @param string $className
+     * @param string[] $methods
+     */
     function getPartialMock($className, $methods) {
         $partialName = $className.'Partial'.uniqid();
         Mock::generatePartial($className, $partialName, $methods);

plugins/git/tests/Git_GitoliteDriverTest.php

Doc Comments +15 added lines

@@ -130,6 +130,9 @@ 
         return file_get_contents($this->_glAdmDir.'/conf/gitolite.conf');
     }
 
+    /**
+     * @param string $filename
+     */
     protected function getFileConf($filename) {
         return file_get_contents($this->_glAdmDir.'/conf/'.$filename.'.conf');
     }
@@ -213,6 +216,11 @@ 
         return $groupInfo['name'];
     }
 
+    /**
+     * @param string $repoPath
+     * @param string $namespace
+     * @param string $group
+     */
     protected function assertNameSpaceFileHasBeenInitialized($repoPath, $namespace, $group) {
         $namespaceInfoFile = $repoPath.'/tuleap_namespace';
         $this->assertTrue(file_exists($namespaceInfoFile), 'the file (' . $namespaceInfoFile . ') does not exists');
@@ -221,6 +229,10 @@ 
 
     }
 
+    /**
+     * @param string $new_root_dir
+     * @param string $group
+     */
     protected function assertWritableByGroup($new_root_dir, $group) {
         $this->assertEqual($group, $this->_getFileGroupName($new_root_dir));
         $this->assertEqual($group, $this->_getFileGroupName($new_root_dir .'/hooks/gitolite_hook.sh'));
@@ -230,6 +242,9 @@ 
         $this->assertPattern('/.*770$/', decoct($rootStats[2]));
     }
 
+    /**
+     * @param string $new_root_dir
+     */
     public function assertRepoIsClonedWithHooks($new_root_dir) {
         $this->assertTrue(is_dir($new_root_dir), "the new git repo dir ($new_root_dir) wasn't found.");
         $new_repo_HEAD = $new_root_dir . '/HEAD';

plugins/git/tests/GitActionsTest.php

Doc Comments +4 added lines

@@ -617,6 +617,10 @@
         $this->changeProjectRepositoriesAccess($project_id, $is_private);
     }
 
+    /**
+     * @param integer $project_id
+     * @param boolean $is_private
+     */
     private function changeProjectRepositoriesAccess($project_id, $is_private) {
         return GitActions::changeProjectRepositoriesAccess($project_id, $is_private, $this->dao, $this->factory);
     }

plugins/git/tests/GitRepositoryManagerTest.php

Doc Comments +15 added lines

@@ -94,6 +94,9 @@ 
         $this->manager    = new GitRepositoryManager($this->factory, mock('Git_SystemEventManager'), $this->dao, $this->backup_directory);
     }
 
+    /**
+     * @param string $path
+     */
     private function aRepoWithPath($path) {
         return aGitRepository()->withPath($this->project_name.'/'.$path.'.git')->withProject($this->project)->build();
     }
@@ -229,6 +232,9 @@ 
 class GitRepositoryIdMatchExpectation extends SimpleExpectation {
     private $repository_id;
 
+    /**
+     * @param integer $repository_id
+     */
     public function __construct($repository_id) {
         parent::__construct();
         $this->repository_id = $repository_id;
@@ -397,6 +403,9 @@ 
         $this->manager->forkRepositories($repos, $to_project, $this->user, '', null, $this->forkPermissions);
     }
 
+    /**
+     * @param string[] $repo_ids
+     */
     protected function getRepoCollectionUnreadableFor($repo_ids, $user) {
         $return = array();
         foreach ($repo_ids as $id) {
@@ -487,6 +496,9 @@ 
         $this->forkRepositories(array($this->repository), '^toto/pouet');
     }
 
+    /**
+     * @param integer $id
+     */
     private function GivenARepository($id) {
         $repo = new MockGitRepository();
         $repo->setReturnValue('getId', $id);
@@ -496,6 +508,9 @@ 
         return $repo;
     }
 
+    /**
+     * @param string $namespace
+     */
     private function forkRepositories($repositories, $namespace=null) {
         $this->manager->forkRepositories($repositories, $this->project, $this->user, $namespace, null, $this->forkPermissions);
     }

plugins/git/tests/GitRepositoryTest.php

Doc Comments +4 added lines

@@ -140,6 +140,10 @@
         $this->assertEqual('u/johan/tulip', $repo->getFullName());
     }
 
+    /**
+     * @param string $name
+     * @param null|string $namespace
+     */
     protected function _GivenARepositoryWithNameAndNamespace($name, $namespace) {
         $repo = new GitRepository();
         $repo->setName($name);