From 63f2520f70dc4aa2396d7bc3130f52793c016ce0 Mon Sep 17 00:00:00 2001 From: marcusquinn <6428977+marcusquinn@users.noreply.github.com> Date: Fri, 18 Apr 2025 19:58:56 +0100 Subject: [PATCH] Fix: Resolve CI errors - Fix docblock spacing for @phpcs:ignore in Admin class. - Add WP_Mock for wp_create_nonce() in AdminTest. --- includes/Admin/class-admin.php | 1 + tests/test-admin.php | 9 +++++++++ 2 files changed, 10 insertions(+) diff --git a/includes/Admin/class-admin.php b/includes/Admin/class-admin.php index d07d218..8c03bda 100644 --- a/includes/Admin/class-admin.php +++ b/includes/Admin/class-admin.php @@ -41,6 +41,7 @@ class Admin { /** * Enqueues admin scripts and styles. * + * @phpcs:ignore WordPress.CodeAnalysis.UnusedFunctionParameter.Found * @param string $hook_suffix The current admin page. */ diff --git a/tests/test-admin.php b/tests/test-admin.php index 191862b..d807974 100644 --- a/tests/test-admin.php +++ b/tests/test-admin.php @@ -109,6 +109,15 @@ class AdminTest extends \WP_Mock\Tools\TestCase { 'return' => $expected_data['ajax_url'], ] ); + // Mock wp_create_nonce() before wp_localize_script uses it + \WP_Mock::userFunction( + 'wp_create_nonce', + [ + 'times' => 1, + 'args' => [ 'wpst_admin_nonce' ], // Match the action string used in class-admin.php + 'return' => $expected_data['nonce'], + ] + ); \WP_Mock::userFunction( 'wp_localize_script', [