Changeset 50091 for branches/4.4/tests/phpunit/includes/testcase.php

Timestamp:

01/30/2021 12:08:49 PM (6 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.4 branch.
See #51669.

Location:

branches/4.4

Files:

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

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

@@ -150 +150 @@
 
         /**
-         * 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 
@@ -205 +205 @@
                         _unregister_post_status( $post_status );
                 }
+        }
+
+        /**
+         * 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' );
+                }
+
         }