Changeset 54671 for branches/6.1/tests/phpunit/tests/blocks/register.php

Timestamp:

10/24/2022 02:30:20 PM (3 years ago)

Author:

Bernhard Reiter

Message:

Blocks: Allow arrays for deprecated asset types in block registration.

In register_block_type, continue to allow passing arrays as the editor_script, script, view_script, editor_style, and style arguments. Note that those fields were soft-deprecated in favor of their _handles counterparts in [54155], which would allow specifying multiple items. At the same time, the deprecated fields were limited to string or null.

However, this broke existing code that passed an array as one of those arguments. For backwards compatibility, this change thus restores the previous behavior. It is implemented in WP_Block_Type as a pair of __get() and __set() methods that wrap around the corresponding _handles members, which are arrays of strings.

It also affects the REST API endpoint for block types. The latter’s schema has never allowed for anything other than string or null for any of those fields. For this reason, it now returns the first element of the array stored in the corresponding _handles member in WP_Block_Type.

Follow-up [54155].
Props nendeb55, costdev, gziolo, spacedmonkey, mukesh27, sergeybiryukov, audrasjb.
Merges [54670] to the 6.1 branch.
Fixes #56707.

File:

• branches/6.1/tests/phpunit/tests/blocks/register.php (modified) (1 diff)

branches/6.1/tests/phpunit/tests/blocks/register.php

@@ -554 +554 @@
 
     /**
+     * Tests that an array value for 'editor_script' is correctly set and retrieved.
+     *
+     * As 'editor_script' is now a deprecated property, this should also set
+     * the value for the 'editor_script_handles' property.
+     *
+     * @ticket 56707
+     *
+     * @covers ::register_block_type
+     * @covers WP_Block_Type::__set
+     * @covers WP_Block_Type::__get
+     *
+     * @dataProvider data_register_block_type_accepts_editor_script_array
+     *
+     * @param array $editor_script The editor script array to register.
+     * @param array $expected      The expected registered editor script.
+     */
+    public function test_register_block_type_accepts_editor_script_array( $editor_script, $expected ) {
+        $settings = array( 'editor_script' => $editor_script );
+        register_block_type( 'core/test-static', $settings );
+
+        $registry   = WP_Block_Type_Registry::get_instance();
+        $block_type = $registry->get_registered( 'core/test-static' );
+        $this->assertObjectHasAttribute( 'editor_script_handles', $block_type );
+        $actual_script         = $block_type->editor_script;
+        $actual_script_handles = $block_type->editor_script_handles;
+
+        $this->assertSame(
+            $expected,
+            $actual_script,
+            'editor_script was not set to the correct value.'
+        );
+
+        $this->assertSame(
+            (array) $expected,
+            $actual_script_handles,
+            'editor_script_handles was not set to the correct value.'
+        );
+    }
+
+    /**
+     * Data provider for test_register_block_type_accepts_editor_script_array().
+     *
+     * @return array
+     */
+    public function data_register_block_type_accepts_editor_script_array() {
+        return array(
+            'an empty array'      => array(
+                'editor_script' => array(),
+                'expected'      => null,
+            ),
+            'a single item array' => array(
+                'editor_script' => array( 'hello' ),
+                'expected'      => 'hello',
+            ),
+            'a multi-item array'  => array(
+                'editor_script' => array( 'hello', 'world' ),
+                'expected'      => array( 'hello', 'world' ),
+            ),
+        );
+    }
+
+    /**
+     * Tests that an array value for 'editor_script' containing invalid values
+     * correctly triggers _doing_it_wrong(), filters the value, and sets the
+     * property to the result.
+     *
+     * As 'editor_script' is now a deprecated property, this should also set
+     * the value for the 'editor_script_handles' property.
+     *
+     * @ticket 56707
+     *
+     * @covers ::register_block_type
+     * @covers WP_Block_Type::__set
+     * @covers WP_Block_Type::__get
+     *
+     * @dataProvider data_register_block_type_throws_doing_it_wrong
+     *
+     * @expectedIncorrectUsage WP_Block_Type::__set
+     *
+     * @param array $editor_script The editor script array to register.
+     * @param array $expected      The expected registered editor script.
+     */
+    public function test_register_block_type_throws_doing_it_wrong( $editor_script, $expected ) {
+        $settings = array( 'editor_script' => $editor_script );
+        register_block_type( 'core/test-static', $settings );
+
+        $registry   = WP_Block_Type_Registry::get_instance();
+        $block_type = $registry->get_registered( 'core/test-static' );
+        $this->assertObjectHasAttribute( 'editor_script_handles', $block_type );
+        $actual_script         = $block_type->editor_script;
+        $actual_script_handles = $block_type->editor_script_handles;
+
+        $this->assertSame(
+            $expected,
+            $actual_script,
+            'editor_script was not set to the correct value.'
+        );
+
+        $this->assertSame(
+            (array) $expected,
+            $actual_script_handles,
+            'editor_script_handles was not set to the correct value.'
+        );
+    }
+
+    /**
+     * Data provider for test_register_block_type_throws_doing_it_wrong().
+     *
+     * @return array
+     */
+    public function data_register_block_type_throws_doing_it_wrong() {
+        return array(
+            'a non-string array'     => array(
+                'editor_script' => array( null, false, true, -1, 0, 1, -1.0, 0.0, 1.0, INF, NAN, new stdClass() ),
+                'expected'      => null,
+            ),
+            'a partial string array' => array(
+                'editor_script' => array( null, false, 'script.js', true, 0, 'actions.js', 1, INF ),
+                'expected'      => array( 'script.js', 'actions.js' ),
+            ),
+            'a partial string array that results in one item with non-zero index' => array(
+                'editor_script' => array( null, false, 'script.js' ),
+                'expected'      => 'script.js',
+            ),
+        );
+    }
+
+    /**
      * @ticket 52301
      */