Changeset 60148 for trunk/tests/phpunit/tests/multisite/wpmuLogNewRegistrations.php

Timestamp:

04/09/2025 01:29:39 PM (6 weeks ago)

Author:

SergeyBiryukov

Message:

Tests: Use the ms-required group where appropriate.

This replaces the if ( is_multisite() ) conditional wrapping entire test classes with the ms-required group for more consistency across the test suite.

Follow-up to [40520].

See #63167.

File:

• trunk/tests/phpunit/tests/multisite/wpmuLogNewRegistrations.php (modified) (1 diff)

trunk/tests/phpunit/tests/multisite/wpmuLogNewRegistrations.php

@@ -1 +1 @@
 <?php
 
-if ( is_multisite() ) :
+/**
+ * @group ms-required
+ * @group multisite
+ */
+class Tests_Multisite_wpmuLogNewRegistrations extends WP_UnitTestCase {
 
-    /**
-     * @group multisite
-     */
-    class Tests_Multisite_wpmuLogNewRegistrations extends WP_UnitTestCase {
+    public function test_wpmu_log_new_registrations() {
+        global $wpdb;
 
-        public function test_wpmu_log_new_registrations() {
-            global $wpdb;
+        $user = new WP_User( 1 );
+        $ip   = preg_replace( '/[^0-9., ]/', '', $_SERVER['REMOTE_ADDR'] );
 
-            $user = new WP_User( 1 );
-            $ip   = preg_replace( '/[^0-9., ]/', '', $_SERVER['REMOTE_ADDR'] );
+        wpmu_log_new_registrations( 1, 1 );
 
-            wpmu_log_new_registrations( 1, 1 );
-
-            // Currently there is no wrapper function for the registration_log.
-            $reg_blog = $wpdb->get_col( $wpdb->prepare( "SELECT email FROM {$wpdb->registration_log} WHERE {$wpdb->registration_log}.blog_id = 1 AND IP LIKE %s", $ip ) );
-            $this->assertSame( $user->user_email, $reg_blog[ count( $reg_blog ) - 1 ] );
-        }
+        // Currently there is no wrapper function for the registration_log.
+        $reg_blog = $wpdb->get_col( $wpdb->prepare( "SELECT email FROM {$wpdb->registration_log} WHERE {$wpdb->registration_log}.blog_id = 1 AND IP LIKE %s", $ip ) );
+        $this->assertSame( $user->user_email, $reg_blog[ count( $reg_blog ) - 1 ] );
     }
-
-endif;
+}