silverstripe / silverstripe-framework

tests/core/startup/ErrorControlChainTest.php

Doc Comments +4 added lines

@@ -9,6 +9,10 @@
 class ErrorControlChainTest_Chain extends ErrorControlChain {
 
 	// Change function visibility to be testable directly
+
+	/**
+	 * @param string $memstring
+	 */
 	public function translateMemstring($memstring) {
 		return parent::translateMemstring($memstring);
 	}

tests/core/startup/ParameterConfirmationTokenTest.php

Doc Comments +3 added lines

@@ -23,6 +23,9 @@
 
 class ParameterConfirmationTokenTest extends SapphireTest {
 
+	/**
+	 * @param string $part
+	 */
 	private function addPart($answer, $slash, $part) {
 		$bare = str_replace('/', '', $part);
 

tests/dev/CsvBulkLoaderTest.php

Doc Comments +3 added lines

@@ -222,6 +222,9 @@
 	}
 
 
+	/**
+	 * @param resource $file
+	 */
 	protected function getLineCount(&$file) {
 		$i = 0;
 		while(fgets($file) !== false) $i++;

tests/dev/DevAdminControllerTest.php

Doc Comments +6 added lines

@@ -47,6 +47,9 @@ 
 
 
 
+	/**
+	 * @param string $url
+	 */
 	protected function getCapture($url){
 		$this->logInWithPermission('ADMIN');
 
@@ -58,6 +61,9 @@ 
 		return $r;
 	}
 
+	/**
+	 * @param string $url
+	 */
 	protected function getAndCheckForError($url){
 		$this->logInWithPermission('ADMIN');
 

tests/filesystem/AssetStoreTest.php

Doc Comments +11 added lines

@@ -487,15 +487,26 @@
 		return parent::cleanFilename($filename);
 	}
 	
+	/**
+	 * @param string $filename
+	 * @param string $hash
+	 * @param string $variant
+	 */
 	public function getFileID($filename, $hash, $variant = null) {
 		return parent::getFileID($filename, $hash, $variant);
 	}
 
 
+	/**
+	 * @param string $fileID
+	 */
 	public function getOriginalFilename($fileID, &$variant = '') {
 		return parent::getOriginalFilename($fileID, $variant);
 	}
 
+	/**
+	 * @param null|string $variant
+	 */
 	public function getDefaultConflictResolution($variant) {
 		return parent::getDefaultConflictResolution($variant);
 	}

tests/filesystem/FileFinderTest.php

Doc Comments +5 added lines

@@ -98,6 +98,11 @@
 			'The finder respects the max depth setting.');
 	}
 
+	/**
+	 * @param SS_FileFinder $finder
+	 * @param string[] $expect
+	 * @param string $message
+	 */
 	public function assertFinderFinds($finder, $expect, $message = null) {
 		$found = $finder->find($this->base);
 

tests/forms/EmailFieldTest.php

Doc Comments +5 added lines

@@ -23,6 +23,11 @@
 		$this->internalCheck("domain.but.no.user", "Invalid, no user part", false);
 	}
 
+	/**
+	 * @param string $email
+	 * @param string $checkText
+	 * @param boolean $expectSuccess
+	 */
 	public function internalCheck($email, $checkText, $expectSuccess) {
 		$field = new EmailField("MyEmail");
 		$field->setValue($email);

tests/forms/gridfield/GridField_URLHandlerTest.php

Doc Comments +3 added lines

@@ -111,6 +111,9 @@
 
 	protected $id;
 
+	/**
+	 * @param string $link
+	 */
 	public function __construct($gridField, $id, $link) {
 		$this->gridField = $gridField;
 		$this->id = $id;

tests/forms/MemberDatetimeOptionsetFieldTest.php

Doc Comments +6 added lines

@@ -7,6 +7,9 @@ 
 
 	protected static $fixture_file = 'MemberDatetimeOptionsetFieldTest.yml';
 
+	/**
+	 * @param DataObject|null $member
+	 */
 	protected function createDateFormatFieldForMember($member) {
 		require_once 'Zend/Date.php';
 		$defaultDateFormat = Zend_Locale_Format::getDateFormat($member->Locale);
@@ -26,6 +29,9 @@ 
 		return $field;
 	}
 
+	/**
+	 * @param DataObject|null $member
+	 */
 	protected function createTimeFormatFieldForMember($member) {
 		require_once 'Zend/Date.php';
 		$defaultTimeFormat = Zend_Locale_Format::getTimeFormat($member->Locale);