From 8dcb562c3977bcc8d3ea701c4a17d123c5eca2f9 Mon Sep 17 00:00:00 2001
From: Vladimir Reznichenko <kalessil@gmail.com>
Date: Mon, 9 Sep 2024 11:26:55 +0200
Subject: [PATCH] [dev] PSR-4 naming fixes in tests (part 2) (#51198)

In this PR we address PSR-4 naming-related violations (class FQN vs path mismatches and co) in tests.
---
 .../changelog/dev-psr4-naming-fixes-part-2    |  4 ++
 plugins/woocommerce/phpcs.xml                 |  5 ++
 .../DefaultShippingPartnersTest.php           |  3 +-
 .../src/Blocks/BlockTypes/BlockHooksTests.php |  2 +-
 .../{Migration.php => MigrationTest.php}      | 37 +++++++++-
 .../Blocks/Mocks/AssetDataRegistryMock.php    | 70 +++++++++++++++++++
 .../{mocks => Mocks}/BlockHooksTestBlock.php  |  0
 .../CartCheckoutUtilsMock.php                 |  8 ++-
 .../{mocks => Mocks}/MockTestDependency.php   | 16 ++++-
 .../OrderConfirmation/TotalsMock.php          | 21 ++++--
 .../ProductCollectionMock.php                 |  2 +
 .../{mocks => Mocks}/ProductQueryMock.php     |  2 +
 ...Controller.php => CartControllerTests.php} | 25 +++----
 ...ticeHandler.php => NoticeHandlerTests.php} |  4 +-
 ...ontroller.php => OrderControllerTests.php} |  4 +-
 ...ilters.php => ProductQueryFiltersTest.php} |  9 ++-
 .../Utilities/SanitizationUtilTest.php        |  3 +-
 .../src/Blocks/Utils/MiniCartUtilsTest.php    |  4 +-
 .../src/Blocks/mocks/AssetDataRegistry.php    | 30 --------
 .../MarketingRecommendations/InitTest.php     |  7 +-
 .../Orders/COTRedirectionControllerTest.php   |  5 +-
 .../BatchProcessingControllerTests.php        |  8 +--
 .../ComingSoonCacheInvalidatorTest.php        |  3 +-
 .../ComingSoon/ComingSoonHelperTest.php       |  3 +-
 .../ComingSoonRequestHandlerTest.php          |  4 +-
 .../src/Internal/Orders/IppFunctionsTest.php  |  1 +
 .../ApprovedDirectories/RegisterTest.php      |  4 +-
 .../ApprovedDirectories/SynchronizeTest.php   |  6 +-
 .../Utilities/{Types.php => TypesTest.php}    |  0
 29 files changed, 207 insertions(+), 83 deletions(-)
 create mode 100644 plugins/woocommerce/changelog/dev-psr4-naming-fixes-part-2
 rename plugins/woocommerce/tests/php/src/Blocks/{Migration.php => MigrationTest.php} (80%)
 create mode 100644 plugins/woocommerce/tests/php/src/Blocks/Mocks/AssetDataRegistryMock.php
 rename plugins/woocommerce/tests/php/src/Blocks/{mocks => Mocks}/BlockHooksTestBlock.php (100%)
 rename plugins/woocommerce/tests/php/src/Blocks/{mocks => Mocks}/CartCheckoutUtilsMock.php (59%)
 rename plugins/woocommerce/tests/php/src/Blocks/{mocks => Mocks}/MockTestDependency.php (52%)
 rename plugins/woocommerce/tests/php/src/Blocks/{mocks => Mocks}/OrderConfirmation/TotalsMock.php (73%)
 rename plugins/woocommerce/tests/php/src/Blocks/{mocks => Mocks}/ProductCollectionMock.php (97%)
 rename plugins/woocommerce/tests/php/src/Blocks/{mocks => Mocks}/ProductQueryMock.php (97%)
 rename plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/{CartController.php => CartControllerTests.php} (86%)
 rename plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/{NoticeHandler.php => NoticeHandlerTests.php} (97%)
 rename plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/{OrderController.php => OrderControllerTests.php} (98%)
 rename plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/{ProductQueryFilters.php => ProductQueryFiltersTest.php} (93%)
 delete mode 100644 plugins/woocommerce/tests/php/src/Blocks/mocks/AssetDataRegistry.php
 rename plugins/woocommerce/tests/php/src/Internal/Utilities/{Types.php => TypesTest.php} (100%)

