Changeset 43982 for trunk/tests/phpunit/tests/post/meta.php

Timestamp:

12/12/2018 03:02:00 AM (6 years ago)

Author:

jeremyfelt

Message:

REST API: Move object type-specific metadata integrations from the wrapper functions to the low-level Meta API functions.

Object type-specific actions that should happen before or after modification of metadata have so far been part of the respective wrapper functions. By using action and filter hooks, this changeset ensures they are always executed, even when calling the lower-level Meta API functions directly, which the REST API does as a prime example.

Merges [43729] to trunk.

Props flixos90, spacedmonkey.
Fixes #44467.

Location:

trunk

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/post/meta.php (modified) (1 diff)

trunk/tests/phpunit/tests/post/meta.php

@@ -310 +310 @@
         );
     }
+
+    /**
+     * @ticket 44467
+     */
+    public function test_add_metadata_sets_posts_last_changed() {
+        $post_id = self::factory()->post->create();
+
+        wp_cache_delete( 'last_changed', 'posts' );
+
+        $this->assertInternalType( 'integer', add_metadata( 'post', $post_id, 'foo', 'bar' ) );
+        $this->assertNotFalse( wp_cache_get_last_changed( 'posts' ) );
+    }
+
+    /**
+     * @ticket 44467
+     */
+    public function test_update_metadata_sets_posts_last_changed() {
+        $post_id = self::factory()->post->create();
+
+        wp_cache_delete( 'last_changed', 'posts' );
+
+        $this->assertInternalType( 'integer', update_metadata( 'post', $post_id, 'foo', 'bar' ) );
+        $this->assertNotFalse( wp_cache_get_last_changed( 'posts' ) );
+    }
+
+    /**
+     * @ticket 44467
+     */
+    public function test_delete_metadata_sets_posts_last_changed() {
+        $post_id = self::factory()->post->create();
+
+        update_metadata( 'post', $post_id, 'foo', 'bar' );
+        wp_cache_delete( 'last_changed', 'posts' );
+
+        $this->assertTrue( delete_metadata( 'post', $post_id, 'foo' ) );
+        $this->assertNotFalse( wp_cache_get_last_changed( 'posts' ) );
+    }
 }