Changeset 38684 for trunk/tests/phpunit/tests/oembed.php

Timestamp:

09/30/2016 11:24:08 AM (8 years ago)

Author:

SergeyBiryukov

Message:

Embeds: Escape periods in oEmbed regex.

Props jrf.
Fixes #38187.

File:

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

trunk/tests/phpunit/tests/oembed.php

@@ -9 +9 @@
         'vimeo'                => '#https?://(.+\.)?vimeo\.com/.*#i',
         'dailymotion'          => '#https?://(www\.)?dailymotion\.com/.*#i',
-        'dailymotion-shorturl' => '#https?://dai.ly/.*#i',
+        'dailymotion-shorturl' => '#https?://dai\.ly/.*#i',
         'flickr'               => '#https?://(www\.)?flickr\.com/.*#i',
         'flickr-shorturl'      => '#https?://flic\.kr/.*#i',
@@ -17 +17 @@
         'photobucket-group'    => 'http://gi*.photobucket.com/groups/*',
         'scribd'               => '#https?://(www\.)?scribd\.com/doc/.*#i',
-        'wordpress-tv'         => '#https?://wordpress.tv/.*#i',
+        'wordpress-tv'         => '#https?://wordpress\.tv/.*#i',
         'polldaddy'            => '#https?://(.+\.)?polldaddy\.com/.*#i',
         'polldaddy-shorturl'   => '#https?://poll\.fm/.*#i',
@@ -24 +24 @@
         'twitter-timeline'     => '#https?://(www\.)?twitter\.com/.+?/timelines/.*#i',
         'twitter-moment'       => '#https?://(www\.)?twitter\.com/i/moments/.*#i',
-        'vine'                 => '#https?://vine.co/v/.*#i',
+        'vine'                 => '#https?://vine\.co/v/.*#i',
         'soundcloud'           => '#https?://(www\.)?soundcloud\.com/.*#i',
         'slideshare'           => '#https?://(.+?\.)?slideshare\.net/.*#i',
@@ -41 +41 @@
         'cloudup'              => '#https?://cloudup\.com/.*#i',
         'reverbnation'         => '#https?://(www\.)?reverbnation\.com/.*#i',
-        'videopress'           => '#https?://videopress.com/v/.*#',
+        'videopress'           => '#https?://videopress\.com/v/.*#',
         'reddit-comments'      => '#https?://(www\.)?reddit\.com/r/[^/]+/comments/.*#i',
         'speakerdeck'          => '#https?://(www\.)?speakerdeck\.com/.*#i',
@@ -813 +813 @@
     }
 
+    /**
+     * Test URLs that should not match any of the predefined providers.
+     *
+     * @group oembed
+     * @ticket 38187
+     *
+     * @dataProvider dataShouldNotMatchOembedRegex
+     */
+    public function testUrlShouldNotMatchOembedRegex( $url ) {
+
+        $this->assertSame( false, self::$oembed->get_provider( $url, array( 'discover' => false ) ) );
+
+    }
+
+    /**
+     * Data provider for testUrlShouldNotMatchOembedRegex().
+     *
+     * @return array
+     */
+    public function dataShouldNotMatchOembedRegex() {
+        $providers = self::$provider_map;
+
+        return array(
+            array( 'http://dairly/something' ),
+            array( 'https://daisly/' ),
+            array( 'http://wordpressstv/' ),
+            array( 'https://wordpressstv/somethingelse' ),
+            array( 'http://vinerco/v/andanother' ),
+            array( 'https://vineqco/v/' ),
+            array( 'http://videopressscom/v/' ),
+            array( 'https://videopresstcom/v/covered' ),
+        );
+    }
 }