owncloud / core

tests/lib/command/asyncbus.php

Doc Comments +3 added lines

@@ -24,6 +24,9 @@
 class StateFullCommand implements ICommand {
 	private $state;
 
+	/**
+	 * @param string $state
+	 */
 	function __construct($state) {
 		$this->state = $state;
 	}

tests/lib/contacts/localadressbook.php

Doc Comments +4 added lines

@@ -46,6 +46,10 @@
 
 class SimpleUserForTesting implements \OCP\IUser {
 
+	/**
+	 * @param string $uid
+	 * @param string $displayName
+	 */
 	public function __construct($uid, $displayName) {
 
 		$this->uid = $uid;

tests/lib/datetimeformatter.php

Doc Comments +3 added lines

@@ -37,6 +37,9 @@
 		$this->formatter = new \OC\DateTimeFormatter(new \DateTimeZone('UTC'), new \OC_L10N('lib', 'en'));
 	}
 
+	/**
+	 * @param integer $time
+	 */
 	protected function getTimestampAgo($time, $seconds = 0, $minutes = 0, $hours = 0, $days = 0, $years = 0) {
 		return $time - $seconds - $minutes * 60 - $hours * 3600 - $days * 24*3600 - $years * 365*24*3600;
 	}

tests/lib/db.php

Doc Comments +8 added lines

@@ -322,11 +322,19 @@
 		$this->assertSame(2, $query->execute(array('uri1')));
 	}
 
+	/**
+	 * @param string $fullname
+	 * @param string $uri
+	 */
 	protected function insertCardData($fullname, $uri) {
 		$query = OC_DB::prepare("INSERT INTO `*PREFIX*{$this->table2}` (`fullname`, `uri`, `carddata`) VALUES (?, ?, ?)");
 		$this->assertSame(1, $query->execute(array($fullname, $uri, $this->getUniqueID())));
 	}
 
+	/**
+	 * @param string $fullname
+	 * @param string $uri
+	 */
 	protected function updateCardData($fullname, $uri) {
 		$query = OC_DB::prepare("UPDATE `*PREFIX*{$this->table2}` SET `uri` = ? WHERE `fullname` = ?");
 		return $query->execute(array($uri, $fullname));

tests/lib/db/migrator.php

Doc Comments +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
 	}
 
 	/**
-	 * @return \Doctrine\DBAL\Schema\Schema[]
+	 * @return Schema[]
 	 */
 	private function getDuplicateKeySchemas() {
 		$startSchema = new Schema(array(), array(), $this->getSchemaConfig());

tests/lib/encryption/updatetest.php

Doc Comments +1 added lines

@@ -217,6 +217,7 @@
 	 * create mock of the update method
 	 *
 	 * @param array$methods methods which should be set
+	 * @param string[] $methods
 	 * @return \OC\Encryption\Update | \PHPUnit_Framework_MockObject_MockObject
 	 */
 	protected function getUpdateMock($methods) {

tests/lib/files/view.php

Doc Comments +16 added lines, -2 removed lines

@@ -415,6 +415,10 @@ 
 		$this->copyBetweenStorages($storage1, $storage2);
 	}
 
+	/**
+	 * @param \OC\Files\Storage\Storage $storage1
+	 * @param \OC\Files\Storage\Storage $storage2
+	 */
 	function copyBetweenStorages($storage1, $storage2) {
 		\OC\Files\Filesystem::mount($storage1, array(), '/');
 		\OC\Files\Filesystem::mount($storage2, array(), '/substorage');
@@ -461,6 +465,10 @@ 
 		$this->moveBetweenStorages($storage1, $storage2);
 	}
 
+	/**
+	 * @param \OC\Files\Storage\Storage $storage1
+	 * @param \OC\Files\Storage\Storage $storage2
+	 */
 	function moveBetweenStorages($storage1, $storage2) {
 		\OC\Files\Filesystem::mount($storage1, array(), '/');
 		\OC\Files\Filesystem::mount($storage2, array(), '/substorage');
@@ -895,6 +903,9 @@ 
 		return $this->relativePathProvider(null);
 	}
 
+	/**
+	 * @param string|null $missingRootExpectedPath
+	 */
 	public function relativePathProvider($missingRootExpectedPath) {
 		return array(
 			// No root - returns the path
@@ -1095,6 +1106,9 @@ 
 		$this->doTestCopyRenameFail('copy');
 	}
 
+	/**
+	 * @param string $operation
+	 */
 	private function doTestCopyRenameFail($operation) {
 		$storage1 = new Temporary(array());
 		/** @var \PHPUnit_Framework_MockObject_MockObject | \OC\Files\Storage\Temporary $storage2 */
@@ -1509,7 +1523,7 @@ 
 	/**
 	 * Create test movable mount points
 	 *
-	 * @param array $mountPoints array of mount point locations
+	 * @param string[] $mountPoints array of mount point locations
 	 * @return array array of MountPoint objects
 	 */
 	private function createTestMovableMountPoints($mountPoints) {
@@ -1735,7 +1749,7 @@ 
 	 * @param string $lockedPath path of the locked item to check
 	 * @param string $hookType hook type
 	 * @param int $expectedLockBefore expected lock during pre hooks
-	 * @param int $expectedLockduring expected lock during operation
+	 * @param int $expectedLockDuring expected lock during operation
 	 * @param int $expectedLockAfter expected lock during post hooks
 	 * @param int $expectedStrayLock expected lock after returning, should
 	 * be null (unlock) for most operations

tests/lib/helper.php

Doc Comments +2 added lines, -2 removed lines

@@ -517,8 +517,8 @@
 	/**
 	 * Allows us to test private methods/properties
 	 *
-	 * @param $object
-	 * @param $methodName
+	 * @param OC\Files\View $object
+	 * @param string $methodName
 	 * @param array $parameters
 	 * @return mixed
 	 * @deprecated Please extend \Test\TestCase and use self::invokePrivate() then

tests/lib/repair/repairlegacystorage.php

Doc Comments +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
 	 * Returns the storage id based on the numeric id
 	 *
 	 * @param int $storageId numeric id of the storage
-	 * @return string storage id or null if not found
+	 * @return integer|null storage id or null if not found
 	 */
 	private function getStorageId($storageId) {
 		$numericId = Storage::getNumericStorageId($storageId);