Changeset 61529

Timestamp:

01/26/2026 04:18:46 PM (3 days ago)

Author:

jonsurrell

Message:

Block Processor: Remove use of NumberFormatter from test suite.

The Block Processor tests call NumberFormatter for generating its error messages, but in environments lacking the intl extension this led to crashes while running the tests.

This patch reworks the tests so that they avoid calling NumberFormatter so that the tests run on more diverse setups.

Developed in ​https://github.com/WordPress/wordpress-develop/pull/10576
Discussed in https://core.trac.wordpress.org/ticket/64329

Follow-up to [60939].

Reviewed by jonsurrell, jorbin.
Merges [61342] to the 6.9 branch.

Props desrosj, ellatrix, peterwilsoncc, westonruter.

Fixes #64329.

Location:

branches/6.9

Files:

• . (modified) (1 prop)
• tests/phpunit/tests/block-processor/wpBlockProcessor-BlockProcessing.php (modified) (3 diffs)

branches/6.9/tests/phpunit/tests/block-processor/wpBlockProcessor-BlockProcessing.php

@@ -50 +50 @@
 
         $processor = new WP_Block_Processor( $html );
-        $n         = new NumberFormatter( 'en-US', NumberFormatter::ORDINAL );
 
         for ( $i = 0; $i < $max_depth; $i++ ) {
+            $nth = $i + 1;
+
             $this->assertTrue(
                 $processor->next_delimiter(),
-                "Should have found {$n->format( $i + 1 )} opening delimiter: check test setup."
+                "Should have found opening delimiter #{$nth}: check test setup."
             );
 
@@ -61 +62 @@
                 $i + 1,
                 $processor->get_depth(),
-                "Should have identified the proper depth of the {$n->format( $i + 1 )} opening delimiter."
+                "Should have identified the proper depth of opening delimiter #{$nth}."
             );
         }
 
         for ( $i = 0; $i < $max_depth; $i++ ) {
+            $nth = $i + 1;
+
             $this->assertTrue(
                 $processor->next_delimiter(),
-                "Should have found {$n->format( $i + 1 )} closing delimiter: check test setup."
+                "Should have found closing delimiter #{$nth}: check test setup."
             );
 
@@ -74 +77 @@
                 $max_depth - $i - 1,
                 $processor->get_depth(),
-                "Should have identified the proper depth of the {$n->format( $i + 1 )} closing delimiter."
+                "Should have identified the proper depth of closing delimiter #{$nth}."
             );
         }