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.
This commit is contained in:
@@ -42,9 +42,8 @@ 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(
|
||||||
|
|||||||
@@ -96,6 +96,19 @@ class AdminTest extends \WP_Mock\Tools\TestCase {
|
|||||||
);
|
);
|
||||||
|
|
||||||
// Expect wp_localize_script to be called
|
// 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_Mock::userFunction(
|
||||||
'wp_localize_script',
|
'wp_localize_script',
|
||||||
[
|
[
|
||||||
|
|||||||
Reference in New Issue
Block a user