Debug

Author: westonruter

.github/workflows/phpunit-tests.yml

@@ -69,7 +69,7 @@ jobs:
     secrets: inherit
     if: ${{ startsWith( github.repository, 'WordPress/' ) && ( github.repository == 'WordPress/wordpress-develop' || github.event_name == 'pull_request' ) }}
     strategy:
-      fail-fast: false
+      fail-fast: true
       matrix:
         event: ['${{ github.event_name }}']
         os: [ ubuntu-24.04 ]
@@ -205,7 +205,7 @@ jobs:
     secrets: inherit
     if: ${{ startsWith( github.repository, 'WordPress/' ) && ( github.repository == 'WordPress/wordpress-develop' || github.event_name == 'pull_request' ) }}
     strategy:
-      fail-fast: false
+      fail-fast: true
       matrix:
         event: ['${{ github.event_name }}']
         os: [ ubuntu-24.04 ]
@@ -428,7 +428,7 @@ jobs:
     secrets: inherit
     if: ${{ startsWith( github.repository, 'WordPress/' ) && ( github.repository == 'WordPress/wordpress-develop' || github.event_name == 'pull_request' ) }}
     strategy:
-      fail-fast: false
+      fail-fast: true
       matrix:
         event: ['${{ github.event_name }}']
         os: [ ubuntu-24.04 ]
@@ -495,7 +495,7 @@ jobs:
     secrets: inherit
     if: ${{ startsWith( github.repository, 'WordPress/' ) && ( github.repository == 'WordPress/wordpress-develop' || github.event_name == 'pull_request' ) }}
     strategy:
-      fail-fast: false
+      fail-fast: true
       matrix:
         php: [ '7.2', '7.4', '8.0', '8.4' ]
         db-type: [ 'mysql' ]
@@ -524,7 +524,7 @@ jobs:
     secrets: inherit
     if: ${{ ! startsWith( github.repository, 'WordPress/' ) && github.event_name == 'pull_request' }}
     strategy:
-      fail-fast: false
+      fail-fast: true
       matrix:
         php: [ '7.2', '8.4' ]
         db-version: [ '8.4', '11.8' ]

src/wp-includes/class-wp-block.php

@@ -628,6 +628,9 @@ public function render( $options = array() ) {
 		 */
 		if ( ( ! empty( $this->block_type->style_handles ) ) ) {
 			foreach ( $this->block_type->style_handles as $style_handle ) {
+				if ( ! empty( $GLOBALS['debug_on_demand_block_style'] ) ) {
+					error_log( __FILE__ . ':' . __LINE__ . ' enqueue: ' . json_encode( $style_handle ) );
+				}
 				wp_enqueue_style( $style_handle );
 			}
 		}
@@ -675,6 +678,9 @@ public function render( $options = array() ) {
 		$after_styles_queue         = wp_styles()->queue;
 		$after_scripts_queue        = wp_scripts()->queue;
 		$after_script_modules_queue = wp_script_modules()->get_queue();
+		if ( ! empty( $GLOBALS['debug_on_demand_block_style'] ) ) {
+			error_log( __FILE__ . ':' . __LINE__ . ' $after_styles_queue: ' . json_encode( $after_styles_queue ) );
+		}
 
 		/*
 		 * As a very special case, a dynamic block may in fact include a call to wp_head() (and thus wp_enqueue_scripts()),

tests/phpunit/tests/template.php

@@ -1647,6 +1647,8 @@ function (): void {
 	 * @dataProvider data_wp_hoist_late_printed_styles
 	 */
 	public function test_wp_hoist_late_printed_styles( ?Closure $set_up, int $inline_size_limit, array $expected_styles ): void {
+		$GLOBALS['debug_on_demand_block_style'] = true; // TODO: Remove.
+
 		switch_theme( 'default' );
 		global $wp_styles;
 		$wp_styles = null;
@@ -1674,6 +1676,7 @@ static function () {
 
 		// Ensure that separate core block assets get registered.
 		register_core_block_style_handles();
+		$this->assertTrue( WP_Block_Type_Registry::get_instance()->is_registered( 'core/separator' ), 'Expected the core/separator block to be registered.' );
 		if ( wp_should_load_separate_core_block_assets() ) {
 			$this->assertTrue( wp_style_is( 'wp-block-separator', 'registered' ), 'Expected the wp-block-separator style to be registered.' );
 		}
@@ -1757,6 +1760,8 @@ static function () {
 			$found_subset_styles,
 			'Expected the same styles. Snapshot: ' . self::get_array_snapshot_export( $found_styles )
 		);
+
+		unset( $GLOBALS['debug_on_demand_block_style'] ); // TODO: Remove.
 	}
 
 	public function assertTemplateHierarchy( $url, array $expected, $message = '' ) {