Changeset 41651 for trunk/tests/phpunit/tests/oembed/WpEmbed.php

Timestamp:

09/30/2017 01:14:34 AM (7 years ago)

Author:

westonruter

Message:

Embeds: Cache oEmbeds in an oembed_cache custom post type instead of postmeta when there is no global $post.

Add processing of embeds to rich Text widget.

Props swissspidy, westonruter, ocean90, johnbillion.
See #40854, #39994, #40935.
Fixes #34115.

File:

• trunk/tests/phpunit/tests/oembed/WpEmbed.php (modified) (3 diffs)

trunk/tests/phpunit/tests/oembed/WpEmbed.php

@@ -162 +162 @@
     }
 
-    public function test_shortcode_should_cache_data_in_post_meta_for_known_post() {
-        $GLOBALS['post'] = $this->factory()->post->create_and_get();
+    public function test_shortcode_should_get_cached_data_from_post_meta_for_known_post() {
+        global $post;
+
+        $post = $this->factory()->post->create_and_get();
         $url             = 'https://example.com/';
         $expected        = '<b>Embedded content</b>';
         $key_suffix      = md5( $url . serialize( wp_embed_defaults( $url ) ) );
         $cachekey        = '_oembed_' . $key_suffix;
-        $cachekey_time   = '_oembed_time_' . $key_suffix;
+
+        add_post_meta( $post->ID, $cachekey, $expected );
 
         add_filter( 'pre_oembed_result', array( $this, '_pre_oembed_result_callback' ) );
@@ -174 +177 @@
         remove_filter( 'pre_oembed_result', array( $this, '_pre_oembed_result_callback' ) );
 
+        $actual_2 = $this->wp_embed->shortcode( array(), $url );
+
+        $cached = get_post_meta( $post->ID, $cachekey, true );
+
+        // Cleanup.
+        unset( $post );
+
         $this->assertEquals( $expected, $actual );
-
-        $this->assertEquals( $expected, get_post_meta( $GLOBALS['post']->ID, $cachekey, true ) );
-        $this->assertNotEmpty( get_post_meta( $GLOBALS['post']->ID, $cachekey_time, true ) );
-
-        // Result should be cached.
-        $actual = $this->wp_embed->shortcode( array(), $url );
-        $this->assertEquals( $expected, $actual );
-    }
-
-    public function test_shortcode_should_cache_failure_in_post_meta_for_known_post() {
-        $GLOBALS['post'] = $this->factory()->post->create_and_get();
+        $this->assertEquals( $expected, $actual_2 );
+        $this->assertEquals( $expected, $cached );
+    }
+
+    public function test_shortcode_should_get_cached_failure_from_post_meta_for_known_post() {
+        global $post;
+
+        $post = $this->factory()->post->create_and_get();
         $url             = 'https://example.com/';
         $expected        = '<a href="' . esc_url( $url ) . '">' . esc_html( $url ) . '</a>';
@@ -192 +199 @@
         $cachekey_time   = '_oembed_time_' . $key_suffix;
 
+        add_post_meta( $post->ID, $cachekey, '{{unknown}}' );
+        add_post_meta( $post->ID, $cachekey_time, 0 );
+
         add_filter( 'pre_oembed_result', '__return_empty_string' );
         $actual = $this->wp_embed->shortcode( array(), $url );
         remove_filter( 'pre_oembed_result', '__return_empty_string' );
 
+        // Result should be cached.
+        $actual_2 = $this->wp_embed->shortcode( array(), $url );
+
+        $cached = get_post_meta( $post->ID, $cachekey, true );
+        $cached_time = get_post_meta( $post->ID, $cachekey_time, true );
+
+        // Cleanup.
+        unset( $post );
+
         $this->assertEquals( $expected, $actual );
-
-        $this->assertEquals( '{{unknown}}', get_post_meta( $GLOBALS['post']->ID, $cachekey, true ) );
-        $this->assertEmpty( get_post_meta( $GLOBALS['post']->ID, $cachekey_time, true ) );
+        $this->assertEquals( '{{unknown}}', $cached );
+        $this->assertEmpty( $cached_time );
+        $this->assertEquals( $expected, $actual_2 );
+    }
+
+    /**
+     * @ticket 34115
+     */
+    public function test_shortcode_should_cache_data_in_custom_post() {
+        $url        = 'https://example.com/';
+        $expected   = '<b>Embedded content</b>';
+        $key_suffix = md5( $url . serialize( wp_embed_defaults( $url ) ) );
+
+        add_filter( 'pre_oembed_result', array( $this, '_pre_oembed_result_callback' ) );
+        $actual = $this->wp_embed->shortcode( array(), $url );
+        remove_filter( 'pre_oembed_result', array( $this, '_pre_oembed_result_callback' ) );
+
+        $oembed_post_id = $this->wp_embed->find_oembed_post_id( $key_suffix );
+        $post_content = get_post( $oembed_post_id )->post_content;
 
         // Result should be cached.
-        $actual = $this->wp_embed->shortcode( array(), $url );
+        $actual_2 = $this->wp_embed->shortcode( array(), $url );
+
+        wp_delete_post( $oembed_post_id );
+
+        $this->assertNotNull( $oembed_post_id );
+        $this->assertEquals( $expected, $post_content );
         $this->assertEquals( $expected, $actual );
+        $this->assertEquals( $expected, $actual_2 );
+    }
+
+    /**
+     * @ticket 34115
+     */
+    public function test_shortcode_should_cache_failure_in_custom_post() {
+        $url        = 'https://example.com/';
+        $expected   = '<a href="' . esc_url( $url ) . '">' . esc_html( $url ) . '</a>';
+        $key_suffix = md5( $url . serialize( wp_embed_defaults( $url ) ) );
+
+        add_filter( 'pre_oembed_result', '__return_empty_string' );
+        $actual = $this->wp_embed->shortcode( array(), $url );
+        remove_filter( 'pre_oembed_result', '__return_empty_string' );
+
+        $oembed_post_id = $this->wp_embed->find_oembed_post_id( $key_suffix );
+        $post_content = get_post( $oembed_post_id )->post_content;
+
+        // Result should be cached.
+        $actual_2 = $this->wp_embed->shortcode( array(), $url );
+
+        wp_delete_post( $oembed_post_id );
+
+        $this->assertEquals( $expected, $actual );
+        $this->assertEquals( $expected, $actual_2 );
+        $this->assertNotNull( $oembed_post_id );
+        $this->assertEquals( '{{unknown}}', $post_content );
     }