Changeset 42043 for trunk/tests/phpunit/tests/multisite/getMainSiteId.php

Timestamp:

10/30/2017 07:55:09 PM (7 years ago)

Author:

flixos90

Message:

Multisite: Pass network object instead of ID to pre_get_main_site_id.

In the original function introduced in [41380], and subsequently [41861], only the network ID was passed to the new pre_get_main_site_id filter. In order to make give quick access to other network properties, it was decided to pass the whole object instead. The changeset includes an additional test.

Fixes #29684.

File:

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

trunk/tests/phpunit/tests/multisite/getMainSiteId.php

@@ -108 +108 @@
 
     /**
+     * @ticket 29684
+     */
+    public function test_get_main_site_id_filtered_depending_on_network() {
+        add_filter( 'pre_get_main_site_id', array( $this, 'filter_get_main_site_id_depending_on_network' ), 10, 2 );
+        $result = get_main_site_id( self::$network_ids['wordpress.org/'] );
+
+        $this->assertSame( 333, $result );
+    }
+
+    public function filter_get_main_site_id_depending_on_network( $main_site_id, $network ) {
+        // Override main site ID for a specific network for the test.
+        if ( $network->id === (int) self::$network_ids['wordpress.org/'] ) {
+            return 333;
+        }
+
+        return $main_site_id;
+    }
+
+    /**
      * @ticket 41936
      */