Changeset 40104 for trunk

Timestamp:

02/23/2017 03:49:44 PM (8 years ago)

Author:

jnylen0

Message:

REST API: Ensure that tests pass if extra endpoints are registered.

Many plugins and themes use the WP core test suite to run their unit tests, so the API tests shouldn't fail if there are extra endpoints registered in non-core namespaces.

Props rachelbaker.
Fixes #39264.

File:

• trunk/tests/phpunit/tests/rest-api/rest-schema-setup.php (modified) (2 diffs)

trunk/tests/phpunit/tests/rest-api/rest-schema-setup.php

@@ -37 +37 @@
         $this->assertTrue( is_array( $routes ), '`get_routes` should return an array.' );
         $this->assertTrue( ! empty( $routes ), 'Routes should not be empty.' );
+
+        $routes = array_filter( array_keys( $routes ), array( $this, 'is_builtin_route' ) );
 
         $expected_routes = array(
@@ -71 +73 @@
         );
 
-        $this->assertEquals( $expected_routes, array_keys( $routes ) );
+        $this->assertEquals( $expected_routes, $routes );
+    }
+
+    private function is_builtin_route( $route ) {
+        return (
+            '/' === $route ||
+            preg_match( '#^/oembed/1\.0(/.+)?$#', $route ) ||
+            preg_match( '#^/wp/v2(/.+)?$#', $route )
+        );
     }