Changeset 50102 for branches/4.2/tests/phpunit/includes/testcase.php

Timestamp:

01/30/2021 08:46:48 PM (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 [38757], [43511], [43512], [46682], [46996] to the 4.2 branch.
See #51669.

Location:

branches/4.2

Files:

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

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

@@ -110 +110 @@
             $this->markTestSkipped( 'For automated test runs, this test is only run on trunk/master' );
         }
+    }
+
+    /**
+     * 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' );
+        }
+
     }