mirror of
https://github.com/gosticks/wordpress-develop.git
synced 2026-07-01 15:50:09 +00:00
Users: Correct the error code in wp_insert_user() when login matches an existing email.
Move the test next to the other tests for `user_login`. Follow-up to [55358]. See #57394. git-svn-id: https://develop.svn.wordpress.org/trunk@55360 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
@@ -897,6 +897,24 @@ class Tests_User extends WP_UnitTestCase {
|
||||
$this->assertSame( 'user_login_too_long', $u->get_error_code() );
|
||||
}
|
||||
|
||||
/**
|
||||
* @ticket 57394
|
||||
*/
|
||||
public function test_wp_insert_user_should_reject_user_login_that_matches_existing_user_email() {
|
||||
$existing_email = get_option( 'admin_email' );
|
||||
$user_id = wp_insert_user(
|
||||
array(
|
||||
'user_login' => $existing_email,
|
||||
'user_email' => 'whatever@example.com',
|
||||
'user_pass' => 'whatever',
|
||||
'user_nicename' => 'whatever',
|
||||
)
|
||||
);
|
||||
|
||||
$this->assertWPError( $user_id );
|
||||
$this->assertSame( 'existing_user_email_as_login', $user_id->get_error_code() );
|
||||
}
|
||||
|
||||
/**
|
||||
* @ticket 33793
|
||||
*/
|
||||
@@ -934,24 +952,6 @@ class Tests_User extends WP_UnitTestCase {
|
||||
$this->assertSame( $expected, $user->user_nicename );
|
||||
}
|
||||
|
||||
/**
|
||||
* @ticket 57394
|
||||
*/
|
||||
public function test_wp_insert_user_should_reject_username_that_matches_existing_user_email() {
|
||||
$existing_email = get_option( 'admin_email' );
|
||||
$username = wp_insert_user(
|
||||
array(
|
||||
'user_login' => $existing_email,
|
||||
'user_email' => 'whatever@example.com',
|
||||
'user_pass' => 'whatever',
|
||||
'user_nicename' => 'whatever',
|
||||
)
|
||||
);
|
||||
|
||||
$this->assertWPError( $username );
|
||||
$this->assertSame( 'existing_user_login_as_email', $username->get_error_code() );
|
||||
}
|
||||
|
||||
/**
|
||||
* @ticket 33793
|
||||
*/
|
||||
|
||||
Reference in New Issue
Block a user