Changeset 904 in tests for trunk/tests/http/http.php

Timestamp:

07/18/2012 07:01:41 PM (13 years ago)

Author:

nacin

Message:

Rename tests to conform to this standard: the class Tests_A_B_C.php should be found in Tests/A/B/C.php.

One class per file. Tests are now organized by general component, rather than by the core file they are found in.

Work in progress. Some classes need to be renamed, and some files still need to be moved (and split up).

Location:

trunk/tests/http

Files:

• . (added)
• http.php (copied) (copied from trunk/tests/test_http.php) (2 diffs)

trunk/tests/http/http.php

@@ -1 +1 @@
 <?php
-/**
- * Note, When running these tests, remember that some things are done differently
- * based on safe_mode. You can run the test in safe_mode like such:
- *
- *   phpunit -d safe_mode=on --group http
- *
- * You may also need `-d safe_mode_gid=1` to relax the safe_mode checks to allow
- * inclusion of PEAR.
- *
- * The WP_HTTP tests require a class-http.php file of r17550 or later.
- */
-
-/**
- * @group http
- */
-abstract class WP_HTTP_UnitTestCase extends WP_UnitTestCase {
-    // You can use your own version of data/WPHTTP-testcase-redirection-script.php here.
-    var $redirection_script = 'http://api.wordpress.org/core/tests/1.0/redirection.php';
-
-    function setUp() {
-
-        if ( is_callable( array('WP_HTTP', '_getTransport') ) ) {
-            $this->markTestSkipped('The WP_HTTP tests require a class-http.php file of r17550 or later.');
-            return;
-        }
-
-        $class = "WP_HTTP_" . $this->transport;
-        if ( !call_user_func( array($class, 'test') ) ) {
-            $this->markTestSkipped( sprintf('The transport %s is not supported on this system', $this->transport) );
-        }
-
-        // Disable all transports aside from this one.
-        foreach ( array( 'curl', 'streams', 'fsockopen' ) as $t ) {
-            remove_filter( "use_{$t}_transport", '__return_false' ); // Just strip them all
-            if ( $t != $this->transport )
-                add_filter( "use_{$t}_transport", '__return_false' ); // and add it back if need be..
-        }
-    }
-
-    function tearDown() {
-        foreach ( array( 'curl', 'streams', 'fsockopen' ) as $t ) {
-            remove_filter( "use_{$t}_transport", '__return_false' );
-        }
-    }
-
-    function test_redirect_on_301() {
-        // 5 : 5 & 301
-        $res = wp_remote_request($this->redirection_script . '?code=301&rt=' . 5, array('redirection' => 5) );
-        $this->assertFalse( is_wp_error($res) );
-        $this->assertEquals(200, (int)$res['response']['code'] );
-    }
-
-    function test_redirect_on_302() {
-        // 5 : 5 & 302
-        $res = wp_remote_request($this->redirection_script . '?code=302&rt=' . 5, array('redirection' => 5) );
-        $this->assertFalse( is_wp_error($res) );
-        $this->assertEquals(200, (int)$res['response']['code'] );
-    }
-
-    /**
-     * @ticket 16855
-     */
-    function test_redirect_on_301_no_redirect() {
-        // 5 > 0 & 301
-        $res = wp_remote_request($this->redirection_script . '?code=301&rt=' . 5, array('redirection' => 0) );
-        $this->assertFalse( is_wp_error($res) );
-        $this->assertEquals(301, (int)$res['response']['code'] );
-    }
-
-    /**
-     * @ticket 16855
-     */
-    function test_redirect_on_302_no_redirect() {
-        // 5 > 0 & 302
-        $res = wp_remote_request($this->redirection_script . '?code=302&rt=' . 5, array('redirection' => 0) );
-        $this->assertFalse( is_wp_error($res) );
-        $this->assertEquals(302, (int)$res['response']['code'] );
-    }
-
-    function test_redirections_equal() {
-        // 5 - 5
-        $res = wp_remote_request($this->redirection_script . '?rt=' . 5, array('redirection' => 5) );
-        $this->assertFalse( is_wp_error($res) );
-        $this->assertEquals(200, (int)$res['response']['code'] );
-    }
-
-    function test_no_head_redirections() {
-        // No redirections on HEAD request:
-        $res = wp_remote_request($this->redirection_script . '?code=302&rt=' . 1, array('method' => 'HEAD') );
-        $this->assertFalse( is_wp_error($res) );
-        $this->assertEquals( 302, (int)$res['response']['code'] );
-    }
-
-    /**
-     * @ticket 16855
-     */
-    function test_redirect_on_head() {
-        // Redirections on HEAD request when Requested
-        $res = wp_remote_request($this->redirection_script . '?rt=' . 5, array('redirection' => 5, 'method' => 'HEAD') );
-        $this->assertFalse( is_wp_error($res) );
-        $this->assertEquals( 200, (int)$res['response']['code'] );
-    }
-
-    function test_redirections_greater() {
-        // 10 > 5
-        $res = wp_remote_request($this->redirection_script . '?rt=' . 10, array('redirection' => 5) );
-        $this->assertTrue( is_wp_error($res), print_r($res, true) );
-    }
-
-    function test_redirections_greater_edgecase() {
-        // 6 > 5 (close edgecase)
-        $res = wp_remote_request($this->redirection_script . '?rt=' . 6, array('redirection' => 5) );
-        $this->assertTrue( is_wp_error($res) );
-    }
-
-    function test_redirections_less_edgecase() {
-        // 4 < 5 (close edgecase)
-        $res = wp_remote_request($this->redirection_script . '?rt=' . 4, array('redirection' => 5) );
-        $this->assertFalse( is_wp_error($res) );
-    }
-
-    /**
-     * @ticket 16855
-     */
-    function test_redirections_zero_redirections_specified() {
-        // 0 redirections asked for, Should return the document?
-        $res = wp_remote_request($this->redirection_script . '?code=302&rt=' . 5, array('redirection' => 0) );
-        $this->assertFalse( is_wp_error($res) );
-        $this->assertEquals( 302, (int)$res['response']['code'] );
-    }
-
-    /**
-     * 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() {
-        // 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() {
-        // Test that the headers sent are recieved by the server
-        $headers = array('test1' => 'test', 'test2' => 0, 'test3' => '');
-        $res = wp_remote_request( $this->redirection_script . '?header-check', array('headers' => $headers) );
-
-        $this->assertFalse( is_wp_error($res) );
-
-        $headers = array();
-        foreach ( explode("\n", $res['body']) as $key => $value ) {
-            if ( empty($value) )
-                continue;
-            $parts = explode(':', $value,2);
-            unset($heaers[$key]);
-            $headers[ $parts[0] ] = $parts[1];
-        }
-
-        $this->assertTrue( isset($headers['test1']) && 'test' == $headers['test1'] );
-        $this->assertTrue( isset($headers['test2']) && '0' === $headers['test2'] );
-        // cURL/HTTP Extension Note: Will never pass, cURL does not pass headers with an empty value.
-        // Should it be that empty headers with empty values are NOT sent?
-        //$this->assertTrue( isset($headers['test3']) && '' === $headers['test3'] );
-    }
-
-    function test_file_stream() {
-        $url = 'http://unit-tests.svn.wordpress.org/trunk/data/images/2004-07-22-DSC_0007.jpg'; // we'll test against a file in the unit test data
-        $size = 87348;
-        $res = wp_remote_request( $url, array( 'stream' => true, 'timeout' => 30 ) ); //Auto generate the filename.
-
-        $this->assertFalse( is_wp_error( $res ) );
-        $this->assertEquals( '', $res['body'] ); // The body should be empty.
-        $this->assertEquals( $size, $res['headers']['content-length'] ); // Check the headers are returned (and the size is the same..)
-        $this->assertEquals( $size, filesize($res['filename']) ); // Check that the file is written to disk correctly without any extra characters
-
-        unlink($res['filename']); // Remove the temporary file
-    }
-}
-
-// Stubs to test each transport
-/**
- * @group http
- */
-class WPHTTP_curl extends WP_HTTP_UnitTestCase {
-    var $transport = 'curl';
-}
-/**
- * @group http
- */
-class WPHTTP_streams extends WP_HTTP_UnitTestCase {
-    var $transport = 'streams';
-}
-/**
- * @group http
- */
-class WPHTTP_fsockopen extends WP_HTTP_UnitTestCase {
-    var $transport = 'fsockopen';
-}
-
 /**
  * Non-transport-specific WP_HTTP Tests
@@ -210 +5 @@
  * @group http
  */
-class WPHTTP extends WP_UnitTestCase {
+class Tests_HTTP_HTTP extends WP_UnitTestCase {
 
     /**