Skip to content

Commit 7924b71

Browse files
Tests: Move test_illegal_user_logins_single() to the correct place.
Follow-up to [35189], [35629], [35772], [61656]. See #64225. git-svn-id: https://develop.svn.wordpress.org/trunk@61714 602fd350-edb4-49c9-b593-d223f7449a82
1 parent 95899ee commit 7924b71

1 file changed

Lines changed: 24 additions & 24 deletions

File tree

tests/phpunit/tests/user.php

Lines changed: 24 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1018,30 +1018,6 @@ public function test_user_update_email_error() {
10181018
}
10191019
}
10201020

1021-
/**
1022-
* @ticket 27317
1023-
* @dataProvider data_illegal_user_logins
1024-
*/
1025-
public function test_illegal_user_logins_single( $user_login ) {
1026-
$user_data = array(
1027-
'user_login' => $user_login,
1028-
'user_email' => 'testuser@example.com',
1029-
'user_pass' => wp_generate_password(),
1030-
);
1031-
1032-
add_filter( 'illegal_user_logins', array( $this, 'illegal_user_logins' ) );
1033-
1034-
$response = wp_insert_user( $user_data );
1035-
$this->assertInstanceOf( 'WP_Error', $response );
1036-
$this->assertSame( 'invalid_username', $response->get_error_code() );
1037-
1038-
remove_filter( 'illegal_user_logins', array( $this, 'illegal_user_logins' ) );
1039-
1040-
$user_id = wp_insert_user( $user_data );
1041-
$user = get_user_by( 'id', $user_id );
1042-
$this->assertInstanceOf( 'WP_User', $user );
1043-
}
1044-
10451021
/**
10461022
* @ticket 61175
10471023
* @covers ::wp_insert_user
@@ -1181,6 +1157,30 @@ public function offsetUnset( $offset ): void {
11811157
$this->assertSame( $internal_data['user_login'], $user->user_login );
11821158
}
11831159

1160+
/**
1161+
* @ticket 27317
1162+
* @dataProvider data_illegal_user_logins
1163+
*/
1164+
public function test_illegal_user_logins_single( $user_login ) {
1165+
$user_data = array(
1166+
'user_login' => $user_login,
1167+
'user_email' => 'testuser@example.com',
1168+
'user_pass' => wp_generate_password(),
1169+
);
1170+
1171+
add_filter( 'illegal_user_logins', array( $this, 'illegal_user_logins' ) );
1172+
1173+
$response = wp_insert_user( $user_data );
1174+
$this->assertInstanceOf( 'WP_Error', $response );
1175+
$this->assertSame( 'invalid_username', $response->get_error_code() );
1176+
1177+
remove_filter( 'illegal_user_logins', array( $this, 'illegal_user_logins' ) );
1178+
1179+
$user_id = wp_insert_user( $user_data );
1180+
$user = get_user_by( 'id', $user_id );
1181+
$this->assertInstanceOf( 'WP_User', $user );
1182+
}
1183+
11841184
/**
11851185
* @ticket 27317
11861186
* @dataProvider data_illegal_user_logins

0 commit comments

Comments
 (0)