Changeset 54631

Timestamp:

10/18/2022 01:28:59 PM (2 years ago)

Author:

SergeyBiryukov

Message:

Tests: Correct the wp_theme_json_data_default filter name in assertion messages.

The theme_json_default filter was previously renamed to wp_theme_json_data_default.

This commit updates the messages in WP_Theme_JSON_Resolver tests to match the new name.

Follow-up to [54501], [54630].

See #56796, #56835.

File:

• trunk/tests/phpunit/tests/theme/wpThemeJsonResolver.php (modified) (2 diffs)

trunk/tests/phpunit/tests/theme/wpThemeJsonResolver.php

@@ -397 +397 @@
         }
 
-        $this->assertSame( $expected_filter_count, did_filter( 'wp_theme_json_data_default' ), 'The filter "theme_json_default" should fire the given number of times' );
+        $this->assertSame( $expected_filter_count, did_filter( 'wp_theme_json_data_default' ), 'The filter "wp_theme_json_data_default" should fire the given number of times' );
         $this->assertInstanceOf( WP_Theme_JSON::class, $actual, 'WP_Theme_JSON_Resolver::get_core_data() should return instance of WP_Theme_JSON' );
         $this->assertSame( static::$property_core->getValue(), $actual, 'WP_Theme_JSON_Resolver::$core property should be the same object as returned from WP_Theme_JSON_Resolver::get_core_data()' );
@@ -706 +706 @@
 
         // Test that wp_theme_json_data_theme filter has been called.
-        $this->assertGreaterThan( 0, did_filter( 'wp_theme_json_data_default' ), 'The filter "theme_json_default" should fire.' );
+        $this->assertGreaterThan( 0, did_filter( 'wp_theme_json_data_default' ), 'The filter "wp_theme_json_data_default" should fire.' );
 
         // Test that data from theme.json is backfilled from existing theme supports.