From cc6acc2a4e5661f0db4f7a79ef064236b6997162 Mon Sep 17 00:00:00 2001
From: marcusquinn <6428977+marcusquinn@users.noreply.github.com>
Date: Fri, 18 Apr 2025 19:55:20 +0100
Subject: [PATCH] Fix: Address CI failures

- Move @phpcs:ignore for unused param above function signature.
- Add WP_Mock for admin_url() in AdminTest to fix PHPUnit error.
---
 includes/Admin/class-admin.php |  3 +--
 tests/test-admin.php           | 13 +++++++++++++
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/includes/Admin/class-admin.php b/includes/Admin/class-admin.php
index 649cb57..4a82685 100644
--- a/includes/Admin/class-admin.php
+++ b/includes/Admin/class-admin.php
@@ -42,9 +42,8 @@ class Admin {
 	 * Enqueues admin scripts and styles.
 	 *
 	 * @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 ) {
 		// Enqueue admin styles.
 		\wp_enqueue_style(
diff --git a/tests/test-admin.php b/tests/test-admin.php
index 7e61580..191862b 100644
--- a/tests/test-admin.php
+++ b/tests/test-admin.php
@@ -96,6 +96,19 @@ class AdminTest extends \WP_Mock\Tools\TestCase {
         );
 
         // Expect wp_localize_script to be called
+        $expected_data = [
+            'ajax_url' => 'mock_ajax_url',
+            'nonce'    => 'mock_nonce',
+        ];
+        // Mock admin_url() before wp_localize_script uses it
+        \WP_Mock::userFunction(
+            'admin_url',
+            [
+                'times'  => 1,
+                'args'   => [ 'admin-ajax.php' ],
+                'return' => $expected_data['ajax_url'],
+            ]
+        );
         \WP_Mock::userFunction(
             'wp_localize_script',
             [