Changeset 38840

Timestamp:

10/20/2016 09:15:10 AM (8 years ago)

Author:

johnbillion

Message:

oEmbed: Remove the oEmbed provider unit tests.

This reverts [38454] along with its follow-up commits, [38512], [38514], and [38692]. These tests are currently not pass
ing, and maybe they never will. The tests are in a group which does not run by default without a flag, making them quest
ionably useful.

We can re-visit this at a later date.

See #32360

Location:

trunk

Files:

• Gruntfile.js (modified) (1 diff)
• phpunit.xml.dist (modified) (1 diff)
• tests/phpunit/includes/bootstrap.php (modified) (1 diff)
• tests/phpunit/multisite.xml (modified) (1 diff)
• tests/phpunit/tests/oembed.php (deleted)

trunk/Gruntfile.js

@@ -441 +441 @@
                 args: ['-c', 'phpunit.xml.dist', '--group', 'external-http']
             }
-            //,
-            //'external-oembed': {
-            //  cmd: 'phpunit',
-            //  args: ['-c', 'phpunit.xml.dist', '--group', 'external-oembed']
-            //}
         },
         uglify: {

trunk/phpunit.xml.dist

@@ -25 +25 @@
             <group>ms-files</group>
             <group>external-http</group>
-            <group>external-oembed</group>
         </exclude>
     </groups>

trunk/tests/phpunit/includes/bootstrap.php

@@ -138 +138 @@
             'ms-files' => true,
             'external-http' => true,
-            'external-oembed' => true,
         );
 

trunk/tests/phpunit/multisite.xml

@@ -26 +26 @@
             <group>ms-files</group>
             <group>external-http</group>
-            <group>external-oembed</group>
             <group>oembed-headers</group>
         </exclude>