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

Timestamp:

08/29/2024 06:04:02 AM (7 months ago)

Author:

swissspidy

Message:

Script Loader: Remove importmap polyfill.

The polyfill was added in [57492], but all browsers supported by WordPress already support import maps.

This not only disables the polyfill, but completely removes it as it was only added recently and there is no usage outside of core.

Props swissspidy, desrosj, luisherranz, gziolo.
Fixes #60970.

File:

• trunk/tests/phpunit/tests/script-modules/wpScriptModules.php (modified) (1 diff)

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

@@ -699 +699 @@
         $this->assertCount( 1, $import_map );
         $this->assertStringStartsWith( '/dep.js', $import_map['dep'] );
-    }
-
-    /**
-     * @ticket 60348
-     *
-     * @covers ::print_import_map_polyfill()
-     */
-    public function test_wp_print_import_map_has_no_polyfill_when_no_modules_registered() {
-        $import_map_polyfill = get_echo( array( $this->script_modules, 'print_import_map' ) );
-
-        $this->assertSame( '', $import_map_polyfill );
-    }
-
-    /**
-     * @ticket 60348
-     *
-     * @covers ::print_import_map_polyfill()
-     */
-    public function test_wp_print_import_map_has_polyfill_when_modules_registered() {
-        $script_name = 'wp-polyfill-importmap';
-        wp_register_script( $script_name, '/wp-polyfill-importmap.js' );
-
-        $this->script_modules->enqueue( 'foo', '/foo.js', array( 'dep' ), '1.0' );
-        $this->script_modules->register( 'dep', '/dep.js' );
-        $import_map_polyfill = get_echo( array( $this->script_modules, 'print_import_map' ) );
-
-        wp_deregister_script( $script_name );
-
-        $p = new WP_HTML_Tag_Processor( $import_map_polyfill );
-        $p->next_tag( array( 'tag' => 'SCRIPT' ) );
-        $id = $p->get_attribute( 'id' );
-
-        $this->assertSame( 'wp-load-polyfill-importmap', $id );
     }