Changeset 50087 for branches/4.8/tests/phpunit/includes/testcase.php

Timestamp:

01/30/2021 11:13:04 AM (4 years ago)

Author:

SergeyBiryukov

Message:

Tests: Use skipTestOnTimeout() in more HTTP tests.

Adjust it to handle more types of timeouts, e.g. "Resolving timed out", "Connection timed out".

Merges [43511], [43512], [46682], [46996] to the 4.8 branch.
See #51669.

Location:

branches/4.8

Files:

• . (modified) (1 prop)
• tests/phpunit/includes/testcase.php (modified) (2 diffs)

branches/4.8/tests/phpunit/includes/testcase.php

@@ -178 +178 @@
 
     /**
-     * Allow tests to be skipped on some automated runs
+     * Allow tests to be skipped on some automated runs.
      *
      * For test runs on Travis for something other than trunk/master 
@@ -218 +218 @@
             $this->markTestSkipped( 'Test does not run on Multisite' );
         }
+    }
+
+    /**
+     * Allow tests to be skipped if the HTTP request times out.
+     *
+     * @param array|WP_Error $response HTTP response.
+     */
+    public 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' );
+        }
+
     }