Divide tests into two files and improve covers

sirreal · Jan 2, 2024 · 0960481 · 0960481
1 parent 4fe7851
commit 0960481
Show file tree

Hide file tree

Showing 2 changed files with 42 additions and 23 deletions.
diff --git a/tests/phpunit/tests/modules.php → tests/phpunit/tests/modules/modules.php b/tests/phpunit/tests/modules.php → tests/phpunit/tests/modules/modules.php
@@ -4,8 +4,11 @@
  * @covers ::wp_register_module
  * @covers ::wp_enqueue_module
  * @covers ::wp_dequeue_module
+ * @covers WP_Modules::print_enqueued_modules
+ * @covers WP_Modules::print_import_map
+ * @covers WP_Modules::print_module_preloads
  */
-class Tests_Modules extends WP_UnitTestCase {
+class Tests_Modules_Functions extends WP_UnitTestCase {
 	/**
 	 * Stores a reference to the ReflectionProperty instance of the
 	 * WP_Modules::$registered property.
@@ -119,6 +122,7 @@ public function get_preloaded_modules() {
 	 *
 	 * @covers ::wp_register_module
 	 * @covers ::wp_enqueue_module
+	 * @covers WP_Modules::print_enqueued_modules
 	 */
 	public function test_wp_enqueue_module() {
 		wp_register_module( 'foo', '/foo.js' );
@@ -141,6 +145,7 @@ public function test_wp_enqueue_module() {
 	* @covers ::wp_register_module
 	* @covers ::wp_enqueue_module
 	* @covers ::wp_dequeue_module
+	 * @covers WP_Modules::print_enqueued_modules
 	*/
 	public function test_wp_dequeue_module() {
 		wp_register_module( 'foo', '/foo.js' );
@@ -164,6 +169,7 @@ public function test_wp_dequeue_module() {
 	*
 	* @covers ::wp_register_module
 	* @covers ::wp_enqueue_module
+	 * @covers WP_Modules::print_enqueued_modules
 	*/
 	public function test_wp_enqueue_module_works_before_register() {
 		wp_enqueue_module( 'foo' );
@@ -186,6 +192,7 @@ public function test_wp_enqueue_module_works_before_register() {
 	 * @covers ::wp_register_module
 	 * @covers ::wp_enqueue_module
 	 * @covers ::wp_dequeue_module
+	 * @covers WP_Modules::print_enqueued_modules
 	 */
 	public function test_wp_dequeue_module_works_before_register() {
 		wp_enqueue_module( 'foo' );
@@ -209,6 +216,7 @@ public function test_wp_dequeue_module_works_before_register() {
 	 *
 	 * @covers ::wp_register_module
 	 * @covers ::wp_enqueue_module
+	 * @covers WP_Modules::print_import_map
 	 */
 	public function test_wp_import_map_dependencies() {
 		wp_register_module( 'foo', '/foo.js', array( 'dep' ) );
@@ -231,6 +239,7 @@ public function test_wp_import_map_dependencies() {
 	 *
 	 * @covers ::wp_register_module
 	 * @covers ::wp_enqueue_module
+	 * @covers WP_Modules::print_import_map
 	 */
 	public function test_wp_import_map_no_duplicate_dependencies() {
 		wp_register_module( 'foo', '/foo.js', array( 'dep' ) );
@@ -253,6 +262,7 @@ public function test_wp_import_map_no_duplicate_dependencies() {
 	 *
 	 * @covers ::wp_register_module
 	 * @covers ::wp_enqueue_module
+	 * @covers WP_Modules::print_import_map
 	 */
 	public function test_wp_import_map_recursive_dependencies() {
 		wp_register_module(
@@ -303,6 +313,7 @@ public function test_wp_import_map_recursive_dependencies() {
 	 *
 	 * @covers ::wp_register_module
 	 * @covers ::wp_enqueue_module
+	 * @covers WP_Modules::print_module_preloads
 	 */
 	public function test_wp_enqueue_preloaded_static_dependencies() {
 		wp_register_module(
@@ -353,6 +364,7 @@ public function test_wp_enqueue_preloaded_static_dependencies() {
 	 *
 	 * @covers ::wp_register_module
 	 * @covers ::wp_enqueue_module
+	 * @covers WP_Modules::print_module_preloads
 	 */
 	public function test_wp_preloaded_dependencies_filter_enqueued_modules() {
 		wp_register_module(
@@ -383,6 +395,7 @@ public function test_wp_preloaded_dependencies_filter_enqueued_modules() {
 	 *
 	 * @covers ::wp_register_module
 	 * @covers ::wp_enqueue_module
+	 * @covers WP_Modules::print_import_map
 	 */
 	public function test_wp_enqueued_modules_with_dependants_add_import_map() {
 		wp_register_module(
@@ -404,26 +417,4 @@ public function test_wp_enqueued_modules_with_dependants_add_import_map() {
 		$this->assertEquals( true, isset( $import_map['dep'] ) );
 		$this->assertEquals( true, isset( $import_map['enqueued-dep'] ) );
 	}
-
-	/**
-	 * Tests the functionality of the `get_version_query_string` method to ensure
-	 * proper version strings are returned.
-	 *
-	 * @ticket 56313
-	 *
-	 * @covers WP_Modules::get_version_query_string
-	 */
-	public function test_get_version_query_string() {
-		$get_version_query_string = new ReflectionMethod( 'WP_Modules', 'get_version_query_string' );
-		$get_version_query_string->setAccessible( true );
-
-		$result = $get_version_query_string->invoke( null, '1.0' );
-		$this->assertEquals( '?ver=1.0', $result );
-
-		$result = $get_version_query_string->invoke( null, false );
-		$this->assertEquals( '?ver=' . get_bloginfo( 'version' ), $result );
-
-		$result = $get_version_query_string->invoke( null, null );
-		$this->assertEquals( '', $result );
-	}
 }
diff --git a/tests/phpunit/tests/modules/wpModules.php b/tests/phpunit/tests/modules/wpModules.php
@@ -0,0 +1,28 @@
+<?php
+/**
+ * @group modules
+ * @covers WP_Modules::get_version_query_string
+ */
+class Tests_WP_Modules extends WP_UnitTestCase {
+	/**
+	 * Tests the functionality of the `get_version_query_string` method to ensure
+	 * proper version strings are returned.
+	 *
+	 * @ticket 56313
+	 *
+	 * @covers WP_Modules::get_version_query_string
+	 */
+	public function test_get_version_query_string() {
+		$get_version_query_string = new ReflectionMethod( 'WP_Modules', 'get_version_query_string' );
+		$get_version_query_string->setAccessible( true );
+
+		$result = $get_version_query_string->invoke( null, '1.0' );
+		$this->assertEquals( '?ver=1.0', $result );
+
+		$result = $get_version_query_string->invoke( null, false );
+		$this->assertEquals( '?ver=' . get_bloginfo( 'version' ), $result );
+
+		$result = $get_version_query_string->invoke( null, null );
+		$this->assertEquals( '', $result );
+	}
+}