Changeset 43512

Timestamp:

07/19/2018 07:52:41 PM (8 years ago)

Author:

SergeyBiryukov

Message:

Tests: Introduce Tests_HTTP_Functions::skipTestOnTimeout(), mirroring the same WP_HTTP_UnitTestCase method.

See #44613.

Location:

trunk/tests/phpunit/tests/http

Files:

• base.php (modified) (1 diff)
• functions.php (modified) (9 diffs)

trunk/tests/phpunit/tests/http/base.php

@@ -19 +19 @@
 
         /**
-         * Mark test as skipped if the HTTP request times out
+         * Mark test as skipped if the HTTP request times out.
          */
         function skipTestOnTimeout( $response ) {

trunk/tests/phpunit/tests/http/functions.php

@@ -6 +6 @@
  */
 class Tests_HTTP_Functions extends WP_UnitTestCase {
+
+        /**
+         * Mark test as skipped if the HTTP request times out.
+         */
+        function skipTestOnTimeout( $response ) {
+                if ( ! is_wp_error( $response ) ) {
+                        return;
+                }
+                if ( 'connect() timed out!' === $response->get_error_message() ) {
+                        $this->markTestSkipped( 'HTTP timeout' );
+                }
+
+                if ( false !== strpos( $response->get_error_message(), 'timed out after' ) ) {
+                        $this->markTestSkipped( 'HTTP timeout' );
+                }
+
+                if ( 0 === strpos( $response->get_error_message(), 'stream_socket_client(): unable to connect to tcp://s.w.org:80' ) ) {
+                        $this->markTestSkipped( 'HTTP timeout' );
+                }
+
+        }
+
         public function setUp() {
                 if ( ! extension_loaded( 'openssl' ) ) {
@@ -18 +40 @@
                 $url      = 'https://asdftestblog1.files.wordpress.com/2007/09/2007-06-30-dsc_4700-1.jpg';
                 $response = wp_remote_head( $url );
-                $headers  = wp_remote_retrieve_headers( $response );
+
+                $this->skipTestOnTimeout( $response );
+
+                $headers = wp_remote_retrieve_headers( $response );
 
                 $this->assertInternalType( 'array', $response );
@@ -31 +56 @@
                 $url      = 'https://asdftestblog1.wordpress.com/files/2007/09/2007-06-30-dsc_4700-1.jpg';
                 $response = wp_remote_head( $url );
+
+                $this->skipTestOnTimeout( $response );
                 $this->assertEquals( '301', wp_remote_retrieve_response_code( $response ) );
         }
 
         function test_head_404() {
-                $url     = 'https://asdftestblog1.files.wordpress.com/2007/09/awefasdfawef.jpg';
-                $headers = wp_remote_head( $url );
-
-                $this->assertEquals( '404', wp_remote_retrieve_response_code( $headers ) );
+                $url      = 'https://asdftestblog1.files.wordpress.com/2007/09/awefasdfawef.jpg';
+                $response = wp_remote_head( $url );
+
+                $this->skipTestOnTimeout( $response );
+                $this->assertEquals( '404', wp_remote_retrieve_response_code( $response ) );
         }
 
@@ -45 +73 @@
 
                 $response = wp_remote_get( $url );
-                $headers  = wp_remote_retrieve_headers( $response );
+
+                $this->skipTestOnTimeout( $response );
+
+                $headers = wp_remote_retrieve_headers( $response );
 
                 $this->assertInternalType( 'array', $response );
@@ -60 +91 @@
 
                 $response = wp_remote_get( $url );
-                $headers  = wp_remote_retrieve_headers( $response );
+
+                $this->skipTestOnTimeout( $response );
+
+                $headers = wp_remote_retrieve_headers( $response );
 
                 // should return the same headers as a head request
@@ -74 +108 @@
                 // pretend we've already redirected 5 times
                 $response = wp_remote_get( $url, array( 'redirection' => -1 ) );
+
+                $this->skipTestOnTimeout( $response );
                 $this->assertWPError( $response );
         }
@@ -84 +120 @@
 
                 $response = wp_remote_head( $url );
-                $cookies  = wp_remote_retrieve_cookies( $response );
+
+                $this->skipTestOnTimeout( $response );
+
+                $cookies = wp_remote_retrieve_cookies( $response );
 
                 $this->assertNotEmpty( $cookies );
@@ -121 +160 @@
                         )
                 );
-                $cookies  = wp_remote_retrieve_cookies( $response );
+
+                $this->skipTestOnTimeout( $response );
+
+                $cookies = wp_remote_retrieve_cookies( $response );
 
                 $this->assertNotEmpty( $cookies );
@@ -144 +186 @@
                         )
                 );
-                $cookies  = wp_remote_retrieve_cookies( $response );
+
+                $this->skipTestOnTimeout( $response );
+
+                $cookies = wp_remote_retrieve_cookies( $response );
 
                 $this->assertNotEmpty( $cookies );