Add CI/CD pipeline with automated testing and SFTP deployment
Introduces GitHub Actions workflows for continuous integration and deployment. CI runs PHP linting (7.4, 8.1, 8.3), PHPCS with WordPress coding standards, PHPUnit tests across PHP versions, and JS syntax checks. Deployment to Hostinger via FTPS triggers on push to main after all CI checks pass, with optional LiteSpeed cache purge. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Terence Ong
109
tests/unit/AdminTest.php
Normal file
109
tests/unit/AdminTest.php
Normal file
@@ -0,0 +1,109 @@
|
||||
<?php
|
||||
/**
|
||||
* Unit tests for Peptide_News_Admin.
|
||||
*
|
||||
* @package PeptideNews\Tests
|
||||
*/
|
||||
|
||||
namespace PeptideNews\Tests\Unit;
|
||||
|
||||
use PHPUnit\Framework\TestCase;
|
||||
|
||||
// Load the class under test.
|
||||
require_once PEPTIDE_NEWS_PLUGIN_DIR . 'admin/class-peptide-news-admin.php';
|
||||
|
||||
class AdminTest extends TestCase {
|
||||
|
||||
/**
|
||||
* @var \Peptide_News_Admin
|
||||
*/
|
||||
private $admin;
|
||||
|
||||
protected function setUp(): void {
|
||||
$this->admin = new \Peptide_News_Admin( 'peptide-news', '1.2.1' );
|
||||
|
||||
// Reset global test state.
|
||||
global $_test_settings_errors;
|
||||
$_test_settings_errors = array();
|
||||
}
|
||||
|
||||
/**
|
||||
* Constructor should set plugin_name property.
|
||||
*/
|
||||
public function test_constructor_sets_plugin_name(): void {
|
||||
$reflection = new \ReflectionClass( $this->admin );
|
||||
$prop = $reflection->getProperty( 'plugin_name' );
|
||||
$prop->setAccessible( true );
|
||||
|
||||
$this->assertEquals( 'peptide-news', $prop->getValue( $this->admin ) );
|
||||
}
|
||||
|
||||
/**
|
||||
* Constructor should set version property.
|
||||
*/
|
||||
public function test_constructor_sets_version(): void {
|
||||
$reflection = new \ReflectionClass( $this->admin );
|
||||
$prop = $reflection->getProperty( 'version' );
|
||||
$prop->setAccessible( true );
|
||||
|
||||
$this->assertEquals( '1.2.1', $prop->getValue( $this->admin ) );
|
||||
}
|
||||
|
||||
/**
|
||||
* sanitize_model_id() should accept a valid model ID.
|
||||
*/
|
||||
public function test_sanitize_model_id_accepts_valid_id(): void {
|
||||
$valid = 'openai/gpt-4o-mini';
|
||||
$this->assertEquals( $valid, $this->admin->sanitize_model_id( $valid ) );
|
||||
}
|
||||
|
||||
/**
|
||||
* sanitize_model_id() should accept model IDs with colons and dots.
|
||||
*/
|
||||
public function test_sanitize_model_id_accepts_complex_ids(): void {
|
||||
$valid = 'google/gemini-2.0-flash:free';
|
||||
$this->assertEquals( $valid, $this->admin->sanitize_model_id( $valid ) );
|
||||
}
|
||||
|
||||
/**
|
||||
* sanitize_model_id() should reject empty strings.
|
||||
*/
|
||||
public function test_sanitize_model_id_allows_empty_to_clear(): void {
|
||||
$result = $this->admin->sanitize_model_id( '' );
|
||||
$this->assertEquals( '', $result );
|
||||
}
|
||||
|
||||
/**
|
||||
* sanitize_model_id() should reject strings with spaces.
|
||||
*/
|
||||
public function test_sanitize_model_id_rejects_spaces(): void {
|
||||
global $_test_settings_errors;
|
||||
|
||||
$result = $this->admin->sanitize_model_id( 'invalid model id' );
|
||||
|
||||
$this->assertNotEmpty( $_test_settings_errors, 'Expected a settings error for invalid model ID.' );
|
||||
}
|
||||
|
||||
/**
|
||||
* sanitize_model_id() should reject strings with special characters.
|
||||
*/
|
||||
public function test_sanitize_model_id_rejects_special_chars(): void {
|
||||
global $_test_settings_errors;
|
||||
|
||||
$result = $this->admin->sanitize_model_id( '<script>alert(1)</script>' );
|
||||
|
||||
$this->assertNotEmpty( $_test_settings_errors, 'Expected a settings error for XSS attempt.' );
|
||||
}
|
||||
|
||||
/**
|
||||
* enqueue_styles() should not enqueue on non-plugin pages.
|
||||
*
|
||||
* We verify it returns early by checking no errors are thrown
|
||||
* when called with a non-matching hook.
|
||||
*/
|
||||
public function test_enqueue_styles_skips_non_plugin_pages(): void {
|
||||
// Should return without error when hook doesn't match.
|
||||
$this->admin->enqueue_styles( 'edit.php' );
|
||||
$this->assertTrue( true ); // No exception = pass.
|
||||
}
|
||||
}
|
||||
81
tests/unit/FetcherTest.php
Normal file
81
tests/unit/FetcherTest.php
Normal file
@@ -0,0 +1,81 @@
|
||||
<?php
|
||||
/**
|
||||
* Unit tests for Peptide_News_Fetcher.
|
||||
*
|
||||
* @package PeptideNews\Tests
|
||||
*/
|
||||
|
||||
namespace PeptideNews\Tests\Unit;
|
||||
|
||||
use PHPUnit\Framework\TestCase;
|
||||
|
||||
// Load the class under test.
|
||||
require_once PEPTIDE_NEWS_PLUGIN_DIR . 'includes/class-peptide-news-fetcher.php';
|
||||
|
||||
class FetcherTest extends TestCase {
|
||||
|
||||
/**
|
||||
* @var \Peptide_News_Fetcher
|
||||
*/
|
||||
private $fetcher;
|
||||
|
||||
protected function setUp(): void {
|
||||
$this->fetcher = new \Peptide_News_Fetcher();
|
||||
}
|
||||
|
||||
/**
|
||||
* add_custom_cron_schedules() should add the expected intervals.
|
||||
*/
|
||||
public function test_add_custom_cron_schedules_adds_intervals(): void {
|
||||
$existing = array(
|
||||
'hourly' => array(
|
||||
'interval' => 3600,
|
||||
'display' => 'Once Hourly',
|
||||
),
|
||||
);
|
||||
|
||||
$result = $this->fetcher->add_custom_cron_schedules( $existing );
|
||||
|
||||
// Original schedule must be preserved.
|
||||
$this->assertArrayHasKey( 'hourly', $result );
|
||||
|
||||
// Custom schedules must be added.
|
||||
$this->assertArrayHasKey( 'every_fifteen_minutes', $result );
|
||||
$this->assertArrayHasKey( 'every_thirty_minutes', $result );
|
||||
$this->assertArrayHasKey( 'every_four_hours', $result );
|
||||
$this->assertArrayHasKey( 'every_six_hours', $result );
|
||||
}
|
||||
|
||||
/**
|
||||
* Custom intervals should have correct durations.
|
||||
*/
|
||||
public function test_cron_schedule_intervals_are_correct(): void {
|
||||
$result = $this->fetcher->add_custom_cron_schedules( array() );
|
||||
|
||||
$this->assertEquals( 900, $result['every_fifteen_minutes']['interval'] );
|
||||
$this->assertEquals( 1800, $result['every_thirty_minutes']['interval'] );
|
||||
$this->assertEquals( 14400, $result['every_four_hours']['interval'] );
|
||||
$this->assertEquals( 21600, $result['every_six_hours']['interval'] );
|
||||
}
|
||||
|
||||
/**
|
||||
* Custom schedules should have display names.
|
||||
*/
|
||||
public function test_cron_schedules_have_display_names(): void {
|
||||
$result = $this->fetcher->add_custom_cron_schedules( array() );
|
||||
|
||||
foreach ( $result as $key => $schedule ) {
|
||||
$this->assertArrayHasKey( 'display', $schedule, "Schedule '{$key}' missing 'display' key." );
|
||||
$this->assertNotEmpty( $schedule['display'], "Schedule '{$key}' has empty display name." );
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Passing an empty array should still return all custom schedules.
|
||||
*/
|
||||
public function test_cron_schedules_work_with_empty_input(): void {
|
||||
$result = $this->fetcher->add_custom_cron_schedules( array() );
|
||||
|
||||
$this->assertCount( 4, $result );
|
||||
}
|
||||
}
|
||||
84
tests/unit/PluginBootstrapTest.php
Normal file
84
tests/unit/PluginBootstrapTest.php
Normal file
@@ -0,0 +1,84 @@
|
||||
<?php
|
||||
/**
|
||||
* Unit tests for the main plugin bootstrap file.
|
||||
*
|
||||
* Validates constants, version strings, and file structure.
|
||||
*
|
||||
* @package PeptideNews\Tests
|
||||
*/
|
||||
|
||||
namespace PeptideNews\Tests\Unit;
|
||||
|
||||
use PHPUnit\Framework\TestCase;
|
||||
|
||||
class PluginBootstrapTest extends TestCase {
|
||||
|
||||
/**
|
||||
* Plugin version constant should be defined and non-empty.
|
||||
*/
|
||||
public function test_version_constant_is_defined(): void {
|
||||
$this->assertTrue( defined( 'PEPTIDE_NEWS_VERSION' ) );
|
||||
$this->assertNotEmpty( PEPTIDE_NEWS_VERSION );
|
||||
}
|
||||
|
||||
/**
|
||||
* Plugin directory constant should point to a real directory.
|
||||
*/
|
||||
public function test_plugin_dir_constant_is_valid(): void {
|
||||
$this->assertTrue( defined( 'PEPTIDE_NEWS_PLUGIN_DIR' ) );
|
||||
$this->assertDirectoryExists( PEPTIDE_NEWS_PLUGIN_DIR );
|
||||
}
|
||||
|
||||
/**
|
||||
* Main plugin file should exist.
|
||||
*/
|
||||
public function test_main_plugin_file_exists(): void {
|
||||
$this->assertFileExists( PEPTIDE_NEWS_PLUGIN_DIR . 'peptide-news-plugin.php' );
|
||||
}
|
||||
|
||||
/**
|
||||
* All required class files should exist.
|
||||
*/
|
||||
public function test_required_class_files_exist(): void {
|
||||
$files = array(
|
||||
'includes/class-peptide-news.php',
|
||||
'includes/class-peptide-news-activator.php',
|
||||
'includes/class-peptide-news-deactivator.php',
|
||||
'includes/class-peptide-news-fetcher.php',
|
||||
'includes/class-peptide-news-analytics.php',
|
||||
'includes/class-peptide-news-rest-api.php',
|
||||
'admin/class-peptide-news-admin.php',
|
||||
);
|
||||
|
||||
foreach ( $files as $file ) {
|
||||
$path = PEPTIDE_NEWS_PLUGIN_DIR . $file;
|
||||
$this->assertFileExists( $path, "Required file missing: {$file}" );
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* Admin JavaScript file should exist.
|
||||
*/
|
||||
public function test_admin_script_exists(): void {
|
||||
$this->assertFileExists( PEPTIDE_NEWS_PLUGIN_DIR . 'admin/js/admin-script.js' );
|
||||
}
|
||||
|
||||
/**
|
||||
* Admin stylesheet should exist.
|
||||
*/
|
||||
public function test_admin_style_exists(): void {
|
||||
$this->assertFileExists( PEPTIDE_NEWS_PLUGIN_DIR . 'admin/css/admin-style.css' );
|
||||
}
|
||||
|
||||
/**
|
||||
* Version string should follow SemVer pattern.
|
||||
*/
|
||||
public function test_version_follows_semver(): void {
|
||||
// Allow semver with optional pre-release suffix (e.g. 1.2.1-test).
|
||||
$this->assertMatchesRegularExpression(
|
||||
'/^\d+\.\d+\.\d+(-[\w.]+)?$/',
|
||||
PEPTIDE_NEWS_VERSION,
|
||||
'Version does not follow SemVer pattern.'
|
||||
);
|
||||
}
|
||||
}
|
||||
102
tests/unit/RestApiTest.php
Normal file
102
tests/unit/RestApiTest.php
Normal file
@@ -0,0 +1,102 @@
|
||||
<?php
|
||||
/**
|
||||
* Unit tests for Peptide_News_Rest_API.
|
||||
*
|
||||
* @package PeptideNews\Tests
|
||||
*/
|
||||
|
||||
namespace PeptideNews\Tests\Unit;
|
||||
|
||||
use PHPUnit\Framework\TestCase;
|
||||
|
||||
// Load the class under test.
|
||||
require_once PEPTIDE_NEWS_PLUGIN_DIR . 'includes/class-peptide-news-rest-api.php';
|
||||
|
||||
class RestApiTest extends TestCase {
|
||||
|
||||
/**
|
||||
* @var \Peptide_News_Rest_API
|
||||
*/
|
||||
private $api;
|
||||
|
||||
protected function setUp(): void {
|
||||
$this->api = new \Peptide_News_Rest_API();
|
||||
}
|
||||
|
||||
/**
|
||||
* validate_date_format() should accept valid Y-m-d dates.
|
||||
*/
|
||||
public function test_validate_date_accepts_valid_format(): void {
|
||||
$request = new \WP_REST_Request();
|
||||
$result = $this->api->validate_date_format( '2026-01-15', $request, 'start_date' );
|
||||
|
||||
$this->assertTrue( $result );
|
||||
}
|
||||
|
||||
/**
|
||||
* validate_date_format() should accept today's date.
|
||||
*/
|
||||
public function test_validate_date_accepts_today(): void {
|
||||
$request = new \WP_REST_Request();
|
||||
$result = $this->api->validate_date_format( date( 'Y-m-d' ), $request, 'end_date' );
|
||||
|
||||
$this->assertTrue( $result );
|
||||
}
|
||||
|
||||
/**
|
||||
* validate_date_format() should reject invalid formats.
|
||||
*/
|
||||
public function test_validate_date_rejects_wrong_format(): void {
|
||||
$request = new \WP_REST_Request();
|
||||
$result = $this->api->validate_date_format( '01-15-2026', $request, 'start_date' );
|
||||
|
||||
$this->assertInstanceOf( \WP_Error::class, $result );
|
||||
}
|
||||
|
||||
/**
|
||||
* validate_date_format() should reject empty strings.
|
||||
*/
|
||||
public function test_validate_date_rejects_empty(): void {
|
||||
$request = new \WP_REST_Request();
|
||||
$result = $this->api->validate_date_format( '', $request, 'start_date' );
|
||||
|
||||
$this->assertInstanceOf( \WP_Error::class, $result );
|
||||
}
|
||||
|
||||
/**
|
||||
* validate_date_format() should reject non-date strings.
|
||||
*/
|
||||
public function test_validate_date_rejects_garbage(): void {
|
||||
$request = new \WP_REST_Request();
|
||||
$result = $this->api->validate_date_format( 'not-a-date', $request, 'start_date' );
|
||||
|
||||
$this->assertInstanceOf( \WP_Error::class, $result );
|
||||
}
|
||||
|
||||
/**
|
||||
* validate_date_format() should reject dates with slashes.
|
||||
*/
|
||||
public function test_validate_date_rejects_slashes(): void {
|
||||
$request = new \WP_REST_Request();
|
||||
$result = $this->api->validate_date_format( '2026/01/15', $request, 'start_date' );
|
||||
|
||||
$this->assertInstanceOf( \WP_Error::class, $result );
|
||||
}
|
||||
|
||||
/**
|
||||
* check_admin_permissions() should return WP_Error when user lacks capability.
|
||||
*/
|
||||
public function test_check_admin_permissions_denies_non_admin(): void {
|
||||
// Stub current_user_can to return false.
|
||||
if ( ! function_exists( 'current_user_can' ) ) {
|
||||
function current_user_can() {
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
$request = new \WP_REST_Request();
|
||||
$result = $this->api->check_admin_permissions( $request );
|
||||
|
||||
$this->assertInstanceOf( \WP_Error::class, $result );
|
||||
}
|
||||
}
|
||||
Reference in New Issue
Block a user