Changeset 57559 for trunk/tests/phpunit/tests/blocks/register.php

Timestamp:

02/08/2024 08:16:59 AM (3 years ago)

Author:

gziolo

Message:

Editor: Make asset file optional for block scripts

It is no longer a hard requirement that a *.asset.php file is present to register a script for block.

Fixes #57234.
Props joefusco, gziolo, spacedmonkey, colorful-tones.

File:

• trunk/tests/phpunit/tests/blocks/register.php (modified) (3 diffs)

trunk/tests/phpunit/tests/blocks/register.php

@@ -61 +61 @@
                 }
 
+                foreach ( wp_scripts()->registered as $script_handle => $script ) {
+                        if ( str_starts_with( $script_handle, 'unit-tests-' ) ) {
+                                wp_deregister_script( $script_handle );
+                        }
+                }
+
                 parent::tear_down();
         }
@@ -227 +233 @@
 
         /**
-         * @expectedIncorrectUsage register_block_script_handle
-         * @ticket 50263
-         */
-        public function test_missing_asset_file_register_block_script_handle() {
+         * @ticket 50263
+         */
+        public function test_handle_passed_register_block_script_handle() {
+                $metadata = array(
+                        'script' => 'test-script-handle',
+                );
+                $result   = register_block_script_handle( $metadata, 'script' );
+
+                $this->assertSame( 'test-script-handle', $result );
+        }
+
+        public function test_handles_passed_register_block_script_handles() {
+                $metadata = array(
+                        'script' => array( 'test-script-handle', 'test-script-handle-2' ),
+                );
+
+                $result = register_block_script_handle( $metadata, 'script' );
+                $this->assertSame( 'test-script-handle', $result );
+
+                $result = register_block_script_handle( $metadata, 'script', 1 );
+                $this->assertSame( 'test-script-handle-2', $result, 1 );
+        }
+
+        /**
+         * @ticket 50263
+         * @ticket 60460
+         */
+        public function test_missing_asset_file_register_block_script_handle_with_default_settings() {
                 $metadata = array(
                         'file'   => __FILE__,
@@ -238 +268 @@
                 $result   = register_block_script_handle( $metadata, 'script' );
 
-                $this->assertFalse( $result );
-        }
-
-        /**
-         * @ticket 50263
-         */
-        public function test_handle_passed_register_block_script_handle() {
-                $metadata = array(
-                        'script' => 'test-script-handle',
-                );
-                $result   = register_block_script_handle( $metadata, 'script' );
-
-                $this->assertSame( 'test-script-handle', $result );
-        }
-
-        public function test_handles_passed_register_block_script_handles() {
-                $metadata = array(
-                        'script' => array( 'test-script-handle', 'test-script-handle-2' ),
-                );
-
-                $result = register_block_script_handle( $metadata, 'script' );
-                $this->assertSame( 'test-script-handle', $result );
-
-                $result = register_block_script_handle( $metadata, 'script', 1 );
-                $this->assertSame( 'test-script-handle-2', $result, 1 );
+                $this->assertSame( 'unit-tests-test-block-script', $result );
         }