Merge pull request #20853 from woocommerce/reduce-test-suite-execution-time

Reduce WC_Tests_REST_System_Status tests execution time

tests/bootstrap.php

@@ -110,10 +110,10 @@ class WC_Unit_Tests_Bootstrap {
 		require_once $this->tests_dir . '/framework/vendor/class-wp-test-spy-rest-server.php';
 
 		// test cases
+		require_once $this->tests_dir . '/includes/wp-http-testcase.php';
 		require_once $this->tests_dir . '/framework/class-wc-unit-test-case.php';
 		require_once $this->tests_dir . '/framework/class-wc-api-unit-test-case.php';
 		require_once $this->tests_dir . '/framework/class-wc-rest-unit-test-case.php';
-		require_once $this->tests_dir . '/includes/wp-http-testcase.php';
 
 		// Helpers
 		require_once $this->tests_dir . '/framework/helpers/class-wc-helper-product.php';

tests/framework/class-wc-unit-test-case.php

@@ -13,7 +13,7 @@
  *
  * @since 2.2
  */
-class WC_Unit_Test_Case extends WP_UnitTestCase {
+class WC_Unit_Test_Case extends WP_HTTP_TestCase {
 
 	/**
 	 * Holds the WC_Unit_Test_Factory instance.

tests/unit-tests/api/system-status.php

@@ -24,6 +24,9 @@ class WC_Tests_REST_System_Status extends WC_REST_Unit_Test_Case {
 				'role' => 'administrator',
 			)
 		);
+
+		// Callback used by WP_HTTP_TestCase to decide whether to perform HTTP requests or to provide a mocked response.
+		$this->http_responder = array( $this, 'mock_http_responses' );
 	}
 
 	/**
@@ -376,4 +379,33 @@ class WC_Tests_REST_System_Status extends WC_REST_Unit_Test_Case {
 		$this->assertNotEmpty( get_post_meta( $order2->get_id(), '_shipping_address_index', true ) );
 
 	}
+
+	/**
+	 * Provides a mocked response for external requests performed by WC_REST_System_Status_Controller.
+	 * This way it is not necessary to perform a regular request to an external server which would
+	 * significantly slow down the tests.
+	 *
+	 * This function is called by WP_HTTP_TestCase::http_request_listner().
+	 *
+	 * @param array $request Request arguments.
+	 * @param string $url URL of the request.
+	 *
+	 * @return array|false mocked response or false to let WP perform a regular request.
+	 */
+	protected function mock_http_responses( $request, $url ) {
+		$mocked_response = false;
+
+		if ( in_array( $url, array( 'https://www.paypal.com/cgi-bin/webscr', 'https://woocommerce.com/wc-api/product-key-api?request=ping&network=0' ), true ) ) {
+			$mocked_response = array(
+				'response' => array( 'code' => 200 ),
+			);
+		} else if ( 'https://api.wordpress.org/themes/info/1.0/' === $url ) {
+			$mocked_response = array(
+				'body'     => 'O:8:"stdClass":12:{s:4:"name";s:7:"Default";s:4:"slug";s:7:"default";s:7:"version";s:5:"1.7.2";s:11:"preview_url";s:29:"https://wp-themes.com/default";s:6:"author";s:15:"wordpressdotorg";s:14:"screenshot_url";s:61:"//ts.w.org/wp-content/themes/default/screenshot.png?ver=1.7.2";s:6:"rating";d:100;s:11:"num_ratings";s:1:"3";s:10:"downloaded";i:296618;s:12:"last_updated";s:10:"2010-06-14";s:8:"homepage";s:37:"https://wordpress.org/themes/default/";s:13:"download_link";s:55:"https://downloads.wordpress.org/theme/default.1.7.2.zip";}',
+				'response' => array( 'code' => 200 ),
+			);
+		}
+
+		return $mocked_response;
+	}
 }

tests/unit-tests/formatting/functions.php

@@ -10,7 +10,7 @@
  *
  * @since 2.2
  */
-class WC_Tests_Formatting_Functions extends WP_HTTP_TestCase {
+class WC_Tests_Formatting_Functions extends WC_Unit_Test_Case {
 
 	public function setUp() {
 		parent::setUp();

tests/unit-tests/importer/product.php

@@ -4,7 +4,7 @@
  * Meta
  * @package WooCommerce\Tests\Importer
  */
-class WC_Tests_Product_CSV_Importer extends WP_HTTP_TestCase {
+class WC_Tests_Product_CSV_Importer extends WC_Unit_Test_Case {
 
 	/**
 	 * Test CSV file path.