Changeset 894 in tests for trunk/tests/test_http.php

Timestamp:

07/09/2012 03:32:24 PM (12 years ago)

Author:

nacin

Message:

Convert knownWPBug and knownUTBug to @ticket annotations. fixes #102.

Allows for an extra description to be included with the annotation,
which will be passed to markTestSkipped().

File:

• trunk/tests/test_http.php (modified) (5 diffs)

trunk/tests/test_http.php

@@ -59 +59 @@
     }
 
+    /**
+     * @ticket 16855
+     */
     function test_redirect_on_301_no_redirect() {
-        $this->knownWPBug(16855);
         // 5 > 0 & 301
         $res = wp_remote_request($this->redirection_script . '?code=301&rt=' . 5, array('redirection' => 0) );
@@ -67 +69 @@
     }
 
+    /**
+     * @ticket 16855
+     */
     function test_redirect_on_302_no_redirect() {
-        $this->knownWPBug(16855);
         // 5 > 0 & 302
         $res = wp_remote_request($this->redirection_script . '?code=302&rt=' . 5, array('redirection' => 0) );
@@ -89 +93 @@
     }
 
+    /**
+     * @ticket 16855
+     */
     function test_redirect_on_head() {
-        $this->knownWPBug(16855);
         // Redirections on HEAD request when Requested
         $res = wp_remote_request($this->redirection_script . '?rt=' . 5, array('redirection' => 5, 'method' => 'HEAD') );
@@ -115 +121 @@
     }
 
+    /**
+     * @ticket 16855
+     */
     function test_redirections_zero_redirections_specified() {
-        $this->knownWPBug(16855);
         // 0 redirections asked for, Should return the document?
         $res = wp_remote_request($this->redirection_script . '?code=302&rt=' . 5, array('redirection' => 0) );
@@ -123 +131 @@
     }
 
+    /**
+     * Do not redirect on non 3xx status codes
+     *
+     * @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() {
-        //$this->knownWPBug(16889);
-        // Is this valid? Streams, cURL and therefor, PHP Extension (ie. all PHP Internal) follow redirects on all status codes, currently all of WP_HTTP follows this template.
-
-        // Do not redirect on non 3xx status codes
-        //$res = wp_remote_request( $this->redirection_script . '?200-location=true' ); // Prints PASS on initial load, FAIL if the client follows the specified redirection
-        //$this->assertEquals( 'PASS', $res['body']);
-    }
-
+        // Prints PASS on initial load, FAIL if the client follows the specified redirection
+        $res = wp_remote_request( $this->redirection_script . '?200-location=true' );
+        $this->assertEquals( 'PASS', $res['body']);
+    }
+
+    /**
+     * @ticket 11888
+     */
     function test_send_headers() {
-        $this->knownWPBug(11888);
         // Test that the headers sent are recieved by the server
         $headers = array('test1' => 'test', 'test2' => 0, 'test3' => '');