From 07e3b258b5f111f2d3d5d093f929bbd040ac19b9 Mon Sep 17 00:00:00 2001
From: Peter Wilson <peterwilsoncc@git.wordpress.org>
Date: Tue, 11 Mar 2025 00:39:10 +0000
Subject: [PATCH] Build/Test Tools: Update `external-http` group to include all
 network tests.

Updates the `external-http` group in the PHPUnit test suite to include all tests that rely on network requests. This is to ensure the main test suite runs do not contain any tests that can fail due to network conditions.

Props sukhendu2002, azaozz, audrasjb.
Fixes #62325.


git-svn-id: https://develop.svn.wordpress.org/trunk@59964 602fd350-edb4-49c9-b593-d223f7449a82
---
 tests/phpunit/tests/http/wpGetHttpHeaders.php                 | 4 ++++
 tests/phpunit/tests/multisite/updateBlogStatus.php            | 3 +++
 tests/phpunit/tests/rest-api/rest-site-health-controller.php  | 3 +++
 .../phpunit/tests/rest-api/wpRestBlockPatternsController.php  | 4 ++++
 4 files changed, 14 insertions(+)

diff --git a/tests/phpunit/tests/http/wpGetHttpHeaders.php b/tests/phpunit/tests/http/wpGetHttpHeaders.php
index aa5be431d8..855497674b 100644
--- a/tests/phpunit/tests/http/wpGetHttpHeaders.php
+++ b/tests/phpunit/tests/http/wpGetHttpHeaders.php
@@ -26,6 +26,8 @@ class Tests_HTTP_wpGetHttpHeaders extends WP_UnitTestCase {
 
 	/**
 	 * Test with an invalid URL
+	 *
+	 * @group external-http
 	 */
 	public function test_wp_get_http_headers_invalid_url() {
 		$result = wp_get_http_headers( 'not_an_url' );
@@ -34,6 +36,8 @@ class Tests_HTTP_wpGetHttpHeaders extends WP_UnitTestCase {
 
 	/**
 	 * Test to see if the deprecated argument is working
+	 *
+	 * @group external-http
 	 */
 	public function test_wp_get_http_headers_deprecated_argument() {
 		$this->setExpectedDeprecated( 'wp_get_http_headers' );
diff --git a/tests/phpunit/tests/multisite/updateBlogStatus.php b/tests/phpunit/tests/multisite/updateBlogStatus.php
index 092a76e8cb..02854ca219 100644
--- a/tests/phpunit/tests/multisite/updateBlogStatus.php
+++ b/tests/phpunit/tests/multisite/updateBlogStatus.php
@@ -45,6 +45,9 @@ if ( is_multisite() ) :
 			$this->assertSame( 1, $test_action_counter->get_call_count() );
 		}
 
+		/**
+		 * @group external-http
+		 */
 		public function test_content_from_spam_blog_is_not_available() {
 			$spam_blog_id = self::factory()->blog->create();
 			switch_to_blog( $spam_blog_id );
diff --git a/tests/phpunit/tests/rest-api/rest-site-health-controller.php b/tests/phpunit/tests/rest-api/rest-site-health-controller.php
index b4929914a6..9d67401d57 100644
--- a/tests/phpunit/tests/rest-api/rest-site-health-controller.php
+++ b/tests/phpunit/tests/rest-api/rest-site-health-controller.php
@@ -92,6 +92,9 @@ class WP_Test_REST_Site_Health_Controller extends WP_Test_REST_TestCase {
 		$this->assertErrorResponse( 'rest_forbidden', $response, 403 );
 	}
 
+	/**
+	 * @group external-http
+	 */
 	public function test() {
 		wp_set_current_user( self::$admin );
 		$response = rest_do_request( '/wp-site-health/v1/tests/dotorg-communication' );
diff --git a/tests/phpunit/tests/rest-api/wpRestBlockPatternsController.php b/tests/phpunit/tests/rest-api/wpRestBlockPatternsController.php
index 2dc3bb83a3..017a755a36 100644
--- a/tests/phpunit/tests/rest-api/wpRestBlockPatternsController.php
+++ b/tests/phpunit/tests/rest-api/wpRestBlockPatternsController.php
@@ -123,6 +123,9 @@ class Tests_REST_WpRestBlockPatternsController extends WP_Test_REST_Controller_T
 		$this->assertArrayHasKey( static::REQUEST_ROUTE, $routes );
 	}
 
+	/**
+	 * @group external-http
+	 */
 	public function test_get_items() {
 		wp_set_current_user( self::$admin_id );
 
@@ -189,6 +192,7 @@ class Tests_REST_WpRestBlockPatternsController extends WP_Test_REST_Controller_T
 	 * @since 6.2.0
 	 *
 	 * @ticket 57532
+	 * @group external-http
 	 *
 	 * @covers WP_REST_Block_Patterns_Controller::get_items
 	 */