Changeset 61526

Timestamp:

01/26/2026 01:21:56 PM (8 months ago)

Author:

jonsurrell

Message:

Build/Test Tools: Clean obsolete non-HTML5 tests.

Support for non-HTML5 scripts was removed in [61415]. Removes obsolete tests and references to the non-HTML5 script behaviors including CDATA wrappers and type attributes.

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

Follow-up to [61415].

Props jonsurrell, westonruter, mukesh27.
Fixes #64442.

Location:

trunk/tests/phpunit/tests/dependencies

Files:

• scripts.php (modified) (1 diff)
• wpLocalizeScript.php (modified) (1 diff)
• wpScriptTag.php (modified) (6 diffs)

trunk/tests/phpunit/tests/dependencies/scripts.php

@@ -1877 +1877 @@
 
                 $this->assertEqualHTML( $expected, $print_scripts, '<body>', 'Scripts are being incorrectly concatenated when a main script is registered as deferred after other blocking scripts are registered. Deferred scripts should not be part of the script concat loader query string. ' );
-        }
-
-        /**
-         * @ticket 42804
-         */
-        public function test_wp_enqueue_script_with_html5_support_does_not_contain_type_attribute() {
-                global $wp_version;
-
-                $GLOBALS['wp_scripts']                  = new WP_Scripts();
-                $GLOBALS['wp_scripts']->default_version = get_bloginfo( 'version' );
-
-                wp_enqueue_script( 'empty-deps-no-version', 'example.com' );
-
-                $expected = "<script src='http://example.com?ver={$wp_version}' id='empty-deps-no-version-js'></script>\n";
-
-                $this->assertEqualHTML( $expected, get_echo( 'wp_print_scripts' ) );
         }
 

trunk/tests/phpunit/tests/dependencies/wpLocalizeScript.php

@@ -47 +47 @@
          */
         public function test_wp_localize_script_outputs_safe_json() {
-                add_theme_support( 'html5', array( 'script' ) );
-
                 $path     = '/test.js';
                 $base_url = site_url( $path );

trunk/tests/phpunit/tests/dependencies/wpScriptTag.php

@@ -10 +10 @@
 
         public function get_script_tag_type_set() {
-                add_theme_support( 'html5', array( 'script' ) );
-
                 $this->assertEqualHTML(
                         '<script src="https://localhost/PATH/FILE.js" type="application/javascript" nomodule></script>' . "\n",
@@ -23 +21 @@
                         )
                 );
-
-                remove_theme_support( 'html5' );
-
-                $this->assertEqualHTML(
-                        '<script src="https://localhost/PATH/FILE.js" type="application/javascript" nomodule></script>' . "\n",
-                        wp_get_script_tag(
-                                array(
-                                        'src'      => 'https://localhost/PATH/FILE.js',
-                                        'type'     => 'application/javascript',
-                                        'async'    => false,
-                                        'nomodule' => true,
-                                )
-                        )
-                );
         }
 
@@ -43 +27 @@
          */
         public function test_get_script_tag_type_not_set() {
-                add_theme_support( 'html5', array( 'script' ) );
-
                 $this->assertEqualHTML(
                         '<script src="https://localhost/PATH/FILE.js" nomodule></script>' . "\n",
@@ -55 +37 @@
                         )
                 );
-
-                remove_theme_support( 'html5' );
         }
 
@@ -73 +53 @@
                 );
 
-                add_theme_support( 'html5', array( 'script' ) );
-
                 $attributes = array(
                         'src'      => 'https://localhost/PATH/FILE.js',
@@ -80 +58 @@
                         'nomodule' => true,
                 );
-
-                $this->assertEqualHTML(
-                        wp_get_script_tag( $attributes ),
-                        get_echo(
-                                'wp_print_script_tag',
-                                array( $attributes )
-                        )
-                );
-
-                remove_theme_support( 'html5' );
 
                 $this->assertEqualHTML(