Enalean / tuleap

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);

plugins/git/tests/GitTest.php

Doc Comments +6 added lines

@@ -229,6 +229,9 @@ 
         $this->assertItNeedsAValidRepoId();
     }
 
+    /**
+     * @param Codendi_Request $request
+     */
     protected function getGit($request, $factory) {
         $git = parent::getGit($request, $factory);
         $git->setAction('disconnect_gerrit');
@@ -332,6 +335,9 @@ 
         $git->request();
     }
 
+    /**
+     * @param Codendi_Request $request
+     */
     protected function getGit($request, $factory, $template_factory = null) {
         $git = parent::getGit($request, $factory, $template_factory);
         $git->setAction('migrate_to_gerrit');

plugins/git/tests/GitViewsRepositoriesTraversalStrategyTest.class.php

Doc Comments +3 added lines

@@ -26,6 +26,9 @@
 
 abstract class GitViewsRepositoriesTraversalStrategyTest extends TuleapTestCase {
     
+    /**
+     * @param string $classname
+     */
     public function __construct($classname) {
         parent::__construct();
         $this->classname = $classname;

plugins/git/tests/GitViewsTest.php

Doc Comments +7 added lines

@@ -86,6 +86,9 @@ 
 
     }
 
+    /**
+     * @param string $unixName
+     */
     private function GivenAProject($id, $name, $unixName = null, $useGit = true) {
         $project = new MockProject();
         $project->setReturnValue('getId', $id);
@@ -95,6 +98,10 @@ 
         return $project;
     }
 
+    /**
+     * @param string $id
+     * @param string $name
+     */
     private function GivenAProjectWithoutGitService($id, $name) {
         return $this->GivenAProject($id, $name, null, false);
     }