wpallstars/wp-plugin-starter-template-for-ai-coding
Template
1
1
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix: Address PHPCS errors and PHPUnit 7.0 Mockery failure

Browse Source
This commit is contained in:
Marcus Quinn
2025-04-18 18:32:13 +01:00
parent b18f1c46b4
commit 6d3aeb889a
2 changed files with 19 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
includes/Admin/Admin.php → includes/Admin/class-admin.php
View File

@@ -35,7 +35,7 @@ class Admin {
* Initializes WordPress hooks. * Initializes WordPress hooks.
*/ */
private function initialize_hooks() { private function initialize_hooks() {
add_action( 'admin_enqueue_scripts', [ $this, 'enqueue_admin_assets' ] ); add_action( 'admin_enqueue_scripts', array( $this, 'enqueue_admin_assets' ) );
} }
/** /**

40
tests/test-admin.php
View File

@@ -40,12 +40,10 @@ class AdminTest extends \WP_Mock\Tools\TestCase {
$this->core = $this->createMock(Core::class); $this->core = $this->createMock(Core::class);
// Set up WordPress function mocks // Set up WordPress function mocks
WP_Mock::userFunction('add_action', [ WP_Mock::userFunction('is_admin', ['times' => 1, 'return' => true]);
'times' => 1, WP_Mock::expectActionAdded('admin_enqueue_scripts', [$this->admin, 'enqueue_admin_assets']);
'args' => ['admin_enqueue_scripts', \WP_Mock\Functions::type('array')],
]);
// Create instance of Admin class // Instantiate the class under test *after* setting up mocks
$this->admin = new Admin($this->core); $this->admin = new Admin($this->core);
} }
@@ -66,47 +64,45 @@ class AdminTest extends \WP_Mock\Tools\TestCase {
} }
/** /**
* Test enqueue_admin_assets * Test enqueue_admin_assets method
*/ */
public function test_enqueue_admin_assets() { public function test_enqueue_admin_assets() {
// Set up WordPress function mocks // Set up mocks for this specific test
WP_Mock::userFunction('wp_enqueue_style', [ WP_Mock::userFunction('wp_enqueue_style', [
'times' => 1, 'times' => 1,
'args' => ['wpst-admin-styles', \WP_Mock\Functions::type('string'), [], \WP_Mock\Functions::type('string')], 'args' => ['wpst-admin-style', 'path/to/admin/css/admin-styles.css'],
]); ]);
WP_Mock::userFunction('wp_enqueue_script', [ WP_Mock::userFunction('wp_enqueue_script', [
'times' => 1, 'times' => 1,
'args' => ['wpst-admin-scripts', \WP_Mock\Functions::type('string'), ['jquery'], \WP_Mock\Functions::type('string'), true], 'args' => ['wpst-admin-script', 'path/to/admin/js/admin-scripts.js', array('jquery'), null, true],
]); ]);
WP_Mock::userFunction('wp_localize_script', [ WP_Mock::userFunction('wp_localize_script', [
'times' => 1, 'times' => 1,
'args' => ['wpst-admin-scripts', 'wpstData', \WP_Mock\Functions::type('array')], 'args' => [
'wpst-admin-script',
'wpst_admin_params',
array(
'ajax_url' => 'http://example.org/wp-admin/admin-ajax.php',
'nonce' => '1234567890',
),
],
]); ]);
WP_Mock::userFunction('esc_html__', [
'times' => 2,
'args' => [\WP_Mock\Functions::type('string'), 'wp-plugin-starter-template'],
'return' => 'Translated string',
]);
WP_Mock::userFunction('admin_url', [ WP_Mock::userFunction('admin_url', [
'times' => 1, 'times' => 1,
'args' => ['admin-ajax.php'], 'args' => ['admin-ajax.php'],
'return' => 'http://example.org/wp-admin/admin-ajax.php', 'return' => 'http://example.org/wp-admin/admin-ajax.php',
]); ]);
WP_Mock::userFunction('wp_create_nonce', [ WP_Mock::userFunction('wp_create_nonce', [
'times' => 1, 'times' => 1,
'args' => ['wpst-admin-nonce'], 'args' => ['wpst-admin-nonce'],
'return' => '1234567890', 'return' => '1234567890',
]); ]);
// Call the method // Call the method under test
$this->admin->enqueue_admin_assets('plugins.php'); $this->admin->enqueue_admin_assets('plugins.php');
// If we get here, the test passed // WP_Mock::tearDown() in the main tearDown method will verify the expectations
// We can add an assertion here to make the test explicit
$this->assertTrue(true); $this->assertTrue(true);
} }
} }