silverstripe / silverstripe-framework

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/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/NumericFieldTest.php

Doc Comments +3 added lines

@@ -21,6 +21,9 @@
 		return str_replace(' ', $nbsp, trim($input));
 	}
 
+	/**
+	 * @param string $locale
+	 */
 	protected function checkInputValidation($locale, $tests) {
 		i18n::set_locale($locale);
 		$field = new NumericField('Number');

tests/forms/RequirementsTest.php

Doc Comments +4 added lines

@@ -563,6 +563,10 @@
 		}
 	}
 
+	/**
+	 * @param Requirements_Backend $backend
+	 * @param string $type
+	 */
 	public function assertFileNotIncluded($backend, $type, $files) {
 		$includedFiles = $this->getBackendFiles($backend, $type);
 		if(is_array($files)) {

tests/i18n/i18nTextCollectorTest.php

Doc Comments +7 added lines

@@ -827,6 +827,10 @@ 
  * Assist with testing of specific protected methods
  */
 class i18nTextCollectorTest_Collector extends i18nTextCollector implements TestOnly {
+
+	/**
+	 * @param string $directory
+	 */
 	public function getModules_Test($directory) {
 		return $this->getModules($directory);
 	}
@@ -835,6 +839,9 @@ 
 		return $this->resolveDuplicateConflicts($entitiesByModule);
 	}
 
+	/**
+	 * @param string $module
+	 */
 	public function getFileListForModule_Test($module) {
 		return parent::getFileListForModule($module);
 	}

tests/injector/InjectorTest.php

Doc Comments +3 added lines

@@ -933,6 +933,9 @@
 class SSObjectCreator extends InjectionCreator {
 	private $injector;
 
+	/**
+	 * @param Injector $injector
+	 */
 	public function __construct($injector) {
 		$this->injector = $injector;
 	}

tests/model/ArrayListTest.php

Doc Comments +4 added lines

@@ -802,6 +802,10 @@
 	public $First;
 	public $Second;
 
+	/**
+	 * @param string $first
+	 * @param string $second
+	 */
 	public function __construct($first, $second) {
 		$this->First  = $first;
 		$this->Second = $second;