Changeset 1231 in tests

Timestamp:

03/04/2013 06:19:32 AM (12 years ago)

Author:

dd32

Message:

Rename the WP_HTTP testcase 'test_location_header_on_200' to 'test_location_header_on_201' and modify test / test script accordingly, PHP doesn't allow us to issue a Location with a 200 response, and the ticket it refers to is specifically about 201 responses.
See #16889

Location:

trunk

Files:

• data/WPHTTP-testcase-redirection-script.php (modified) (1 diff)
• tests/http/base.php (modified) (1 diff)

trunk/data/WPHTTP-testcase-redirection-script.php

@@ -24 +24 @@
     exit;
 }
-if ( isset($_GET['200-location']) ) {
-    header("HTTP/1.1 200 OK");
+if ( isset($_GET['201-location']) ) {
+    header("HTTP/1.1 201 OK");
     if ( isset($_GET['fail']) ) {
         echo "FAIL";
     } else {
-        header("Location: $url?200-location&fail=true", true, 200);
+        header("Location: $url?201-location&fail=true", true, 201);
         echo "PASS";
     }

trunk/tests/http/base.php

@@ -132 +132 @@
      *
      * @ticket 16889
-     *
-     * Is this valid? Streams, cURL and therefore PHP Extension (ie. all PHP Internal) follow redirects
-     * on all status codes, currently all of WP_HTTP follows this template.
      */
-    function test_location_header_on_200() {
+    function test_location_header_on_201() {
         // Prints PASS on initial load, FAIL if the client follows the specified redirection
-        $res = wp_remote_request( $this->redirection_script . '?200-location=true' );
+        $res = wp_remote_request( $this->redirection_script . '?201-location=true' );
         $this->assertFalse( is_wp_error( $res ) );
         $this->assertEquals( 'PASS', $res['body']);