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

Timestamp:

04/09/2025 01:29:39 PM (5 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/getSite.php (modified) (1 diff)

trunk/tests/phpunit/tests/multisite/getSite.php

@@ -1 +1 @@
 <?php
 
-if ( is_multisite() ) :
-    /**
-     * Test get_site() wrapper of WP_Site in multisite.
-     *
-     * @group ms-site
-     * @group multisite
-     */
-    class Tests_Multisite_GetSite extends WP_UnitTestCase {
-        protected static $site_ids;
+/**
+ * Test get_site() wrapper of WP_Site in multisite.
+ *
+ * @group ms-required
+ * @group ms-site
+ * @group multisite
+ */
+class Tests_Multisite_GetSite extends WP_UnitTestCase {
 
-        public static function wpSetUpBeforeClass( WP_UnitTest_Factory $factory ) {
-            self::$site_ids = array(
-                'wordpress.org/'         => array(
-                    'domain' => 'wordpress.org',
-                    'path'   => '/',
-                ),
-                'wordpress.org/foo/'     => array(
-                    'domain' => 'wordpress.org',
-                    'path'   => '/foo/',
-                ),
-                'wordpress.org/foo/bar/' => array(
-                    'domain' => 'wordpress.org',
-                    'path'   => '/foo/bar/',
-                ),
-            );
+    protected static $site_ids;
 
-            foreach ( self::$site_ids as &$id ) {
-                $id = $factory->blog->create( $id );
-            }
-            unset( $id );
+    public static function wpSetUpBeforeClass( WP_UnitTest_Factory $factory ) {
+        self::$site_ids = array(
+            'wordpress.org/'         => array(
+                'domain' => 'wordpress.org',
+                'path'   => '/',
+            ),
+            'wordpress.org/foo/'     => array(
+                'domain' => 'wordpress.org',
+                'path'   => '/foo/',
+            ),
+            'wordpress.org/foo/bar/' => array(
+                'domain' => 'wordpress.org',
+                'path'   => '/foo/bar/',
+            ),
+        );
+
+        foreach ( self::$site_ids as &$id ) {
+            $id = $factory->blog->create( $id );
+        }
+        unset( $id );
+    }
+
+    public static function wpTearDownAfterClass() {
+        foreach ( self::$site_ids as $id ) {
+            wp_delete_site( $id );
         }
 
-        public static function wpTearDownAfterClass() {
-            foreach ( self::$site_ids as $id ) {
-                wp_delete_site( $id );
-            }
-
-            wp_update_network_site_counts();
-        }
-
-        public function test_get_site_in_switched_state_returns_switched_site() {
-            switch_to_blog( self::$site_ids['wordpress.org/foo/'] );
-            $site = get_site();
-            restore_current_blog();
-
-            $this->assertSame( self::$site_ids['wordpress.org/foo/'], $site->id );
-        }
+        wp_update_network_site_counts();
     }
 
-endif;
+    public function test_get_site_in_switched_state_returns_switched_site() {
+        switch_to_blog( self::$site_ids['wordpress.org/foo/'] );
+        $site = get_site();
+        restore_current_blog();
+
+        $this->assertSame( self::$site_ids['wordpress.org/foo/'], $site->id );
+    }
+}