Changeset 61550 for branches/6.9/tests/phpunit/tests/dependencies/styles.php

Timestamp:

01/28/2026 09:28:06 PM (6 months ago)

Author:

jorbin

Message:

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

First Developed in ​https://github.com/WordPress/wordpress-develop/pull/10545. Backport developed in ​https://github.com/WordPress/wordpress-develop/pull/10789.

Follow-up to [60999].

Reviewed by jorbin, wildworks.
Merges [61323], [61357], and [61542].

Props deepakprajapati, westonruter, mukeshpanchal27, jorbin, wildworks.
See #63486.
Fixes #64229.

Location:

branches/6.9

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/dependencies/styles.php (modified) (1 diff)

branches/6.9/tests/phpunit/tests/dependencies/styles.php

@@ -816 +816 @@
                 $this->assertEqualHTML( $expected, $printed );
         }
+
+        /**
+         * Tests that WP_Styles emits a _doing_it_wrong() notice for missing dependencies.
+         *
+         * @ticket 64229
+         * @covers WP_Dependencies::all_deps
+         */
+        public function test_wp_style_doing_it_wrong_for_missing_dependencies() {
+                $expected_incorrect_usage = 'WP_Styles::add';
+                $this->setExpectedIncorrectUsage( $expected_incorrect_usage );
+
+                wp_enqueue_style(
+                        'main-style',
+                        '/main-style.css',
+                        array( 'missing-style-dep' )
+                );
+
+                $markup = get_echo( 'wp_print_styles' );
+                $this->assertStringNotContainsString( 'main-style.css', $markup, 'Expected style to be absent.' );
+
+                $this->assertArrayHasKey(
+                        $expected_incorrect_usage,
+                        $this->caught_doing_it_wrong,
+                        "Expected $expected_incorrect_usage to trigger a _doing_it_wrong() notice for missing dependency."
+                );
+
+                $this->assertStringContainsString(
+                        'The style with the handle "main-style" was enqueued with dependencies that are not registered: missing-style-dep',
+                        $this->caught_doing_it_wrong[ $expected_incorrect_usage ],
+                        'Expected _doing_it_wrong() notice to indicate missing dependencies for enqueued styles.'
+                );
+        }
 }