Test that get_user_by_username accepts URL encoded input

diff --git a/engine/tests/objects/users.php b/engine/tests/objects/users.php
index dc91293268684d00f23a90983f2bbfae337eb769..7d2ef6961bff5c343fced96f994b0d4bf51c29a1 100644 (file)
--- a/engine/tests/objects/users.php
+++ b/engine/tests/objects/users.php
@@ -159,6 +159,22 @@ class ElggCoreUserTest extends ElggCoreUnitTest {
                $this->assertFalse($user);
        }
 
+       public function testGetUserByUsernameAcceptsUrlEncoded() {
+               $username = (string)time();
+               $this->user->username = $username;
+               $guid = $this->user->save();
+
+               // percent encode first letter
+               $first_letter = $username[0];
+               $first_letter = str_pad('%' . dechex(ord($first_letter)), 2, '0', STR_PAD_LEFT);
+               $username =   $first_letter . substr($username, 1);
+
+               $user = get_user_by_username($username);
+               $this->assertTrue((bool) $user);
+               $this->assertEqual($guid, $user->guid);
+
+               $this->user->delete();
+       }
 
        public function testElggUserMakeAdmin() {
                global $CONFIG;