Changeset 61323 for trunk/tests/phpunit/tests/script-modules/wpScriptModules.php

Timestamp:

11/30/2025 12:54:26 AM (8 months ago)

Author:

westonruter

Message:

Script Loader: Emit notices when enqueueing a script, style, or script module with missing dependencies.

Developed in ​https://github.com/WordPress/wordpress-develop/pull/10545

Follow-up to [60999].

Props deepakprajapati, westonruter.
See #63486.
Fixes #64229.

File:

• trunk/tests/phpunit/tests/script-modules/wpScriptModules.php (modified) (2 diffs)

trunk/tests/phpunit/tests/script-modules/wpScriptModules.php

@@ -2060 +2060 @@
                 );
 
-                $deregister( array( 'b', 'c ' ) );
+                $deregister( array( 'b', 'c' ) );
 
                 // Test that registered dependency in footer doesn't place dependant in footer.
@@ -2313 +2313 @@
                 );
         }
+
+        /**
+         * Tests that a missing script module dependency triggers a _doing_it_wrong() notice.
+         *
+         * @ticket 64229
+         * @covers WP_Script_Modules::sort_item_dependencies
+         */
+        public function test_missing_script_module_dependency_triggers_incorrect_usage() {
+                $expected_incorrect_usage = 'WP_Script_Modules::register';
+                $this->setExpectedIncorrectUsage( $expected_incorrect_usage );
+
+                $this->script_modules->enqueue( 'main-module', '/main-module.js', array( 'missing-mod-dep' ) );
+
+                $markup = get_echo( array( $this->script_modules, 'print_enqueued_script_modules' ) );
+                $this->assertStringNotContainsString( 'main-module.js', $markup, 'Expected script module to be absent.' );
+
+                $this->assertArrayHasKey(
+                        $expected_incorrect_usage,
+                        $this->caught_doing_it_wrong,
+                        'Expected WP_Script_Modules::register to be reported via doing_it_wrong().'
+                );
+
+                // Assert the message mentions the missing dependency handle.
+                $this->assertStringContainsString(
+                        'The script module with the ID "main-module" was enqueued with dependencies that are not registered: missing-mod-dep',
+                        $this->caught_doing_it_wrong[ $expected_incorrect_usage ]
+                );
+        }
 }