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

Timestamp:

01/31/2024 09:09:22 AM (16 months ago)

Author:

gziolo

Message:

Editor: Add viewStyle property to block.json for frontend-only block styles

Related issue in Gutenberg: ​https://github.com/WordPress/gutenberg/issues/54491.

For block scripts there was already script, viewScript and editorScript. For block styles there was only style and editorStyle. This brings the parity.

Props gaambo.
Fixes #59673.

File:

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

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

@@ -157 +157 @@
             generate_block_asset_handle( $block_name, 'style' )
         );
+        // @ticket 59673
+        $this->assertSame(
+            'unit-tests-my-block-view-style',
+            generate_block_asset_handle( $block_name, 'viewStyle' ),
+            'asset handle for viewStyle is not generated correctly'
+        );
     }
 
@@ -440 +446 @@
     public function test_success_register_block_style_handle() {
         $metadata = array(
-            'file'  => DIR_TESTDATA . '/blocks/notice/block.json',
-            'name'  => 'unit-tests/test-block',
-            'style' => 'file:./block.css',
+            'file'      => DIR_TESTDATA . '/blocks/notice/block.json',
+            'name'      => 'unit-tests/test-block',
+            'style'     => 'file:./block.css',
+            'viewStyle' => 'file:./block-view.css',
         );
         $result   = register_block_style_handle( $metadata, 'style' );
@@ -453 +460 @@
             wp_normalize_path( realpath( DIR_TESTDATA . '/blocks/notice/block.css' ) ),
             wp_normalize_path( wp_styles()->get_data( 'unit-tests-test-block-style', 'path' ) )
+        );
+
+        // Test viewStyle property
+        $result = register_block_style_handle( $metadata, 'viewStyle' );
+        $this->assertSame( 'unit-tests-test-block-view-style', $result );
+
+        // @ticket 59673
+        $this->assertSame(
+            wp_normalize_path( realpath( DIR_TESTDATA . '/blocks/notice/block-view.css' ) ),
+            wp_normalize_path( wp_styles()->get_data( 'unit-tests-test-block-view-style', 'path' ) ),
+            'viewStyle asset path is not correct'
         );
 
@@ -838 +856 @@
             $result->style_handles
         );
+        // @ticket 59673
+        $this->assertSameSets(
+            array( 'tests-notice-view-style' ),
+            $result->view_style_handles,
+            'parsed view_style_handles is not correct'
+        );
 
         // @ticket 50328
@@ -843 +867 @@
             wp_normalize_path( realpath( DIR_TESTDATA . '/blocks/notice/block.css' ) ),
             wp_normalize_path( wp_styles()->get_data( 'unit-tests-test-block-style', 'path' ) )
+        );
+
+        // @ticket 59673
+        $this->assertSame(
+            wp_normalize_path( realpath( DIR_TESTDATA . '/blocks/notice/block-view.css' ) ),
+            wp_normalize_path( wp_styles()->get_data( 'unit-tests-test-block-view-style', 'path' ) ),
+            'viewStyle asset path is not correct'
         );