Changeset 38681 for trunk/tests/phpunit/tests/multisite/bootstrap.php

Timestamp:

09/30/2016 04:09:45 AM (8 years ago)

Author:

jeremyfelt

Message:

Multisite: Ensure a consistent WP_Site return from get_site_by_path().

It is possible to short circuit get_site_by_path() using the pre_get_site_by_path filter. When this happens and a standard site object is provided, we can make sure it is upgraded to a proper WP_Site object before passing it on.

Props flixos90.
Fixes #37053.

File:

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

trunk/tests/phpunit/tests/multisite/bootstrap.php

@@ -209 +209 @@
     }
 
+    /**
+     * @ticket 37053
+     */
+    public function test_get_site_by_path_returns_wp_site() {
+        add_filter( 'pre_get_site_by_path', array( $this, 'filter_pre_get_site_by_path' ), 10, 3 );
+
+        $site = get_site_by_path( 'example.com', '/foo/' );
+
+        remove_filter( 'pre_get_site_by_path', array( $this, 'filter_pre_get_site_by_path' ), 10 );
+
+        $this->assertInstanceOf( 'WP_Site', $site );
+    }
+
     public function filter_path_segments_to_two() {
         return 2;
     }
+
+    public function filter_pre_get_site_by_path( $site, $domain, $path ) {
+        $site = new stdClass();
+        $site->blog_id = 100;
+        $site->domain = $domain;
+        $site->path = $path;
+        $site->site_id = 1;
+
+        return $site;
+    }
 }