Changeset 61714

Timestamp:

02/21/2026 10:40:24 PM (7 months ago)

Author:

SergeyBiryukov

Message:

Tests: Move test_illegal_user_logins_single() to the correct place.

Follow-up to [35189], [35629], [35772], [61656].

See #64225.

File:

• trunk/tests/phpunit/tests/user.php (modified) (2 diffs)

trunk/tests/phpunit/tests/user.php

@@ -1020 +1020 @@
 
         /**
-         * @ticket 27317
-         * @dataProvider data_illegal_user_logins
-         */
-        public function test_illegal_user_logins_single( $user_login ) {
-                $user_data = array(
-                        'user_login' => $user_login,
-                        'user_email' => 'testuser@example.com',
-                        'user_pass'  => wp_generate_password(),
-                );
-
-                add_filter( 'illegal_user_logins', array( $this, 'illegal_user_logins' ) );
-
-                $response = wp_insert_user( $user_data );
-                $this->assertInstanceOf( 'WP_Error', $response );
-                $this->assertSame( 'invalid_username', $response->get_error_code() );
-
-                remove_filter( 'illegal_user_logins', array( $this, 'illegal_user_logins' ) );
-
-                $user_id = wp_insert_user( $user_data );
-                $user    = get_user_by( 'id', $user_id );
-                $this->assertInstanceOf( 'WP_User', $user );
-        }
-
-        /**
          * @ticket 61175
          * @covers ::wp_insert_user
@@ -1180 +1156 @@
                 $user = new WP_User( $user_id );
                 $this->assertSame( $internal_data['user_login'], $user->user_login );
+        }
+
+        /**
+         * @ticket 27317
+         * @dataProvider data_illegal_user_logins
+         */
+        public function test_illegal_user_logins_single( $user_login ) {
+                $user_data = array(
+                        'user_login' => $user_login,
+                        'user_email' => 'testuser@example.com',
+                        'user_pass'  => wp_generate_password(),
+                );
+
+                add_filter( 'illegal_user_logins', array( $this, 'illegal_user_logins' ) );
+
+                $response = wp_insert_user( $user_data );
+                $this->assertInstanceOf( 'WP_Error', $response );
+                $this->assertSame( 'invalid_username', $response->get_error_code() );
+
+                remove_filter( 'illegal_user_logins', array( $this, 'illegal_user_logins' ) );
+
+                $user_id = wp_insert_user( $user_data );
+                $user    = get_user_by( 'id', $user_id );
+                $this->assertInstanceOf( 'WP_User', $user );
         }