From 4992fdb95fe0e5e06c9fc2cad9cb3e6ffcdde7a7 Mon Sep 17 00:00:00 2001 From: Boone Gorges Date: Sat, 22 Nov 2014 21:29:40 +0000 Subject: [PATCH] Ensure `sanitize_user()` expected test values are lowercase on multisite. [30524] neglected to account for the fact that multisite forces user logins to lowercase. See #30284. git-svn-id: https://develop.svn.wordpress.org/trunk@30527 602fd350-edb4-49c9-b593-d223f7449a82 --- .../phpunit/tests/formatting/SanitizeUser.php | 18 ++++++++++++++++-- 1 file changed, 16 insertions(+), 2 deletions(-) diff --git a/tests/phpunit/tests/formatting/SanitizeUser.php b/tests/phpunit/tests/formatting/SanitizeUser.php index 5087df7bf0..a6d15ac821 100644 --- a/tests/phpunit/tests/formatting/SanitizeUser.php +++ b/tests/phpunit/tests/formatting/SanitizeUser.php @@ -11,11 +11,25 @@ class Tests_Formatting_SanitizeUser extends WP_UnitTestCase { } public function test_strips_encoded_ampersand() { - $this->assertEquals("ATT", sanitize_user("AT&T")); + $expected = 'ATT'; + + // Multisite forces user logins to lowercase. + if ( is_multisite() ) { + $expected = strtolower( $expected ); + } + + $this->assertEquals( $expected, sanitize_user( "AT&T" ) ); } public function test_strips_encoded_ampersand_when_followed_by_semicolon() { - $this->assertEquals("ATT Test;", sanitize_user("AT&T Test;")); + $expected = 'ATT Test;'; + + // Multisite forces user logins to lowercase. + if ( is_multisite() ) { + $expected = strtolower( $expected ); + } + + $this->assertEquals( $expected, sanitize_user( "AT&T Test;" ) ); } function test_strips_percent_encoded_octets() {