Changeset 56607

Timestamp:

09/18/2023 10:33:24 AM (21 months ago)

Author:

gziolo

Message:

Tests: Add additional tests covering Block Hooks registration

Props ockham.
See #59346.
Follow-up [56587].

Location:

trunk/tests/phpunit

Files:

• data/blocks/hooked-block-error (added)
• data/blocks/hooked-block-error/block.json (added)
• data/blocks/notice/block.json (modified) (1 diff)
• tests/blocks/register.php (modified) (2 diffs)

trunk/tests/phpunit/data/blocks/notice/block.json

@@ -32 +32 @@
     },
     "blockHooks": {
-        "core/post-content": "before"
+        "tests/before": "before",
+        "tests/after": "after",
+        "tests/first-child": "firstChild",
+        "tests/last-child": "lastChild"
     },
     "supports": {

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

@@ -646 +646 @@
         );
         // @ticket 59346
-        $this->assertSame(
-            array( 'core/post-content' => 'before' ),
+        $this->assertSameSets(
+            array(
+                'tests/before'      => 'before',
+                'tests/after'       => 'after',
+                'tests/first-child' => 'first_child',
+                'tests/last-child'  => 'last_child',
+            ),
             $result->block_hooks,
             'Block type should contain block hooks from metadata.'
@@ -1070 +1075 @@
         $this->assertTrue( $actual );
     }
+
+    /**
+     * @ticket 59346
+     *
+     * @covers ::register_block_type
+     *
+     * @expectedIncorrectUsage register_block_type_from_metadata
+     */
+    public function test_register_block_hooks_targeting_itself() {
+        $block_type = register_block_type(
+            DIR_TESTDATA . '/blocks/hooked-block-error'
+        );
+
+        $this->assertSame(
+            array( 'tests/other-block' => 'after' ),
+            $block_type->block_hooks
+        );
+    }
 }