Changeset 41849 for trunk/tests/phpunit/tests/post/getPages.php

Timestamp:

10/12/2017 03:19:30 PM (8 years ago)

Author:

boonebgorges

Message:

Bump 'posts' query cache incrementor when modifying postmeta.

This ensures that the get_pages() query cache doesn't go stale when
postmeta is modified.

Props spacedmonkey.
Fixes #40669.

File:

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

trunk/tests/phpunit/tests/post/getPages.php

@@ -97 +97 @@
 
     /**
+     * @ticket 40669
+     */
+    public function test_cache_should_be_invalidated_by_add_post_meta() {
+        $posts = self::factory()->post->create_many( 2, array(
+            'post_type' => 'page',
+        ) );
+
+        add_post_meta( $posts[0], 'foo', 'bar' );
+
+        $cached = get_pages( array(
+            'meta_key' => 'foo',
+            'meta_value' => 'bar',
+        ) );
+
+        $cached_ids = wp_list_pluck( $cached, 'ID' );
+        $this->assertEqualSets( array( $posts[0] ), $cached_ids );
+
+        add_post_meta( $posts[1], 'foo', 'bar' );
+
+        $found = get_pages( array(
+            'meta_key' => 'foo',
+            'meta_value' => 'bar',
+        ) );
+
+        $found_ids = wp_list_pluck( $found, 'ID' );
+        $this->assertEqualSets( $posts, $found_ids );
+    }
+
+    /**
+     * @ticket 40669
+     */
+    public function test_cache_should_be_invalidated_by_update_post_meta() {
+        $posts = self::factory()->post->create_many( 2, array(
+            'post_type' => 'page',
+        ) );
+
+        add_post_meta( $posts[0], 'foo', 'bar' );
+        add_post_meta( $posts[1], 'foo', 'bar' );
+
+        $cached = get_pages( array(
+            'meta_key' => 'foo',
+            'meta_value' => 'bar',
+        ) );
+
+        $cached_ids = wp_list_pluck( $cached, 'ID' );
+        $this->assertEqualSets( $posts, $cached_ids );
+
+        update_post_meta( $posts[1], 'foo', 'baz' );
+
+        $found = get_pages( array(
+            'meta_key' => 'foo',
+            'meta_value' => 'bar',
+        ) );
+
+        $found_ids = wp_list_pluck( $found, 'ID' );
+        $this->assertEqualSets( array( $posts[0] ), $found_ids );
+    }
+
+    /**
+     * @ticket 40669
+     */
+    public function test_cache_should_be_invalidated_by_delete_post_meta() {
+        $posts = self::factory()->post->create_many( 2, array(
+            'post_type' => 'page',
+        ) );
+
+        add_post_meta( $posts[0], 'foo', 'bar' );
+        add_post_meta( $posts[1], 'foo', 'bar' );
+
+        $cached = get_pages( array(
+            'meta_key' => 'foo',
+            'meta_value' => 'bar',
+        ) );
+
+        $cached_ids = wp_list_pluck( $cached, 'ID' );
+        $this->assertEqualSets( $posts, $cached_ids );
+
+        delete_post_meta( $posts[1], 'foo' );
+
+        $found = get_pages( array(
+            'meta_key' => 'foo',
+            'meta_value' => 'bar',
+        ) );
+
+        $found_ids = wp_list_pluck( $found, 'ID' );
+        $this->assertEqualSets( array( $posts[0] ), $found_ids );
+    }
+
+    /**
+     * @ticket 40669
+     */
+    public function test_cache_should_be_invalidated_by_delete_post_meta_by_key() {
+        $posts = self::factory()->post->create_many( 2, array(
+            'post_type' => 'page',
+        ) );
+
+        add_post_meta( $posts[0], 'foo', 'bar' );
+        add_post_meta( $posts[1], 'foo', 'bar' );
+
+        $cached = get_pages( array(
+            'meta_key' => 'foo',
+            'meta_value' => 'bar',
+        ) );
+
+        $cached_ids = wp_list_pluck( $cached, 'ID' );
+        $this->assertEqualSets( $posts, $cached_ids );
+
+        delete_post_meta_by_key( 'foo' );
+
+        $found = get_pages( array(
+            'meta_key' => 'foo',
+            'meta_value' => 'bar',
+        ) );
+
+        $found_ids = wp_list_pluck( $found, 'ID' );
+        $this->assertEqualSets( array(), $found_ids );
+    }
+
+    /**
      * @ticket 20376
      */