Changeset 59729 for trunk/tests/phpunit/tests/readme.php

Timestamp:

01/29/2025 06:17:34 PM (3 months ago)

Author:

johnbillion

Message:

Build/Test Tools: Add a retry mechanism for tests that perform external HTTP requests.

While the skipTestOnTimeout() method will catch a timeout and prevent it from causing a test to fail, other errors such as a failed DNS lookup or HTTPS handshake can still cause a test to unnecessarily fail. This introduces a simple retry mechanism that will hopefully further reduce the flakiness of tests that perform HTTP API requests.

Fixes #62830

File:

• trunk/tests/phpunit/tests/readme.php (modified) (1 diff)

trunk/tests/phpunit/tests/readme.php

@@ -92 +92 @@
      */
     public function get_response_body( $url ) {
-        $response = wp_remote_get( $url );
+        $response = $this->wp_remote_get( $url );
 
-        $this->skipTestOnTimeout( $response );
         $this->assertNotWPError( $response );