silverstripe / silverstripe-framework

tests/php/Forms/AssetFieldTest/TestForm.php

Doc Comments +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
 
     /**
      * @skipUpgrade
-     * @param null   $controller
+     * @param TestController   $controller
      * @param string $name
      */
     public function __construct($controller = null, $name = 'Form')

tests/php/Forms/DropdownFieldTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -399,7 +399,7 @@
      * string of HTML.
      *
      * @param  string $html HTML to scan for elements
-     * @return SimpleXMLElement
+     * @return \SimpleXMLElement
      */
     public function findOptionElements($html)
     {

tests/php/Forms/GridField/GridField_URLHandlerTest/TestComponent_ItemRequest.php

Doc Comments +3 added lines

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

tests/php/Forms/NumericFieldTest.php

Doc Comments +3 added lines

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

tests/php/Forms/UploadFieldTest.php

Doc Comments +4 added lines, -1 removed lines

@@ -1023,7 +1023,7 @@ 
      * Simulates a file upload
      *
      * @param  string $fileField   Name of the field to mock upload for
-     * @param  array  $tmpFileName Name of temporary file to upload
+     * @param  string  $tmpFileName Name of temporary file to upload
      * @return HTTPResponse form response
      */
     protected function mockFileUpload($fileField, $tmpFileName)
@@ -1036,6 +1036,9 @@ 
         );
     }
 
+    /**
+     * @param string $fileField
+     */
     protected function mockFileExists($fileField, $fileName)
     {
         return $this->get(

tests/php/ORM/ChangeSetTest/Permissions.php

Doc Comments +3 added lines

@@ -39,6 +39,9 @@
         return $this->can(__FUNCTION__, $member, $context);
     }
 
+    /**
+     * @param string $perm
+     */
     public function can($perm, $member = null, $context = array())
     {
         $perms = [

tests/php/ORM/HierarchyTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -670,7 +670,7 @@
      * Get the HTML class attribute from a node in the sitetree
      *
      * @param  $html
-     * @param  $node
+     * @param  DataObject|null $node
      * @return string
      */
     protected function getNodeClassFromTree($html, $node)

tests/php/Security/GroupTest/MemberForm.php

Doc Comments +4 added lines

@@ -9,6 +9,10 @@
 class MemberForm extends Form
 {
 
+    /**
+     * @param \SilverStripe\Control\Controller $controller
+     * @param string $name
+     */
     public function __construct($controller, $name)
     {
         $fields = TestMember::singleton()->getCMSFields();

tests/php/Security/SecurityTest.php

Doc Comments +4 added lines

@@ -676,6 +676,8 @@ 
     /**
      * Execute a log-in form using Director::test().
      * Helper method for the tests above
+     * @param string $email
+     * @param string $password
      */
     public function doTestLoginForm($email, $password, $backURL = 'test/link')
     {
@@ -697,6 +699,8 @@ 
 
     /**
      * Helper method to execute a change password form
+     * @param string $oldPassword
+     * @param string $newPassword
      */
     public function doTestChangepasswordForm($oldPassword, $newPassword)
     {