Fix: Mock wp_localize_script in AdminTest

- Adds WP_Mock::userFunction for wp_localize_script to resolve PHPUnit error.

Fix: Revert underscore prefix for unused param

- Removes underscore from hook_suffix in Admin class.
- Relies on existing @phpcs:ignore directive.
This commit is contained in:
2025-04-18 19:53:40 +01:00
parent fb8a998487
commit ed7d33e2aa
2 changed files with 15 additions and 2 deletions

View File

@@ -41,11 +41,11 @@ class Admin {
/** /**
* Enqueues admin scripts and styles. * Enqueues admin scripts and styles.
* *
* @param string $_hook_suffix The current admin page. * @param string $hook_suffix The current admin page.
* *
* @phpcs:ignore WordPress.CodeAnalysis.UnusedFunctionParameter.Found * @phpcs:ignore WordPress.CodeAnalysis.UnusedFunctionParameter.Found
*/ */
public function enqueue_admin_assets( string $_hook_suffix ) { public function enqueue_admin_assets( string $hook_suffix ) {
// Enqueue admin styles. // Enqueue admin styles.
\wp_enqueue_style( \wp_enqueue_style(
'wpst-admin-style', 'wpst-admin-style',

View File

@@ -95,6 +95,19 @@ class AdminTest extends \WP_Mock\Tools\TestCase {
] ]
); );
// Expect wp_localize_script to be called
\WP_Mock::userFunction(
'wp_localize_script',
[
'times' => 1,
'args' => [
$script_handle,
'wpst_admin_params',
\Mockery::type( 'array' ), // We don't need to assert the exact array content here
],
]
);
// Call the method under test. // Call the method under test.
$this->admin->enqueue_admin_assets( 'test-hook' ); $this->admin->enqueue_admin_assets( 'test-hook' );