diff --git a/plugins/woocommerce/changelog/dev-psr4-naming-fixes-part-2 b/plugins/woocommerce/changelog/dev-psr4-naming-fixes-part-2
new file mode 100644
index 00000000000..bd60605e513
--- /dev/null
+++ b/plugins/woocommerce/changelog/dev-psr4-naming-fixes-part-2
@@ -0,0 +1,4 @@
+Significance: patch
+Type: dev
+
+Coding standatds: fix PSR-4 naming violation in tests (part 2).
diff --git a/plugins/woocommerce/phpcs.xml b/plugins/woocommerce/phpcs.xml
index 3f0931e607c..fc1aae09313 100644
--- a/plugins/woocommerce/phpcs.xml
+++ b/plugins/woocommerce/phpcs.xml
@@ -55,11 +55,16 @@
 		<include-pattern>tests/php/src/</include-pattern>
 		<exclude-pattern>plugins/woocommerce/src/Admin/Notes/DeprecatedNotes.php</exclude-pattern>
 		<exclude-pattern>plugins/woocommerce/tests/php/src/Proxies/ExampleClasses/</exclude-pattern>
+		<exclude-pattern>plugins/woocommerce/tests/php/src/Internal/DependencyManagement/ExampleClasses/</exclude-pattern>
 		<properties>
 			<property name="composerJsonPath" value="composer.json"/>
 		</properties>
 	</rule>
 
+	<rule ref="Generic.CodeAnalysis.UselessOverridingMethod.Found">
+		<exclude-pattern>plugins/woocommerce/tests/php/src/Blocks/Mocks/</exclude-pattern>
+	</rule>
+
 	<rule ref="Generic.PHP.RequireStrictTypes">
 		<include-pattern>includes/</include-pattern>
 		<include-pattern>lib/</include-pattern>
diff --git a/plugins/woocommerce/tests/php/src/Admin/ShippingPartnerSuggestions/DefaultShippingPartnersTest.php b/plugins/woocommerce/tests/php/src/Admin/ShippingPartnerSuggestions/DefaultShippingPartnersTest.php
index 041488b0342..ecea313caa7 100644
--- a/plugins/woocommerce/tests/php/src/Admin/ShippingPartnerSuggestions/DefaultShippingPartnersTest.php
+++ b/plugins/woocommerce/tests/php/src/Admin/ShippingPartnerSuggestions/DefaultShippingPartnersTest.php
@@ -1,6 +1,7 @@
 <?php
+declare( strict_types = 1 );
 
-namespace Automattic\WooCommerce\Tests\Admin\Features\ShippingPartnerSuggestions;
+namespace Automattic\WooCommerce\Tests\Admin\ShippingPartnerSuggestions;
 
 use Automattic\WooCommerce\Admin\Features\PaymentGatewaySuggestions\EvaluateSuggestion;
 use Automattic\WooCommerce\Admin\Features\ShippingPartnerSuggestions\DefaultShippingPartners;
diff --git a/plugins/woocommerce/tests/php/src/Blocks/BlockTypes/BlockHooksTests.php b/plugins/woocommerce/tests/php/src/Blocks/BlockTypes/BlockHooksTests.php
index d2436006a8e..a26eb8ad52e 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/BlockTypes/BlockHooksTests.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/BlockTypes/BlockHooksTests.php
@@ -1,7 +1,7 @@
 <?php
 declare( strict_types = 1 );
 
-namespace Automattic\WooCommerce\Tests\Blocks\Templates;
+namespace Automattic\WooCommerce\Tests\Blocks\BlockTypes;
 
 use Automattic\WooCommerce\Tests\Blocks\Mocks\BlockHooksTestBlock;
 use WP_UnitTestCase;
diff --git a/plugins/woocommerce/tests/php/src/Blocks/Migration.php b/plugins/woocommerce/tests/php/src/Blocks/MigrationTest.php
similarity index 80%
rename from plugins/woocommerce/tests/php/src/Blocks/Migration.php
rename to plugins/woocommerce/tests/php/src/Blocks/MigrationTest.php
index 9e86463e5c6..3265242218f 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/Migration.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/MigrationTest.php
@@ -1,25 +1,47 @@
 <?php
+declare( strict_types = 1 );
 
-namespace Automattic\WooCommerce\Tests\Blocks\Utils;
+namespace Automattic\WooCommerce\Tests\Blocks;
 
 use Automattic\WooCommerce\Blocks\Migration;
 use Automattic\WooCommerce\Blocks\Options;
 use Mockery;
 
