Changeset 37479 for trunk/tests/phpunit/tests/post/getPageByPath.php

Timestamp:

05/21/2016 05:26:55 PM (7 years ago)

Author:

boonebgorges

Message:

Cache queries in get_page_by_path().

Props spacedmonkey.
Fixes #36711.

File:

• trunk/tests/phpunit/tests/post/getPageByPath.php (modified) (1 diff)

trunk/tests/phpunit/tests/post/getPageByPath.php

@@ -124 +124 @@
         $this->assertNull( $found );
     }
+
+    /**
+     * @ticket 36711
+     */
+    public function test_should_hit_cache() {
+        global $wpdb;
+
+        $page = self::factory()->post->create( array(
+            'post_type' => 'page',
+            'post_name' => 'foo',
+        ) );
+
+        // Prime cache.
+        $found = get_page_by_path( 'foo' );
+        $this->assertSame( $page, $found->ID );
+
+        $num_queries = $wpdb->num_queries;
+
+        $found = get_page_by_path( 'foo' );
+        $this->assertSame( $page, $found->ID );
+        $this->assertSame( $num_queries, $wpdb->num_queries );
+    }
+
+    /**
+     * @ticket 36711
+     */
+    public function test_bad_path_should_be_cached() {
+        global $wpdb;
+
+        // Prime cache.
+        $found = get_page_by_path( 'foo' );
+        $this->assertNull( $found );
+
+        $num_queries = $wpdb->num_queries;
+
+        $found = get_page_by_path( 'foo' );
+        $this->assertNull( $found );
+        $this->assertSame( $num_queries, $wpdb->num_queries );
+    }
+
+    /**
+     * @ticket 36711
+     */
+    public function test_bad_path_served_from_cache_should_not_fall_back_on_current_post() {
+        global $wpdb, $post;
+
+        // Fake the global.
+        $post = self::factory()->post->create_and_get();
+
+        // Prime cache.
+        $found = get_page_by_path( 'foo' );
+        $this->assertNull( $found );
+
+        $num_queries = $wpdb->num_queries;
+
+        $found = get_page_by_path( 'foo' );
+        $this->assertNull( $found );
+        $this->assertSame( $num_queries, $wpdb->num_queries );
+
+        unset( $post );
+    }
+
+    /**
+     * @ticket 36711
+     */
+    public function test_cache_should_not_match_post_in_different_post_type_with_same_path() {
+        global $wpdb;
+
+        register_post_type( 'wptests_pt' );
+
+        $p1 = self::factory()->post->create( array(
+            'post_type' => 'page',
+            'post_name' => 'foo',
+        ) );
+
+        $p2 = self::factory()->post->create( array(
+            'post_type' => 'wptests_pt',
+            'post_name' => 'foo',
+        ) );
+
+        // Prime cache for the page.
+        $found = get_page_by_path( 'foo' );
+        $this->assertSame( $p1, $found->ID );
+
+        $num_queries = $wpdb->num_queries;
+
+        $found = get_page_by_path( 'foo', OBJECT, 'wptests_pt' );
+        $this->assertSame( $p2, $found->ID );
+        $num_queries++;
+        $this->assertSame( $num_queries, $wpdb->num_queries );
+    }
+
+    /**
+     * @ticket 36711
+     */
+    public function test_cache_should_be_invalidated_when_post_name_is_edited() {
+        global $wpdb;
+
+        $page = self::factory()->post->create( array(
+            'post_type' => 'page',
+            'post_name' => 'foo',
+        ) );
+
+        // Prime cache.
+        $found = get_page_by_path( 'foo' );
+        $this->assertSame( $page, $found->ID );
+
+        wp_update_post( array(
+            'ID' => $page,
+            'post_name' => 'bar',
+        ) );
+
+        $num_queries = $wpdb->num_queries;
+
+        $found = get_page_by_path( 'bar' );
+        $this->assertSame( $page, $found->ID );
+        $num_queries++;
+        $this->assertSame( $num_queries, $wpdb->num_queries );
+    }
 }