+/**
+ * Unit tests for the Migration class.
+ */
 class MigrationTest extends \WP_UnitTestCase {
 	use Mockery\Adapter\Phpunit\MockeryPHPUnitIntegration;
 
+	/**
+	 * Setup testing environment.
+	 *
+	 * @return void
+	 */
 	protected function setUp(): void {
 		parent::setUp();
 		delete_option( Options::WC_BLOCK_USE_BLOCKIFIED_PRODUCT_GRID_BLOCK_AS_TEMPLATE );
 		delete_option( Options::WC_BLOCK_VERSION );
 	}
 
+	/**
+	 * Cleanup testing environment.
+	 *
+	 * @return void
+	 */
 	protected function tearDown(): void {
 		\Mockery::close();
 		parent::tearDown();
 	}
 
+	/**
+	 * Inject test data into migration mock.
+	 *
+	 * @param Migration $mock The mock to patch.
+	 * @param array     $data The data to inject.
+	 *
+	 * @return Migration
+	 */
 	private function set_db_upgrades( Migration $mock, array $data ) {
 		$reflection = new \ReflectionClass( Migration::class );
 		$property   = $reflection->getProperty( 'db_upgrades' );
@@ -32,6 +54,9 @@ class MigrationTest extends \WP_UnitTestCase {
 		return $mock;
 	}
 
+	/**
+	 * Test running single migration.
+	 */
 	public function test_migrations_run() {
 		update_option( Options::WC_BLOCK_VERSION, '1.0.0' );
 
@@ -53,6 +78,9 @@ class MigrationTest extends \WP_UnitTestCase {
 		$mock->run_migrations();
 	}
 
+	/**
+	 * Test running multiple migrations.
+	 */
 	public function test_multiple_migrations_run() {
 		update_option( Options::WC_BLOCK_VERSION, '0.0.9' );
 
@@ -76,6 +104,9 @@ class MigrationTest extends \WP_UnitTestCase {
 		$mock->run_migrations();
 	}
 
+	/**
+	 * Test skipping executed migrations.
+	 */
 	public function test_skip_migrations() {
 		update_option( Options::WC_BLOCK_VERSION, '2.0.0' );
 
@@ -95,6 +126,9 @@ class MigrationTest extends \WP_UnitTestCase {
 		$mock->run_migrations();
 	}
 
+	/**
+	 * Test skipping executed migrations.
+	 */
 	public function test_skip_migrations_when_missing_version_option() {
 
 		$mock = Mockery::mock( Migration::class )->makePartial();
@@ -112,5 +146,4 @@ class MigrationTest extends \WP_UnitTestCase {
 
 		$mock->run_migrations();
 	}
-
 }
diff --git a/plugins/woocommerce/tests/php/src/Blocks/Mocks/AssetDataRegistryMock.php b/plugins/woocommerce/tests/php/src/Blocks/Mocks/AssetDataRegistryMock.php
new file mode 100644
index 00000000000..22f5ca430cd
--- /dev/null
+++ b/plugins/woocommerce/tests/php/src/Blocks/Mocks/AssetDataRegistryMock.php
@@ -0,0 +1,70 @@
+<?php
+declare( strict_types = 1 );
+
+namespace Automattic\WooCommerce\Tests\Blocks\Mocks;
+
+use Automattic\WooCommerce\Blocks\Assets\AssetDataRegistry;
+
+/**
+ * A mock class.
+ */
+class AssetDataRegistryMock extends AssetDataRegistry {
+	/**
+	 * Debug flag.
+	 *
+	 * @var bool
+	 */
+	private $debug = true;
+
+	/**
+	 * Sets the debug flag value.
+	 *
+	 * @param bool $debug The debug flag value to set.
+	 *
+	 * @return void
+	 */
+	public function set_debug( $debug ) {
+		$this->debug = $debug;
+	}
+
+	/**
+	 * Returns the debug flag value.
+	 *
+	 * @return bool
+	 */
+	protected function debug() {
+		return $this->debug;
+	}
+
+	/**
+	 * Exposes private registered data to child classes.
+	 *
+	 * @return array  The registered data on the private data property
+	 */
+	public function get() {
+		return parent::get();
+	}
+
+	/**
+	 * Used for on demand initialization of asset data and registering it with
+	 * the internal data registry.
+	 *
+	 * Note: core data will overwrite any externally registered data via the api.
+	 */
+	public function initialize_core_data() {
+		parent::initialize_core_data();
+	}
+
+	/**
+	 * Loops through each registered lazy data callback and adds the returned
+	 * value to the data array.
+	 *
+	 * This method is executed right before preparing the data for printing to
+	 * the rendered screen.
+	 *
+	 * @return void
+	 */
+	public function execute_lazy_data() {
+		parent::execute_lazy_data();
+	}
+}
diff --git a/plugins/woocommerce/tests/php/src/Blocks/mocks/BlockHooksTestBlock.php b/plugins/woocommerce/tests/php/src/Blocks/Mocks/BlockHooksTestBlock.php
similarity index 100%
rename from plugins/woocommerce/tests/php/src/Blocks/mocks/BlockHooksTestBlock.php
rename to plugins/woocommerce/tests/php/src/Blocks/Mocks/BlockHooksTestBlock.php
diff --git a/plugins/woocommerce/tests/php/src/Blocks/mocks/CartCheckoutUtilsMock.php b/plugins/woocommerce/tests/php/src/Blocks/Mocks/CartCheckoutUtilsMock.php
similarity index 59%
rename from plugins/woocommerce/tests/php/src/Blocks/mocks/CartCheckoutUtilsMock.php
rename to plugins/woocommerce/tests/php/src/Blocks/Mocks/CartCheckoutUtilsMock.php
index 1bb4627eca1..b8a42689683 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/mocks/CartCheckoutUtilsMock.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/Mocks/CartCheckoutUtilsMock.php
@@ -1,4 +1,6 @@
 <?php
+declare( strict_types = 1 );
+
 namespace Automattic\WooCommerce\Tests\Blocks\Mocks;
 
 use Automattic\WooCommerce\Blocks\Utils\CartCheckoutUtils;
@@ -10,9 +12,9 @@ class CartCheckoutUtilsMock extends CartCheckoutUtils {
 	/**
 	 * Protected test wrapper for deep_sort_with_accents.
 	 *
-	 * @param array $array The array we want to sort.
+	 * @param array $array_to_sort The array we want to sort.
 	 */
-	public static function deep_sort_test( $array ) {
-		return self::deep_sort_with_accents( $array );
+	public static function deep_sort_test( $array_to_sort ) {
+		return self::deep_sort_with_accents( $array_to_sort );
 	}
 }
diff --git a/plugins/woocommerce/tests/php/src/Blocks/mocks/MockTestDependency.php b/plugins/woocommerce/tests/php/src/Blocks/Mocks/MockTestDependency.php
similarity index 52%
rename from plugins/woocommerce/tests/php/src/Blocks/mocks/MockTestDependency.php
rename to plugins/woocommerce/tests/php/src/Blocks/Mocks/MockTestDependency.php
index 9cd6db14c3b..e90b8b1025e 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/mocks/MockTestDependency.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/Mocks/MockTestDependency.php
@@ -1,11 +1,25 @@
 <?php
+declare( strict_types = 1 );
 
 namespace Automattic\WooCommerce\Tests\Blocks\Mocks;
 
+/**
+ * A mock class.
+ */
 class MockTestDependency {
+	/**
+	 * Dependency.
+	 *
+	 * @var object
+	 */
 	public $dependency;
 
+	/**
+	 * Constructor.
+	 *
+	 * @param object $dependency Another dependency.
+	 */
 	public function __construct( $dependency = null ) {
 		$this->dependency = $dependency;
 	}
-};
+}
diff --git a/plugins/woocommerce/tests/php/src/Blocks/mocks/OrderConfirmation/TotalsMock.php b/plugins/woocommerce/tests/php/src/Blocks/Mocks/OrderConfirmation/TotalsMock.php
similarity index 73%
rename from plugins/woocommerce/tests/php/src/Blocks/mocks/OrderConfirmation/TotalsMock.php
rename to plugins/woocommerce/tests/php/src/Blocks/Mocks/OrderConfirmation/TotalsMock.php
index 611d6317149..4f983b8480f 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/mocks/OrderConfirmation/TotalsMock.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/Mocks/OrderConfirmation/TotalsMock.php
@@ -1,4 +1,6 @@
 <?php
+declare( strict_types = 1 );
+
 namespace Automattic\WooCommerce\Tests\Blocks\Mocks\OrderConfirmation;
 
 use Automattic\WooCommerce\Blocks\Package;
@@ -23,15 +25,24 @@ class TotalsMock extends Totals {
 		);
 	}
 
-	public function render_content( $order, $permission = false, $attributes = [], $content = '' ) {
-		return parent::render_content( $order, $permission, $attributes, $content );
-	}
-
 	/**
 	 * For now don't need to initialize anything in tests so let's
 	 * just override the default behaviour.
 	 */
 	protected function initialize() {
 	}
-}
 
+	/**
+	 * This renders the content of the block within the wrapper.
+	 *
+	 * @param \WC_Order    $order      Order object.
+	 * @param string|false $permission If the current user can view the order details or not.
+	 * @param array        $attributes Block attributes.
+	 * @param string       $content    Original block content.
+	 *
+	 * @return string
+	 */
+	public function render_content( $order, $permission = false, $attributes = [], $content = '' ) {
+		return parent::render_content( $order, $permission, $attributes, $content );
+	}
+}
diff --git a/plugins/woocommerce/tests/php/src/Blocks/mocks/ProductCollectionMock.php b/plugins/woocommerce/tests/php/src/Blocks/Mocks/ProductCollectionMock.php
similarity index 97%
rename from plugins/woocommerce/tests/php/src/Blocks/mocks/ProductCollectionMock.php
rename to plugins/woocommerce/tests/php/src/Blocks/Mocks/ProductCollectionMock.php
index b88a4a0c92e..59e9bae0b4f 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/mocks/ProductCollectionMock.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/Mocks/ProductCollectionMock.php
@@ -1,4 +1,6 @@
 <?php
+declare( strict_types = 1 );
+
 namespace Automattic\WooCommerce\Tests\Blocks\Mocks;
 
 use Automattic\WooCommerce\Blocks\BlockTypes\ProductCollection;
diff --git a/plugins/woocommerce/tests/php/src/Blocks/mocks/ProductQueryMock.php b/plugins/woocommerce/tests/php/src/Blocks/Mocks/ProductQueryMock.php
similarity index 97%
rename from plugins/woocommerce/tests/php/src/Blocks/mocks/ProductQueryMock.php
rename to plugins/woocommerce/tests/php/src/Blocks/Mocks/ProductQueryMock.php
index 40b01fa068f..4cdc34cfaca 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/mocks/ProductQueryMock.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/Mocks/ProductQueryMock.php
@@ -1,4 +1,6 @@
 <?php
+declare( strict_types = 1 );
+
 namespace Automattic\WooCommerce\Tests\Blocks\Mocks;
 
 use Automattic\WooCommerce\Blocks\BlockTypes\ProductQuery;
diff --git a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/CartController.php b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/CartControllerTests.php
similarity index 86%
rename from plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/CartController.php
rename to plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/CartControllerTests.php
index 325fe690948..327b9afa6d0 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/CartController.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/CartControllerTests.php
@@ -1,16 +1,19 @@
 <?php
-/**
- * CartController Tests.
- */
+declare( strict_types = 1 );
 
 namespace Automattic\WooCommerce\Tests\Blocks\StoreApi\Utilities;
 
-use Automattic\WooCommerce\Tests\Blocks\Helpers\FixtureData;
 use Automattic\WooCommerce\StoreApi\Utilities\CartController;
+use Automattic\WooCommerce\Tests\Blocks\Helpers\FixtureData;
 use Yoast\PHPUnitPolyfills\TestCases\TestCase;
 
+/**
+ * Unit tests for the CartController class.
+ */
 class CartControllerTests extends TestCase {
-
+	/**
+	 * Test cart error code is getting exposed.
+	 */
 	public function test_get_cart_errors() {
 		$class    = new CartController();
 		$fixtures = new FixtureData();
@@ -25,7 +28,7 @@ class CartControllerTests extends TestCase {
 		$out_of_stock_product_key = wc()->cart->add_to_cart( $out_of_stock_product->get_id(), 2 );
 		$out_of_stock_in_cart     = wc()->cart->get_cart_item( $out_of_stock_product_key )['data'];
 
-		// This product will have exact levels of stock known
+		// This product will have exact levels of stock known.
 		$partially_out_of_stock_product = $fixtures->get_simple_product(
 			array(
 				'name'          => 'Test Product 2',
@@ -35,7 +38,7 @@ class CartControllerTests extends TestCase {
 		$partially_out_of_stock_key     = wc()->cart->add_to_cart( $partially_out_of_stock_product->get_id(), 4 );
 		$partially_out_of_stock_in_cart = wc()->cart->get_cart_item( $partially_out_of_stock_key )['data'];
 
-		// This product will have exact levels of stock known
+		// This product will have exact levels of stock known.
 		$too_many_in_cart_product     = $fixtures->get_simple_product(
 			array(
 				'name'          => 'Test Product 3',
@@ -50,7 +53,7 @@ class CartControllerTests extends TestCase {
 		$partially_out_of_stock_in_cart->set_stock_quantity( 2 );
 		$too_many_in_cart_in_cart->set_sold_individually( true );
 
-		// This product will not be purchasable
+		// This product will not be purchasable.
 		$not_purchasable_product = $fixtures->get_simple_product(
 			array(
 				'name'          => 'Test Product 4',
@@ -59,10 +62,10 @@ class CartControllerTests extends TestCase {
 		);
 		wc()->cart->add_to_cart( $not_purchasable_product->get_id(), 2 );
 
-		// This function will force the $product->is_purchasable() function to return false for our $not_purchasable_product
+		// This function will force the $product->is_purchasable() function to return false for our $not_purchasable_product.
 		add_filter(
 			'woocommerce_is_purchasable',
-			function( $is_purchasable, $product ) use ( $not_purchasable_product ) {
+			function ( $is_purchasable, $product ) use ( $not_purchasable_product ) {
 				if ( $product->get_id() === $not_purchasable_product->get_id() ) {
 					return false;
 				}
@@ -88,7 +91,5 @@ class CartControllerTests extends TestCase {
 		foreach ( $expected_errors as $expected_error ) {
 			$this->assertContains( $expected_error, $error_codes );
 		}
-
 	}
-
 }
diff --git a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/NoticeHandler.php b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/NoticeHandlerTests.php
similarity index 97%
rename from plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/NoticeHandler.php
rename to plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/NoticeHandlerTests.php
index d0c5c6aa2ae..25ddafb19ba 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/NoticeHandler.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/NoticeHandlerTests.php
@@ -1,7 +1,5 @@
 <?php
-/**
- * NoticeHandler Tests.
- */
+declare( strict_types = 1 );
 
 namespace Automattic\WooCommerce\Tests\Blocks\StoreApi\Utilities;
 
diff --git a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/OrderController.php b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/OrderControllerTests.php
similarity index 98%
rename from plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/OrderController.php
rename to plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/OrderControllerTests.php
index b4dd0742531..76b797fc6b2 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/OrderController.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/OrderControllerTests.php
@@ -1,7 +1,5 @@
 <?php
-/**
- * OrderController Tests.
- */
+declare( strict_types = 1 );
 
 namespace Automattic\WooCommerce\Tests\Blocks\StoreApi\Utilities;
 
diff --git a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/ProductQueryFilters.php b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/ProductQueryFiltersTest.php
similarity index 93%
rename from plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/ProductQueryFilters.php
rename to plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/ProductQueryFiltersTest.php
index 86419256d5c..b6bc894e40c 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/ProductQueryFilters.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/ProductQueryFiltersTest.php
@@ -1,10 +1,15 @@
 <?php
+declare( strict_types = 1 );
+
 namespace Automattic\WooCommerce\Tests\Blocks\StoreApi\Utilities;
 
 use Automattic\WooCommerce\StoreApi\Utilities\ProductQueryFilters;
-use Yoast\PHPUnitPolyfills\TestCases\TestCase;
 use Automattic\WooCommerce\Tests\Blocks\Helpers\FixtureData;
+use Yoast\PHPUnitPolyfills\TestCases\TestCase;
 
+/**
+ * Unit tests for the ProductQueryFilters class.
+ */
 class ProductQueryFiltersTest extends TestCase {
 	/**
 	 * Setup test product data. Called before every test.
@@ -16,7 +21,7 @@ class ProductQueryFiltersTest extends TestCase {
 
 		add_filter(
 			'woocommerce_product_stock_status_options',
-			function( $status ) {
+			function ( $status ) {
 				$status['custom1'] = 'Custom status 1';
 				$status['custom2'] = 'Custom status 2';
 				return $status;
diff --git a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/SanitizationUtilTest.php b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/SanitizationUtilTest.php
index f4d6c09246e..d3f2aa26af9 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/SanitizationUtilTest.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/StoreApi/Utilities/SanitizationUtilTest.php
@@ -1,6 +1,7 @@
 <?php
+declare( strict_types = 1 );
 
-namespace Automattic\WooCommerce\Tests\Utilities;
+namespace Automattic\WooCommerce\Tests\Blocks\StoreApi\Utilities;
 
 use Automattic\WooCommerce\StoreApi\Utilities\SanitizationUtils;
 
diff --git a/plugins/woocommerce/tests/php/src/Blocks/Utils/MiniCartUtilsTest.php b/plugins/woocommerce/tests/php/src/Blocks/Utils/MiniCartUtilsTest.php
index 35ebe0b6662..039b1db3319 100644
--- a/plugins/woocommerce/tests/php/src/Blocks/Utils/MiniCartUtilsTest.php
+++ b/plugins/woocommerce/tests/php/src/Blocks/Utils/MiniCartUtilsTest.php
@@ -1,5 +1,7 @@
 <?php
-namespace Automattic\WooCommerce\Tests\Blocks\BlockTypes;
+declare( strict_types = 1 );
+
+namespace Automattic\WooCommerce\Tests\Blocks\Utils;
 
 use Automattic\WooCommerce\Blocks\Utils\MiniCartUtils;
 
diff --git a/plugins/woocommerce/tests/php/src/Blocks/mocks/AssetDataRegistry.php b/plugins/woocommerce/tests/php/src/Blocks/mocks/AssetDataRegistry.php
deleted file mode 100644
index 7a5c3ceb4ef..00000000000
--- a/plugins/woocommerce/tests/php/src/Blocks/mocks/AssetDataRegistry.php
+++ /dev/null
@@ -1,30 +0,0 @@
-<?php
-
-namespace Automattic\WooCommerce\Tests\Blocks\Mocks;
-
-use Automattic\WooCommerce\Blocks\Assets\AssetDataRegistry;
-
-class AssetDataRegistryMock extends AssetDataRegistry {
-
-	private $debug = true;
-
-	public function execute_lazy_data() {
-		parent::execute_lazy_data();
-	}
-
-	public function get() {
-		return parent::get();
-	}
-
-	public function set_debug( $debug ) {
-		$this->debug = $debug;
-	}
-
-	public function initialize_core_data() {
-		return parent::initialize_core_data();
-	}
-
-	protected function debug() {
-		return $this->debug;
-	}
-}
diff --git a/plugins/woocommerce/tests/php/src/Internal/Admin/MarketingRecommendations/InitTest.php b/plugins/woocommerce/tests/php/src/Internal/Admin/MarketingRecommendations/InitTest.php
index c4fb8a3f715..6ba09798c8f 100644
--- a/plugins/woocommerce/tests/php/src/Internal/Admin/MarketingRecommendations/InitTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/Admin/MarketingRecommendations/InitTest.php
@@ -1,11 +1,12 @@
 <?php
+declare( strict_types = 1 );
 
-namespace Automattic\WooCommerce\Tests\Internal\Admin\ShippingPartnerSuggestions;
+namespace Automattic\WooCommerce\Tests\Internal\Admin\MarketingRecommendations;
 
-use Automattic\WooCommerce\Admin\RemoteSpecs\DataSourcePoller;
-use Automattic\WooCommerce\Admin\Features\MarketingRecommendations\Init;
 use Automattic\WooCommerce\Admin\Features\MarketingRecommendations\DefaultMarketingRecommendations;
+use Automattic\WooCommerce\Admin\Features\MarketingRecommendations\Init;
 use Automattic\WooCommerce\Admin\Features\MarketingRecommendations\MarketingRecommendationsDataSourcePoller;
+use Automattic\WooCommerce\Admin\RemoteSpecs\DataSourcePoller;
 use WC_Unit_Test_Case;
 
 /**
diff --git a/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/COTRedirectionControllerTest.php b/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/COTRedirectionControllerTest.php
index b5e8d13bbf6..45068769cec 100644
--- a/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/COTRedirectionControllerTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/Admin/Orders/COTRedirectionControllerTest.php
@@ -1,11 +1,14 @@
 <?php
+declare( strict_types = 1 );
+
+namespace Automattic\WooCommerce\Tests\Internal\Admin\Orders;
 
 use Automattic\WooCommerce\Internal\Admin\Orders\COTRedirectionController;
 
 /**
  * Describes our redirection logic covering HPOS admin screens when Custom Order Tables are not authoritative.
  */
-class COTRedirectionControllerTest extends WC_Unit_Test_Case {
+class COTRedirectionControllerTest extends \WC_Unit_Test_Case {
 	/**
 	 * @var COTRedirectionController
 	 */
diff --git a/plugins/woocommerce/tests/php/src/Internal/BatchProcessing/BatchProcessingControllerTests.php b/plugins/woocommerce/tests/php/src/Internal/BatchProcessing/BatchProcessingControllerTests.php
index ce01c66706f..b95b70ddd03 100644
--- a/plugins/woocommerce/tests/php/src/Internal/BatchProcessing/BatchProcessingControllerTests.php
+++ b/plugins/woocommerce/tests/php/src/Internal/BatchProcessing/BatchProcessingControllerTests.php
@@ -1,7 +1,7 @@
 <?php
-/**
- * Tests for BatchProcessingController class.
- */
+declare( strict_types = 1 );
+
+namespace Automattic\WooCommerce\Tests\Internal\BatchProcessing;
 
 use Automattic\WooCommerce\Internal\BatchProcessing\BatchProcessingController;
 use Automattic\WooCommerce\Internal\BatchProcessing\BatchProcessorInterface;
@@ -10,7 +10,7 @@ use Automattic\WooCommerce\Internal\DataStores\Orders\DataSynchronizer;
 /**
  * Class BatchProcessingControllerTests.
  */
-class BatchProcessingControllerTests extends WC_Unit_Test_Case {
+class BatchProcessingControllerTests extends \WC_Unit_Test_Case {
 
 	/**
 	 * Instance of BatchProcessingController.
diff --git a/plugins/woocommerce/tests/php/src/Internal/ComingSoon/ComingSoonCacheInvalidatorTest.php b/plugins/woocommerce/tests/php/src/Internal/ComingSoon/ComingSoonCacheInvalidatorTest.php
index 1e9985a32f2..134156b92e5 100644
--- a/plugins/woocommerce/tests/php/src/Internal/ComingSoon/ComingSoonCacheInvalidatorTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/ComingSoon/ComingSoonCacheInvalidatorTest.php
@@ -1,6 +1,7 @@
 <?php
+declare( strict_types = 1 );
 
-namespace Automattic\WooCommerce\Tests\ComingSoon;
+namespace Automattic\WooCommerce\Tests\Internal\ComingSoon;
 
 use Automattic\WooCommerce\Internal\ComingSoon\ComingSoonCacheInvalidator;
 
diff --git a/plugins/woocommerce/tests/php/src/Internal/ComingSoon/ComingSoonHelperTest.php b/plugins/woocommerce/tests/php/src/Internal/ComingSoon/ComingSoonHelperTest.php
index 72928cbc6d1..3dfe39d1b02 100644
--- a/plugins/woocommerce/tests/php/src/Internal/ComingSoon/ComingSoonHelperTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/ComingSoon/ComingSoonHelperTest.php
@@ -1,6 +1,7 @@
 <?php
+declare( strict_types = 1 );
 
-namespace Automattic\WooCommerce\Tests\ComingSoon;
+namespace Automattic\WooCommerce\Tests\Internal\ComingSoon;
 
 use Automattic\WooCommerce\Internal\ComingSoon\ComingSoonHelper;
 
diff --git a/plugins/woocommerce/tests/php/src/Internal/ComingSoon/ComingSoonRequestHandlerTest.php b/plugins/woocommerce/tests/php/src/Internal/ComingSoon/ComingSoonRequestHandlerTest.php
index 6940a62e016..143e47a0305 100644
--- a/plugins/woocommerce/tests/php/src/Internal/ComingSoon/ComingSoonRequestHandlerTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/ComingSoon/ComingSoonRequestHandlerTest.php
@@ -1,9 +1,9 @@
 <?php
+declare( strict_types = 1 );
 
-namespace Automattic\WooCommerce\Tests\ComingSoon;
+namespace Automattic\WooCommerce\Tests\Internal\ComingSoon;
 
 use Automattic\WooCommerce\Internal\ComingSoon\ComingSoonRequestHandler;
-use Automattic\WooCommerce\Admin\Features\Features;
 
 /**
  * Tests for the coming soon cache invalidator class.
diff --git a/plugins/woocommerce/tests/php/src/Internal/Orders/IppFunctionsTest.php b/plugins/woocommerce/tests/php/src/Internal/Orders/IppFunctionsTest.php
index a566ba578a6..ab311dda7e7 100644
--- a/plugins/woocommerce/tests/php/src/Internal/Orders/IppFunctionsTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/Orders/IppFunctionsTest.php
@@ -1,4 +1,5 @@
 <?php
+declare( strict_types = 1 );
 
 namespace Automattic\WooCommerce\Tests\Internal\Orders;
 
diff --git a/plugins/woocommerce/tests/php/src/Internal/ProductDownloads/ApprovedDirectories/RegisterTest.php b/plugins/woocommerce/tests/php/src/Internal/ProductDownloads/ApprovedDirectories/RegisterTest.php
index de027258f2c..7f0d04cb138 100644
--- a/plugins/woocommerce/tests/php/src/Internal/ProductDownloads/ApprovedDirectories/RegisterTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/ProductDownloads/ApprovedDirectories/RegisterTest.php
@@ -1,9 +1,9 @@
 <?php
+declare( strict_types = 1 );
 
-namespace Automattic\WooCommerce\Tests\Internal\ProductDownloads;
+namespace Automattic\WooCommerce\Tests\Internal\ProductDownloads\ApprovedDirectories;
 
 use Automattic\WooCommerce\Internal\ProductDownloads\ApprovedDirectories\Register;
-
 use WC_Unit_Test_Case;
 
 /**
diff --git a/plugins/woocommerce/tests/php/src/Internal/ProductDownloads/ApprovedDirectories/SynchronizeTest.php b/plugins/woocommerce/tests/php/src/Internal/ProductDownloads/ApprovedDirectories/SynchronizeTest.php
index 5ffb93d3ddd..10122b920c6 100644
--- a/plugins/woocommerce/tests/php/src/Internal/ProductDownloads/ApprovedDirectories/SynchronizeTest.php
+++ b/plugins/woocommerce/tests/php/src/Internal/ProductDownloads/ApprovedDirectories/SynchronizeTest.php
@@ -1,12 +1,10 @@
 <?php
+declare( strict_types = 1 );
 
-namespace Automattic\WooCommerce\Tests\Internal\ProductDownloads;
+namespace Automattic\WooCommerce\Tests\Internal\ProductDownloads\ApprovedDirectories;
 
 use Automattic\WooCommerce\Internal\ProductDownloads\ApprovedDirectories\Synchronize;
-use Automattic\WooCommerce\Internal\ProductDownloads\ApprovedDirectories\Register;
 use Automattic\WooCommerce\Proxies\LegacyProxy;
-use Automattic\WooCommerce\RestApi\UnitTests\Helpers\ProductHelper;
-use Automattic\WooCommerce\Testing\Tools\FakeQueue;
 use WC_Queue_Interface;
 use WC_Unit_Test_Case;
 
diff --git a/plugins/woocommerce/tests/php/src/Internal/Utilities/Types.php b/plugins/woocommerce/tests/php/src/Internal/Utilities/TypesTest.php
similarity index 100%
rename from plugins/woocommerce/tests/php/src/Internal/Utilities/Types.php
rename to plugins/woocommerce/tests/php/src/Internal/Utilities/TypesTest.php