[], @@ -67,7 +67,7 @@ $GLOBALS['pr_test_singular_type'] = ''; $GLOBALS['pr_test_the_id'] = 0; $GLOBALS['pr_test_peptide_dto'] = null; -// ── WordPress function stubs ────────────────────────────────────────── +// ── WordPress function stubs ──────────────────────────────────────────. if ( ! function_exists( 'trailingslashit' ) ) { function trailingslashit( string $path ): string { @@ -170,7 +170,7 @@ if ( ! function_exists( 'is_email' ) ) { } } -// ── Option / cron stubs ─────────────────────────────────────────────── +// ── Option / cron stubs ───────────────────────────────────────────────. if ( ! function_exists( 'get_option' ) ) { function get_option( string $key, $default = false ) { @@ -210,7 +210,7 @@ if ( ! function_exists( 'add_settings_field' ) ) { function add_settings_field( string $id, string $title, $cb, string $page, string $section = '' ): void {} } -// ── Post meta stubs ─────────────────────────────────────────────────── +// ── Post meta stubs ───────────────────────────────────────────────────. if ( ! function_exists( 'get_post_meta' ) ) { function get_post_meta( $post_id, string $key = '', bool $single = false ) { @@ -242,7 +242,7 @@ if ( ! function_exists( 'get_posts' ) ) { } } -// ── CPT / taxonomy stubs ────────────────────────────────────────────── +// ── CPT / taxonomy stubs ──────────────────────────────────────────────. if ( ! function_exists( 'post_type_exists' ) ) { function post_type_exists( string $post_type ): bool { @@ -297,7 +297,7 @@ if ( ! function_exists( 'add_filter' ) ) { } } -// ── Singular / permalink stubs ──────────────────────────────────────── +// ── Singular / permalink stubs ────────────────────────────────────────. if ( ! function_exists( 'is_singular' ) ) { function is_singular( $type = '' ): bool { @@ -334,7 +334,7 @@ if ( ! function_exists( 'get_author_posts_url' ) ) { } } -// ── WP_Error stub ───────────────────────────────────────────────────── +// ── WP_Error stub ─────────────────────────────────────────────────────. if ( ! class_exists( 'WP_Error' ) ) { class WP_Error { @@ -367,7 +367,7 @@ if ( ! class_exists( 'WP_Error' ) ) { } -// ── WP_Post stub ────────────────────────────────────────────────────── +// ── WP_Post stub ──────────────────────────────────────────────────────. if ( ! class_exists( 'WP_Post' ) ) { class WP_Post { @@ -411,7 +411,7 @@ if ( ! function_exists( 'wp_count_posts' ) ) { } } -// ── Load the plugin's autoloader ────────────────────────────────────── +// ── Load the plugin's autoloader ──────────────────────────────────────. require_once PR_CORE_PLUGIN_DIR . 'includes/class-pr-core-autoloader.php'; PR_Core_Autoloader::register(); diff --git a/tests/unit/JsonldPeptidePieceTest.php b/tests/unit/JsonldPeptidePieceTest.php index ea07c7c..5184eeb 100644 --- a/tests/unit/JsonldPeptidePieceTest.php +++ b/tests/unit/JsonldPeptidePieceTest.php @@ -45,7 +45,7 @@ class JsonldPeptidePieceTest extends TestCase { require_once PR_CORE_PLUGIN_DIR . 'includes/frontend/class-pr-core-jsonld.php'; } - // ── Helpers ─────────────────────────────────────────────────────── + // ── Helpers ───────────────────────────────────────────────────────. /** * @param array $overrides @@ -97,8 +97,8 @@ class JsonldPeptidePieceTest extends TestCase { $GLOBALS['pr_test_singular_type'] = PR_Core_Peptide_CPT::POST_TYPE; $GLOBALS['pr_test_postmeta'][ $post_id ] = $meta; $GLOBALS['pr_test_peptide_dto'] = $this->make_dto( [ 'id' => $post_id ] ); - // Supply a WP_Post object so PR_Core_Peptide_Repository::find_by_id() passes - // the post_type check and post_to_dto() type hint (our get_post() stub reads + // Supply a WP_Post object so PR_Core_Peptide_Repository::find_by_id() passes. + // the post_type check and post_to_dto() type hint (our get_post() stub reads. // from pr_test_posts). $post = new WP_Post(); $post->ID = $post_id; @@ -111,7 +111,7 @@ class JsonldPeptidePieceTest extends TestCase { $GLOBALS['pr_test_posts'][ $post_id ] = $post; } - // ── Regression: get_class(array) throws ─────────────────────────── + // ── Regression: get_class(array) throws ───────────────────────────. public function test_get_class_on_plain_array_throws(): void { $GLOBALS['pr_test_postmeta'][36] = []; @@ -127,7 +127,7 @@ class JsonldPeptidePieceTest extends TestCase { $this->assertTrue( $caught, 'get_class(array) must throw — confirms the v0.6.1 fatal' ); } - // ── inject_graph_nodes does not throw ───────────────────────────── + // ── inject_graph_nodes does not throw ─────────────────────────────. public function test_inject_graph_nodes_does_not_throw(): void { $this->set_peptide_context( 36, [ @@ -224,7 +224,7 @@ class JsonldPeptidePieceTest extends TestCase { $this->assertCount( 1, $breadcrumb ); } - // ── FAQ injection via inject_graph_nodes ────────────────────────── + // ── FAQ injection via inject_graph_nodes ──────────────────────────. public function test_inject_graph_nodes_faq_present_when_items_exist(): void { $this->set_peptide_context( 36, [ @@ -250,7 +250,7 @@ class JsonldPeptidePieceTest extends TestCase { $this->assertCount( 0, $faq_nodes ); } - // ── Non-peptide page: no injection ──────────────────────────────── + // ── Non-peptide page: no injection ────────────────────────────────. public function test_inject_graph_nodes_noop_on_non_peptide_page(): void { $GLOBALS['pr_test_is_singular'] = false; @@ -262,7 +262,7 @@ class JsonldPeptidePieceTest extends TestCase { $this->assertCount( 1, $result ); } - // ── register_hooks: wpseo_schema_graph_pieces must NOT be hooked ── + // ── register_hooks: wpseo_schema_graph_pieces must NOT be hooked ──. public function test_jsonld_does_not_hook_wpseo_schema_graph_pieces(): void { $src = file_get_contents( PR_CORE_PLUGIN_DIR . 'includes/frontend/class-pr-core-jsonld.php' ); diff --git a/tests/unit/JsonldTest.php b/tests/unit/JsonldTest.php index 3f5836d..a9bc3d9 100644 --- a/tests/unit/JsonldTest.php +++ b/tests/unit/JsonldTest.php @@ -44,7 +44,7 @@ class JsonldTest extends TestCase { require_once PR_CORE_PLUGIN_DIR . 'includes/frontend/class-pr-core-jsonld.php'; } - // ── Helper ──────────────────────────────────────────────────────── + // ── Helper ────────────────────────────────────────────────────────. /** * @param array $overrides @@ -73,7 +73,7 @@ class JsonldTest extends TestCase { ], $overrides ) ); } - // ── Drug @id ───────────────────────────────────────────────────── + // ── Drug @id ─────────────────────────────────────────────────────. public function test_drug_node_has_id(): void { $GLOBALS['pr_test_postmeta'][1] = []; @@ -89,7 +89,7 @@ class JsonldTest extends TestCase { $this->assertSame( 'https://peptiderepo.com/peptides/bpc-157/#drug', $drug_node['@id'] ); } - // ── Drug @type ──────────────────────────────────────────────────── + // ── Drug @type ────────────────────────────────────────────────────. public function test_drug_type_is_array(): void { $GLOBALS['pr_test_postmeta'][1] = []; @@ -112,7 +112,7 @@ class JsonldTest extends TestCase { $this->assertContains( 'MolecularEntity', $drug_node['@type'] ); } - // ── Drug fields present when meta set ──────────────────────────── + // ── Drug fields present when meta set ────────────────────────────. public function test_molecular_formula_present_when_meta_set(): void { $GLOBALS['pr_test_postmeta'][1] = [ '_pr_molecular_formula' => 'C62H98N16O22', '_pr_molecular_weight' => '1419.5', '_pr_aliases' => '[]' ]; @@ -140,7 +140,7 @@ class JsonldTest extends TestCase { $this->assertCount( 2, $drug_node['alternateName'] ); } - // ── Drug fields omitted when meta empty ────────────────────────── + // ── Drug fields omitted when meta empty ──────────────────────────. public function test_molecular_formula_omitted_when_meta_empty(): void { $GLOBALS['pr_test_postmeta'][1] = []; @@ -163,7 +163,7 @@ class JsonldTest extends TestCase { $this->assertArrayNotHasKey( 'alternateName', $drug_node ); } - // ── FAQPage emission ────────────────────────────────────────────── + // ── FAQPage emission ──────────────────────────────────────────────. public function test_faq_piece_not_null_when_items_present(): void { $GLOBALS['pr_test_postmeta'][1] = [ @@ -233,7 +233,7 @@ class JsonldTest extends TestCase { $this->assertNull( $faq_piece ); } - // ── MedicalWebPage retype ───────────────────────────────────────── + // ── MedicalWebPage retype ─────────────────────────────────────────. public function test_retype_returns_medical_webpage_on_peptide_singular(): void { $GLOBALS['pr_test_is_singular'] = true; @@ -249,7 +249,7 @@ class JsonldTest extends TestCase { $this->assertSame( 'WebPage', $result ); } - // ── No duplicate nodes from our builders ────────────────────────── + // ── No duplicate nodes from our builders ──────────────────────────. public function test_builders_do_not_emit_webpage_type(): void { $GLOBALS['pr_test_postmeta'][1] = [ @@ -272,7 +272,7 @@ class JsonldTest extends TestCase { $this->assertNotContains( 'BreadcrumbList', $types ); } - // ── sanitize_faq_items() ────────────────────────────────────────── + // ── sanitize_faq_items() ──────────────────────────────────────────. public function test_sanitize_faq_items_valid_input(): void { $result = PR_Core_Schema_Sanitizers::sanitize_faq_items( wp_json_encode( [ [ 'question' => 'Q?', 'answer' => 'A.' ] ] ) ); diff --git a/tests/unit/JsonldWebpageTest.php b/tests/unit/JsonldWebpageTest.php index 085b347..b08c785 100644 --- a/tests/unit/JsonldWebpageTest.php +++ b/tests/unit/JsonldWebpageTest.php @@ -28,7 +28,7 @@ class JsonldWebpageTest extends TestCase { require_once PR_CORE_PLUGIN_DIR . 'includes/frontend/class-pr-core-jsonld-webpage.php'; } - // ── retype_to_medical_webpage: array input ──────────────────────── + // ── retype_to_medical_webpage: array input ────────────────────────. public function test_retype_non_peptide_singular_array_unchanged(): void { $GLOBALS['pr_test_is_singular'] = true; @@ -51,7 +51,7 @@ class JsonldWebpageTest extends TestCase { $this->assertSame( 'CollectionPage', $result ); } - // ── enrich_webpage_piece: @type as array injects enrichments ────── + // ── enrich_webpage_piece: @type as array injects enrichments ──────. public function test_enrich_webpage_piece_array_type_injects_last_reviewed(): void { $GLOBALS['pr_test_is_singular'] = true; diff --git a/tests/unit/Migration0004Test.php b/tests/unit/Migration0004Test.php index d6b9b7b..ba8135f 100644 --- a/tests/unit/Migration0004Test.php +++ b/tests/unit/Migration0004Test.php @@ -47,7 +47,7 @@ class Migration0004Test extends TestCase { $this->migration = new PR_Core_Migration_0004_Backfill_Peptide_Meta(); } - // ── Helper: call private method via reflection ──────────────────── + // ── Helper: call private method via reflection ────────────────────. /** * @param mixed[] $args @@ -59,7 +59,7 @@ class Migration0004Test extends TestCase { return $ref->invokeArgs( $obj, $args ); } - // ── parse_weight() ──────────────────────────────────────────────── + // ── parse_weight() ────────────────────────────────────────────────. public function test_parse_weight_strips_da_suffix(): void { $this->assertSame( 1419.5, $this->call_private( $this->migration, 'parse_weight', [ '1419.5 Da' ] ) ); @@ -85,7 +85,7 @@ class Migration0004Test extends TestCase { $this->assertSame( 800.123, $this->call_private( $this->migration, 'parse_weight', [ ' 800.123 da ' ] ) ); } - // ── parse_aliases_string() ──────────────────────────────────────── + // ── parse_aliases_string() ────────────────────────────────────────. public function test_parse_aliases_string_splits_csv(): void { $result = json_decode( @@ -111,7 +111,7 @@ class Migration0004Test extends TestCase { $this->assertCount( 1, $result ); } - // ── sanitize_molecular_formula() ────────────────────────────────── + // ── sanitize_molecular_formula() ──────────────────────────────────. public function test_sanitize_formula_valid_passes_unchanged(): void { $this->assertSame( 'C62H98N16O22', PR_Core_Schema_Sanitizers::sanitize_molecular_formula( 'C62H98N16O22' ) ); @@ -141,7 +141,7 @@ class Migration0004Test extends TestCase { $this->assertSame( '', PR_Core_Schema_Sanitizers::sanitize_molecular_formula( ' ' ) ); } - // ── Idempotency ─────────────────────────────────────────────────── + // ── Idempotency ───────────────────────────────────────────────────. public function test_backfill_post_skipped_when_all_keys_populated(): void { $GLOBALS['pr_test_postmeta'][1] = [ @@ -154,7 +154,7 @@ class Migration0004Test extends TestCase { $this->assertEmpty( $GLOBALS['pr_test_updated_meta'] ); } - // ── PSA source path ─────────────────────────────────────────────── + // ── PSA source path ───────────────────────────────────────────────. public function test_backfill_post_copies_psa_data(): void { $GLOBALS['pr_test_postmeta'][2] = [ @@ -220,7 +220,7 @@ class Migration0004Test extends TestCase { $this->assertCount( 2, $aliases ); } - // ── PubChem skip path ───────────────────────────────────────────── + // ── PubChem skip path ─────────────────────────────────────────────. public function test_backfill_post_returns_pubchem_skip_when_network_fails(): void { $mock_pubchem = new class extends PR_Core_Pubchem_Client { diff --git a/tests/unit/PeptideCptTest.php b/tests/unit/PeptideCptTest.php index 0e1284d..0214413 100644 --- a/tests/unit/PeptideCptTest.php +++ b/tests/unit/PeptideCptTest.php @@ -40,7 +40,7 @@ class PeptideCptTest extends TestCase { require_once PR_CORE_PLUGIN_DIR . 'includes/cpt/class-pr-core-peptide-cpt.php'; } - // ── Constants ───────────────────────────────────────────────────── + // ── Constants ─────────────────────────────────────────────────────. public function test_post_type_constant(): void { $this->assertSame( 'peptide', PR_Core_Peptide_CPT::POST_TYPE ); @@ -62,7 +62,7 @@ class PeptideCptTest extends TestCase { ); } - // ── CPT registration guard ──────────────────────────────────────── + // ── CPT registration guard ────────────────────────────────────────. public function test_register_post_type_noop_when_already_exists(): void { $GLOBALS['pr_core_test_state']['existing_post_types'] = [ 'peptide' ]; @@ -73,7 +73,7 @@ class PeptideCptTest extends TestCase { ); } - // ── CPT registration payload ────────────────────────────────────── + // ── CPT registration payload ──────────────────────────────────────. public function test_register_post_type_registers_peptide_slug(): void { PR_Core_Peptide_CPT::register_peptide_post_type(); @@ -177,7 +177,7 @@ class PeptideCptTest extends TestCase { ]; } - // ── Taxonomy guard ──────────────────────────────────────────────── + // ── Taxonomy guard ────────────────────────────────────────────────. public function test_register_taxonomies_noop_when_already_exists(): void { $GLOBALS['pr_core_test_state']['existing_taxonomies'] = [ 'peptide_category' ]; @@ -185,7 +185,7 @@ class PeptideCptTest extends TestCase { $this->assertEmpty( $GLOBALS['pr_core_test_state']['registered_taxonomies'] ); } - // ── Taxonomy registration payload ───────────────────────────────── + // ── Taxonomy registration payload ─────────────────────────────────. public function test_register_taxonomies_registers_peptide_category(): void { PR_Core_Peptide_CPT::register_taxonomies(); @@ -236,7 +236,7 @@ class PeptideCptTest extends TestCase { $this->assertSame( 'peptide-category', $tax['args']['rewrite']['slug'] ?? null ); } - // ── Hook registration ───────────────────────────────────────────── + // ── Hook registration ─────────────────────────────────────────────. public function test_register_hooks_adds_init_action(): void { $cpt = new PR_Core_Peptide_CPT(); diff --git a/tests/unit/VerificationScannerTest.php b/tests/unit/VerificationScannerTest.php index 4cfb051..19c8a11 100644 --- a/tests/unit/VerificationScannerTest.php +++ b/tests/unit/VerificationScannerTest.php @@ -30,13 +30,13 @@ class VerificationScannerTest extends TestCase { require_once PR_CORE_PLUGIN_DIR . 'includes/scanner/class-pr-core-verification-scanner.php'; } - // ── Helper ──────────────────────────────────────────────────────── + // ── Helper ────────────────────────────────────────────────────────. private function days_ago( int $days ): string { return gmdate( 'Y-m-d', time() - ( $days * DAY_IN_SECONDS ) ); } - // ── Empty last_verified ─────────────────────────────────────────── + // ── Empty last_verified ───────────────────────────────────────────. public function test_empty_last_verified_returns_overdue(): void { $this->assertSame( @@ -45,7 +45,7 @@ class VerificationScannerTest extends TestCase { ); } - // ── Medium velocity (default 180-day threshold) ─────────────────── + // ── Medium velocity (default 180-day threshold) ───────────────────. public function test_medium_160_days_is_current(): void { $this->assertSame( 'current', PR_Core_Verification_Scanner::compute_status( $this->days_ago( 160 ), 'medium' ) ); @@ -63,7 +63,7 @@ class VerificationScannerTest extends TestCase { $this->assertSame( 'overdue', PR_Core_Verification_Scanner::compute_status( $this->days_ago( 200 ), 'medium' ) ); } - // ── 90% boundary exactness ──────────────────────────────────────── + // ── 90% boundary exactness ────────────────────────────────────────. public function test_boundary_89_days_threshold_100_is_current(): void { $GLOBALS['pr_core_options'] = [ 'pr_core_default_threshold' => 100 ]; @@ -80,7 +80,7 @@ class VerificationScannerTest extends TestCase { $this->assertSame( 'overdue', PR_Core_Verification_Scanner::compute_status( $this->days_ago( 100 ), 'medium' ) ); } - // ── High velocity (60-day threshold from option) ────────────────── + // ── High velocity (60-day threshold from option) ──────────────────. public function test_high_50_days_is_current(): void { $GLOBALS['pr_core_options'] = [ 'pr_core_high_velocity_threshold' => 60 ]; @@ -97,7 +97,7 @@ class VerificationScannerTest extends TestCase { $this->assertSame( 'overdue', PR_Core_Verification_Scanner::compute_status( $this->days_ago( 61 ), 'high' ) ); } - // ── Low velocity (365-day constant — ignores option) ────────────── + // ── Low velocity (365-day constant — ignores option) ──────────────. public function test_low_300_days_is_current(): void { $GLOBALS['pr_core_options'] = [ 'pr_core_default_threshold' => 999 ]; @@ -114,7 +114,7 @@ class VerificationScannerTest extends TestCase { $this->assertSame( 'overdue', PR_Core_Verification_Scanner::compute_status( $this->days_ago( 366 ), 'low' ) ); } - // ── Unknown velocity falls to medium/default ────────────────────── + // ── Unknown velocity falls to medium/default ──────────────────────. public function test_unknown_velocity_uses_default_threshold(): void { $GLOBALS['pr_core_options'] = [ 'pr_core_default_threshold' => 180 ]; diff --git a/tests/unit/VerificationSettingsTest.php b/tests/unit/VerificationSettingsTest.php index 132afaf..c8c20f8 100644 --- a/tests/unit/VerificationSettingsTest.php +++ b/tests/unit/VerificationSettingsTest.php @@ -36,7 +36,7 @@ class VerificationSettingsTest extends TestCase { require_once PR_CORE_PLUGIN_DIR . 'includes/admin/class-pr-core-settings.php'; } - // ── sanitize_cadence() ──────────────────────────────────────────── + // ── sanitize_cadence() ────────────────────────────────────────────. public function test_sanitize_cadence_daily(): void { $this->assertSame( 'daily', PR_Core_Settings::sanitize_cadence( 'daily' ) ); @@ -62,7 +62,7 @@ class VerificationSettingsTest extends TestCase { $this->assertSame( 'weekly', PR_Core_Settings::sanitize_cadence( 'DAILY' ) ); } - // ── sanitize_emails() ───────────────────────────────────────────── + // ── sanitize_emails() ─────────────────────────────────────────────. public function test_sanitize_emails_empty_returns_empty(): void { $this->assertSame( '', PR_Core_Settings::sanitize_emails( '' ) ); @@ -88,14 +88,14 @@ class VerificationSettingsTest extends TestCase { $this->assertSame( 'trimmed@test.com', PR_Core_Settings::sanitize_emails( ' trimmed@test.com ' ) ); } - // ── reschedule_cron(): no-op when same value ────────────────────── + // ── reschedule_cron(): no-op when same value ──────────────────────. public function test_reschedule_cron_noop_when_value_unchanged(): void { PR_Core_Settings::reschedule_cron( 'weekly', 'weekly' ); $this->assertEmpty( $GLOBALS['pr_core_cron_calls'] ); } - // ── reschedule_cron(): clears + reschedules on change ───────────── + // ── reschedule_cron(): clears + reschedules on change ─────────────. public function test_reschedule_cron_calls_clear_when_changed(): void { PR_Core_Settings::reschedule_cron( 'weekly', 'daily' ); diff --git a/tests/unit/test-jsonld-peptide-piece.php b/tests/unit/test-jsonld-peptide-piece.php index b7668d3..e35f6c7 100644 --- a/tests/unit/test-jsonld-peptide-piece.php +++ b/tests/unit/test-jsonld-peptide-piece.php @@ -31,7 +31,7 @@ declare(strict_types=1); require_once __DIR__ . '/../bootstrap.php'; -// ── Additional stubs ───────────────────────────────────────────────────── +// ── Additional stubs ─────────────────────────────────────────────────────. if ( ! defined( 'PR_CORE_TARGET_SCHEMA_VERSION' ) ) { define( 'PR_CORE_TARGET_SCHEMA_VERSION', 4 ); @@ -129,7 +129,7 @@ if ( ! function_exists( 'sanitize_textarea_field' ) ) { } } -// ── Stub PR_Core_Peptide_Repository ────────────────────────────────────── +// ── Stub PR_Core_Peptide_Repository ──────────────────────────────────────. /** * Minimal repository stub — returns whatever is in $GLOBALS['pr_test_peptide_dto']. @@ -140,7 +140,7 @@ class PR_Core_Peptide_Repository { } } -// ── Load classes under test ─────────────────────────────────────────────── +// ── Load classes under test ───────────────────────────────────────────────. require_once PR_CORE_PLUGIN_DIR . 'includes/cpt/class-pr-core-schema-sanitizers.php'; require_once PR_CORE_PLUGIN_DIR . 'includes/cpt/class-pr-core-peptide-cpt.php'; @@ -150,7 +150,7 @@ require_once PR_CORE_PLUGIN_DIR . 'includes/frontend/class-pr-core-jsonld-faq.ph require_once PR_CORE_PLUGIN_DIR . 'includes/frontend/class-pr-core-jsonld-webpage.php'; require_once PR_CORE_PLUGIN_DIR . 'includes/frontend/class-pr-core-jsonld.php'; -// ── Helper ──────────────────────────────────────────────────────────────── +// ── Helper ────────────────────────────────────────────────────────────────. /** * Create a minimal PR_Core_Peptide_DTO for testing. @@ -184,7 +184,7 @@ function make_peptide_piece_dto( array $overrides = [] ): PR_Core_Peptide_DTO { echo "== JSON-LD peptide-piece regression tests (v0.6.2) ==\n\n"; -// ── Setup ───────────────────────────────────────────────────────────────── +// ── Setup ─────────────────────────────────────────────────────────────────. $GLOBALS['pr_test_the_id'] = 36; $GLOBALS['pr_test_is_singular'] = true; @@ -198,11 +198,11 @@ $GLOBALS['pr_test_peptide_dto'] = make_peptide_piece_dto(); $jsonld = new PR_Core_Jsonld(); -// ── Regression: inject_graph_pieces fatal reproduced against old contract ─ +// ── Regression: inject_graph_pieces fatal reproduced against old contract ─. echo "Regression: inject_graph_pieces (OLD hook) with plain array fatal:\n"; -// Simulate what Yoast's filter_graph_pieces_to_generate does: +// Simulate what Yoast's filter_graph_pieces_to_generate does:. // it calls get_class($piece) on every item. A plain array causes a fatal. $drug_builder = new PR_Core_Jsonld_Drug(); $drug_array = $drug_builder->build( $GLOBALS['pr_test_peptide_dto'] ); @@ -210,8 +210,8 @@ $drug_array = $drug_builder->build( $GLOBALS['pr_test_peptide_dto'] ); $caught_fatal = false; try { // This is the exact operation Yoast does in filter_graph_pieces_to_generate(). - // Against the old v0.6.1 code (inject via wpseo_schema_graph_pieces), this would - // produce: "get_class(): Argument #1 ($object) must be of type object, array given" + // Against the old v0.6.1 code (inject via wpseo_schema_graph_pieces), this would. + // produce: "get_class(): Argument #1 ($object) must be of type object, array given". $class_result = get_class( $drug_array ); } catch ( TypeError $e ) { $caught_fatal = true; @@ -220,7 +220,7 @@ try { } pr_assert( $caught_fatal, 'get_class(array) throws TypeError/Error — confirms the v0.6.1 fatal' ); -// ── Fix: inject_graph_nodes (NEW hook) works without fatal ──────────────── +// ── Fix: inject_graph_nodes (NEW hook) works without fatal ────────────────. echo "\nFix: inject_graph_nodes via wpseo_schema_graph (plain array graph):\n"; @@ -279,7 +279,7 @@ pr_assert( count( $webpage_nodes ) === 1, 'MedicalWebPage node preserved (no dup $breadcrumb_nodes = array_filter( $result_graph, static fn( $n ) => ( $n['@type'] ?? '' ) === 'BreadcrumbList' ); pr_assert( count( $breadcrumb_nodes ) === 1, 'BreadcrumbList node preserved (no duplicate)' ); -// ── FAQ: present when _pr_faq_items has items ───────────────────────────── +// ── FAQ: present when _pr_faq_items has items ─────────────────────────────. echo "\nFAQ injection via inject_graph_nodes:\n"; @@ -301,7 +301,7 @@ pr_assert_equals( 'https://peptiderepo.com/peptides/bpc-157/#faq', $faq_node['@i pr_assert( isset( $faq_node['mainEntity'] ), 'FAQPage has mainEntity' ); pr_assert_equals( 2, count( $faq_node['mainEntity'] ), 'FAQPage has 2 questions' ); -// ── FAQ: absent when _pr_faq_items empty ───────────────────────────────── +// ── FAQ: absent when _pr_faq_items empty ─────────────────────────────────. $GLOBALS['pr_test_postmeta'][36]['_pr_faq_items'] = '[]'; @@ -312,7 +312,7 @@ $faq_nodes_empty = array_filter( ); pr_assert( count( $faq_nodes_empty ) === 0, 'FAQPage NOT injected when _pr_faq_items is empty array' ); -// ── Non-peptide page: no injection ─────────────────────────────────────── +// ── Non-peptide page: no injection ───────────────────────────────────────. echo "\nNon-peptide page: no injection:\n"; @@ -332,7 +332,7 @@ pr_assert( count( $result_non_peptide ) === 1, 'inject_graph_nodes no-ops on non $drug_nodes_np = array_filter( $result_non_peptide, static fn( $n ) => ( is_array( $n['@type'] ?? null ) ? in_array( 'Drug', $n['@type'], true ) : ( $n['@type'] ?? '' ) === 'Drug' ) ); pr_assert( count( $drug_nodes_np ) === 0, 'Drug NOT injected on non-peptide page' ); -// ── register_hooks: does NOT hook wpseo_schema_graph_pieces ────────────── +// ── register_hooks: does NOT hook wpseo_schema_graph_pieces ──────────────. echo "\nregister_hooks: wpseo_schema_graph_pieces must NOT be hooked:\n"; @@ -341,11 +341,11 @@ $GLOBALS['pr_core_test_state']['added_filters'] = []; $filters_added = []; // Override add_filter in the bootstrap to capture calls. -// (bootstrap only stubs add_action; we check via a fresh parse of register_hooks.) +// (bootstrap only stubs add_action; we check via a fresh parse of register_hooks.). $jsonld2 = new PR_Core_Jsonld(); // Use Reflection to inspect what hooks register_hooks would call. -// Instead: inspect the class source for the string — if inject_graph_pieces is +// Instead: inspect the class source for the string — if inject_graph_pieces is. // still registered on wpseo_schema_graph_pieces, the bug is NOT fixed. $src = file_get_contents( PR_CORE_PLUGIN_DIR . 'includes/frontend/class-pr-core-jsonld.php' ); pr_assert( @@ -361,6 +361,6 @@ pr_assert( 'class-pr-core-jsonld.php hooks wpseo_schema_graph for injection' ); -// ── Summary ─────────────────────────────────────────────────────────────── +// ── Summary ───────────────────────────────────────────────────────────────. exit( pr_test_summary() ); diff --git a/tests/unit/test-jsonld-webpage.php b/tests/unit/test-jsonld-webpage.php index 1539792..1fb8212 100644 --- a/tests/unit/test-jsonld-webpage.php +++ b/tests/unit/test-jsonld-webpage.php @@ -21,7 +21,7 @@ declare(strict_types=1); require_once __DIR__ . '/../bootstrap.php'; -// ── Additional stubs ───────────────────────────────────────────────────── +// ── Additional stubs ─────────────────────────────────────────────────────. if ( ! defined( 'PR_CORE_TARGET_SCHEMA_VERSION' ) ) { define( 'PR_CORE_TARGET_SCHEMA_VERSION', 4 ); @@ -81,7 +81,7 @@ echo "== JSON-LD WebPage v0.6.1 regression tests ==\n\n"; $enricher = new PR_Core_Jsonld_Webpage(); -// ── retype_to_medical_webpage: array input, non-peptide singular ────────── +// ── retype_to_medical_webpage: array input, non-peptide singular ──────────. echo "retype_to_medical_webpage — array input:\n"; @@ -106,7 +106,7 @@ $GLOBALS['pr_test_singular_type'] = ''; $result = $enricher->retype_to_medical_webpage( 'CollectionPage' ); pr_assert_equals( 'CollectionPage', $result, 'non-singular string: passes through unchanged' ); -// ── enrich_webpage_piece: @type as array injects enrichments ───────────── +// ── enrich_webpage_piece: @type as array injects enrichments ─────────────. echo "\nenrich_webpage_piece — @type as array:\n"; @@ -160,6 +160,6 @@ $graph_non_peptide = [ $result_non_peptide = $enricher->enrich_webpage_piece( $graph_non_peptide, new stdClass() ); pr_assert( ! isset( $result_non_peptide[0]['lastReviewed'] ), 'non-peptide: lastReviewed NOT injected' ); -// ── Summary ────────────────────────────────────────────────────────────── +// ── Summary ──────────────────────────────────────────────────────────────. exit( pr_test_summary() ); diff --git a/tests/unit/test-jsonld.php b/tests/unit/test-jsonld.php index 978d7dd..951d0c1 100644 --- a/tests/unit/test-jsonld.php +++ b/tests/unit/test-jsonld.php @@ -29,7 +29,7 @@ declare(strict_types=1); require_once __DIR__ . '/../bootstrap.php'; -// ── Additional stubs ───────────────────────────────────────────────────── +// ── Additional stubs ─────────────────────────────────────────────────────. if ( ! defined( 'PR_CORE_TARGET_SCHEMA_VERSION' ) ) { define( 'PR_CORE_TARGET_SCHEMA_VERSION', 4 ); @@ -111,7 +111,7 @@ require_once PR_CORE_PLUGIN_DIR . 'includes/frontend/class-pr-core-jsonld-faq.ph require_once PR_CORE_PLUGIN_DIR . 'includes/frontend/class-pr-core-jsonld-webpage.php'; require_once PR_CORE_PLUGIN_DIR . 'includes/frontend/class-pr-core-jsonld.php'; -// ── Helpers ────────────────────────────────────────────────────────────── +// ── Helpers ──────────────────────────────────────────────────────────────. /** * Create a minimal PR_Core_Peptide_DTO for testing. @@ -145,7 +145,7 @@ function make_peptide_dto( array $overrides = [] ): PR_Core_Peptide_DTO { echo "== JSON-LD v0.6.0 unit tests ==\n\n"; -// ── Drug @id ───────────────────────────────────────────────────────────── +// ── Drug @id ─────────────────────────────────────────────────────────────. echo "Drug @id:\n"; @@ -160,14 +160,14 @@ pr_assert_equals( 'Drug @id = {permalink}#drug' ); -// ── Drug @type ─────────────────────────────────────────────────────────── +// ── Drug @type ───────────────────────────────────────────────────────────. echo "\nDrug @type:\n"; pr_assert( is_array( $drug_node['@type'] ), 'Drug @type is an array' ); pr_assert( in_array( 'Drug', $drug_node['@type'], true ), '@type includes "Drug"' ); pr_assert( in_array( 'MolecularEntity', $drug_node['@type'], true ), '@type includes "MolecularEntity"' ); -// ── Fields present when meta set ───────────────────────────────────────── +// ── Fields present when meta set ─────────────────────────────────────────. echo "\nFields present when _pr_* meta set:\n"; @@ -190,7 +190,7 @@ pr_assert_equals( 'g/mol', $drug_node_populated['molecularWeight']['unitText'], pr_assert( isset( $drug_node_populated['alternateName'] ), 'alternateName present when _pr_aliases set' ); pr_assert_equals( 2, count( $drug_node_populated['alternateName'] ), 'alternateName has correct count' ); -// ── Fields omitted when meta empty ─────────────────────────────────────── +// ── Fields omitted when meta empty ───────────────────────────────────────. echo "\nFields OMITTED when meta empty:\n"; @@ -201,7 +201,7 @@ pr_assert( ! isset( $drug_node_empty['molecularFormula'] ), 'molecularFormula OM pr_assert( ! isset( $drug_node_empty['molecularWeight'] ), 'molecularWeight OMITTED when meta empty' ); pr_assert( ! isset( $drug_node_empty['alternateName'] ), 'alternateName OMITTED when _pr_aliases empty' ); -// ── FAQPage present when _pr_faq_items has items ───────────────────────── +// ── FAQPage present when _pr_faq_items has items ─────────────────────────. echo "\nFAQPage emission:\n"; @@ -224,7 +224,7 @@ pr_assert_equals( 2, count( $faq_piece['mainEntity'] ), 'FAQPage has 2 questions pr_assert_equals( 'Question', $faq_piece['mainEntity'][0]['@type'], 'Question @type correct' ); pr_assert( isset( $faq_piece['mainEntity'][0]['acceptedAnswer'] ), 'Question has acceptedAnswer' ); -// ── FAQPage absent when _pr_faq_items empty ────────────────────────────── +// ── FAQPage absent when _pr_faq_items empty ──────────────────────────────. $GLOBALS['pr_test_postmeta'][1] = [ '_pr_faq_items' => '[]' ]; $faq_empty = $faq_builder->build( 1, $permalink ); @@ -234,7 +234,7 @@ $GLOBALS['pr_test_postmeta'][1] = []; $faq_absent = $faq_builder->build( 1, $permalink ); pr_assert( null === $faq_absent, 'FAQPage piece IS null when _pr_faq_items absent' ); -// ── MedicalWebPage retype ──────────────────────────────────────────────── +// ── MedicalWebPage retype ────────────────────────────────────────────────. echo "\nMedicalWebPage retype:\n"; @@ -250,7 +250,7 @@ $GLOBALS['pr_test_singular_type'] = ''; $result = $webpage_enricher->retype_to_medical_webpage( 'WebPage' ); pr_assert_equals( 'WebPage', $result, 'passes through unchanged on non-peptide pages' ); -// ── No duplicate WebPage / BreadcrumbList nodes ─────────────────────────── +// ── No duplicate WebPage / BreadcrumbList nodes ───────────────────────────. echo "\nNo duplicate WebPage/BreadcrumbList nodes:\n"; @@ -270,7 +270,7 @@ pr_assert( ! in_array( 'BreadcrumbList', $types_from_builders, true ), 'Breadcru pr_assert( in_array( 'Drug', $types_from_builders, true ), 'Drug IS in our emitted pieces' ); pr_assert( in_array( 'FAQPage', $types_from_builders, true ), 'FAQPage IS in our emitted pieces when items present' ); -// ── PR_Core_Schema_Sanitizers::sanitize_faq_items() ────────────────────── +// ── PR_Core_Schema_Sanitizers::sanitize_faq_items() ──────────────────────. echo "\nPR_Core_Schema_Sanitizers::sanitize_faq_items():\n"; @@ -294,6 +294,6 @@ $result_incomplete = PR_Core_Schema_Sanitizers::sanitize_faq_items( $incomplete_ $decoded_incomplete = json_decode( $result_incomplete, true ); pr_assert_equals( 0, count( $decoded_incomplete ), 'item without answer field dropped' ); -// ── Summary ────────────────────────────────────────────────────────────── +// ── Summary ──────────────────────────────────────────────────────────────. exit( pr_test_summary() ); diff --git a/tests/unit/test-migration-0004.php b/tests/unit/test-migration-0004.php index 82562ee..7ad5627 100644 --- a/tests/unit/test-migration-0004.php +++ b/tests/unit/test-migration-0004.php @@ -27,7 +27,7 @@ declare(strict_types=1); // Bootstrap the lightweight WP stubs. require_once __DIR__ . '/../bootstrap.php'; -// ── Additional stubs needed for migration 0004 ────────────────────────── +// ── Additional stubs needed for migration 0004 ──────────────────────────. if ( ! defined( 'PR_CORE_TARGET_SCHEMA_VERSION' ) ) { define( 'PR_CORE_TARGET_SCHEMA_VERSION', 4 ); @@ -85,7 +85,7 @@ require_once PR_CORE_PLUGIN_DIR . 'includes/migrations/class-pr-core-pubchem-cli // Load migration class under test. require_once PR_CORE_PLUGIN_DIR . 'includes/migrations/class-pr-core-migration-0004-backfill-peptide-meta.php'; -// ── Expose private methods via reflection ──────────────────────────────── +// ── Expose private methods via reflection ────────────────────────────────. /** * Call a private/protected method on an object via reflection. @@ -105,7 +105,7 @@ $migration = new PR_Core_Migration_0004_Backfill_Peptide_Meta(); echo "== PR_Core_Migration_0004 unit tests ==\n\n"; -// ── parse_weight() ─────────────────────────────────────────────────────── +// ── parse_weight() ───────────────────────────────────────────────────────. echo "parse_weight():\n"; @@ -145,7 +145,7 @@ pr_assert_equals( 'strips whitespace and lowercase "da"' ); -// ── parse_aliases_string() ─────────────────────────────────────────────── +// ── parse_aliases_string() ───────────────────────────────────────────────. echo "\nparse_aliases_string():\n"; @@ -167,8 +167,8 @@ pr_assert_equals( '[]', $whitespace_result, 'whitespace-only segments dropped, r $single = json_decode( $migration->parse_aliases_string( 'Only One' ), true ); pr_assert_equals( 1, count( $single ), 'single alias produces array of 1' ); -// ── sanitize_formula() ─────────────────────────────────────────────────── -// Five required cases (Option B — validate, don't allowlist): +// ── sanitize_formula() ───────────────────────────────────────────────────. +// Five required cases (Option B — validate, don't allowlist):. // 1. Valid formula unchanged. // 2. Parenthesised formula preserved. // 3. Newline-injected payload — "injection" stripped, formula kept. @@ -219,7 +219,7 @@ pr_assert_equals( 'whitespace-only returns empty string' ); -// ── Idempotency: skips when all three PR Core keys populated ───────────── +// ── Idempotency: skips when all three PR Core keys populated ─────────────. echo "\nIdempotency (all _pr_* keys populated):\n"; @@ -235,7 +235,7 @@ $result = call_private( $migration, 'backfill_post', [ 1 ] ); pr_assert_equals( 'skipped', $result, 'returns "skipped" when all three keys non-empty' ); pr_assert( empty( $GLOBALS['pr_test_updated_meta'] ), 'no update_post_meta calls when skipped' ); -// ── PSA source path ────────────────────────────────────────────────────── +// ── PSA source path ──────────────────────────────────────────────────────. echo "\nPSA source path:\n"; @@ -277,7 +277,7 @@ pr_assert( $formula_written, 'molecular formula meta was written' ); pr_assert( $weight_written, 'molecular weight meta was written' ); pr_assert( $aliases_written, 'aliases meta was written' ); -// ── Partial re-run: existing values preserved ──────────────────────────── +// ── Partial re-run: existing values preserved ────────────────────────────. echo "\nIdempotency: only some keys populated (partial re-run):\n"; @@ -297,7 +297,7 @@ $GLOBALS['pr_test_posts'][3] = (object) [ 'post_title' => 'BPC-157', 'ID' => 3 ] $result = call_private( $migration, 'backfill_post', [ 3 ] ); pr_assert_equals( 'copied_psa', $result, 'runs PSA copy even when only some keys populated' ); -// ── PubChem skip path: no PSA data, no network ────────────────────────── +// ── PubChem skip path: no PSA data, no network ──────────────────────────. echo "\nPubChem skip path (no PSA data, no network in unit tests):\n"; @@ -317,6 +317,6 @@ $result = call_private( $mock_migration, 'backfill_post', [ 10 ] ); pr_assert_equals( 'pubchem_skip', $result, 'returns "pubchem_skip" when PubChem network fails' ); pr_assert( empty( $GLOBALS['pr_test_updated_meta'] ), 'no meta written on pubchem_skip' ); -// ── Summary ────────────────────────────────────────────────────────────── +// ── Summary ──────────────────────────────────────────────────────────────. exit( pr_test_summary() ); diff --git a/tests/unit/test-verification-scanner.php b/tests/unit/test-verification-scanner.php index 369dd67..4127b4f 100644 --- a/tests/unit/test-verification-scanner.php +++ b/tests/unit/test-verification-scanner.php @@ -22,7 +22,7 @@ declare(strict_types=1); require_once __DIR__ . '/../bootstrap.php'; -// ── Additional stubs needed by the scanner ────────────────────────────── +// ── Additional stubs needed by the scanner ──────────────────────────────. if ( ! defined( 'DAY_IN_SECONDS' ) ) { define( 'DAY_IN_SECONDS', 86400 ); @@ -37,11 +37,11 @@ if ( ! function_exists( 'get_option' ) ) { } } -// ── Load the scanner class ────────────────────────────────────────────── +// ── Load the scanner class ──────────────────────────────────────────────. require_once __DIR__ . '/../../includes/scanner/class-pr-core-verification-scanner.php'; -// ── Helper: build a date N days ago ──────────────────────────────────── +// ── Helper: build a date N days ago ────────────────────────────────────. function days_ago( int $days ): string { return gmdate( 'Y-m-d', time() - ( $days * DAY_IN_SECONDS ) ); @@ -49,7 +49,7 @@ function days_ago( int $days ): string { echo "== PR_Core_Verification_Scanner::compute_status() unit tests ==\n\n"; -// ── Empty last_verified ───────────────────────────────────────────────── +// ── Empty last_verified ─────────────────────────────────────────────────. echo "Empty last_verified:\n"; pr_assert_equals( @@ -58,7 +58,7 @@ pr_assert_equals( 'empty string → overdue' ); -// ── Medium velocity (default 180-day threshold) ───────────────────────── +// ── Medium velocity (default 180-day threshold) ─────────────────────────. echo "\nMedium velocity (threshold=180, option default):\n"; $GLOBALS['pr_core_options'] = []; @@ -75,7 +75,7 @@ pr_assert_equals( 'overdue', PR_Core_Verification_Scanner::compute_status( days_ // 200 days ago → overdue. pr_assert_equals( 'overdue', PR_Core_Verification_Scanner::compute_status( days_ago( 200 ), 'medium' ), '200 days → overdue' ); -// ── 90% boundary exactness ────────────────────────────────────────────── +// ── 90% boundary exactness ──────────────────────────────────────────────. echo "\n90% boundary (threshold=100):\n"; $GLOBALS['pr_core_options'] = [ 'pr_core_default_threshold' => 100 ]; @@ -89,7 +89,7 @@ pr_assert_equals( 'due', PR_Core_Verification_Scanner::compute_status( days_ago( // 100 days → 100 >= 100 → overdue. pr_assert_equals( 'overdue', PR_Core_Verification_Scanner::compute_status( days_ago( 100 ), 'medium' ), '100 days (threshold 100) → overdue' ); -// ── High velocity (60-day threshold from option) ──────────────────────── +// ── High velocity (60-day threshold from option) ────────────────────────. echo "\nHigh velocity (threshold=60 from option):\n"; $GLOBALS['pr_core_options'] = [ 'pr_core_high_velocity_threshold' => 60 ]; @@ -103,7 +103,7 @@ pr_assert_equals( 'due', PR_Core_Verification_Scanner::compute_status( days_ago( // 61 days → overdue. pr_assert_equals( 'overdue', PR_Core_Verification_Scanner::compute_status( days_ago( 61 ), 'high' ), 'high: 61 days → overdue' ); -// ── Low velocity (365-day constant — ignores option) ──────────────────── +// ── Low velocity (365-day constant — ignores option) ────────────────────. echo "\nLow velocity (threshold=365, constant, ignores option):\n"; $GLOBALS['pr_core_options'] = [ 'pr_core_default_threshold' => 999 ]; // Should be ignored for low. @@ -117,12 +117,12 @@ pr_assert_equals( 'due', PR_Core_Verification_Scanner::compute_status( days_ago( // 366 days → overdue. pr_assert_equals( 'overdue', PR_Core_Verification_Scanner::compute_status( days_ago( 366 ), 'low' ), 'low: 366 days → overdue' ); -// ── Unknown velocity falls through to medium/default ──────────────────── +// ── Unknown velocity falls through to medium/default ────────────────────. echo "\nUnknown velocity (falls to default/medium):\n"; $GLOBALS['pr_core_options'] = [ 'pr_core_default_threshold' => 180 ]; pr_assert_equals( 'current', PR_Core_Verification_Scanner::compute_status( days_ago( 100 ), 'unknown' ), 'unknown velocity: 100 days → current (uses default 180)' ); -// ── Summary ───────────────────────────────────────────────────────────── +// ── Summary ─────────────────────────────────────────────────────────────. exit( pr_test_summary() ); diff --git a/tests/unit/test-verification-settings.php b/tests/unit/test-verification-settings.php index b3cf0f6..9a1a9c7 100644 --- a/tests/unit/test-verification-settings.php +++ b/tests/unit/test-verification-settings.php @@ -18,7 +18,7 @@ declare(strict_types=1); require_once __DIR__ . '/../bootstrap.php'; -// ── Additional stubs needed by the settings class ─────────────────────── +// ── Additional stubs needed by the settings class ───────────────────────. $GLOBALS['pr_core_options'] = []; $GLOBALS['pr_core_cron_calls'] = []; @@ -66,14 +66,14 @@ if ( ! function_exists( 'wp_schedule_event' ) ) { } } -// ── Load the settings class ───────────────────────────────────────────── +// ── Load the settings class ─────────────────────────────────────────────. require_once __DIR__ . '/../../includes/cpt/class-pr-core-peptide-cpt.php'; require_once __DIR__ . '/../../includes/admin/class-pr-core-settings.php'; echo "== PR_Core_Settings verification methods unit tests ==\n\n"; -// ── sanitize_cadence() ────────────────────────────────────────────────── +// ── sanitize_cadence() ──────────────────────────────────────────────────. echo "sanitize_cadence():\n"; pr_assert_equals( 'daily', PR_Core_Settings::sanitize_cadence( 'daily' ), 'daily → daily' ); @@ -83,7 +83,7 @@ pr_assert_equals( 'weekly', PR_Core_Settings::sanitize_cadence( 'hourly' ), 'h pr_assert_equals( 'weekly', PR_Core_Settings::sanitize_cadence( '' ), 'empty → weekly default' ); pr_assert_equals( 'weekly', PR_Core_Settings::sanitize_cadence( 'DAILY' ), 'uppercase → weekly (case-sensitive)' ); -// ── sanitize_emails() ─────────────────────────────────────────────────── +// ── sanitize_emails() ───────────────────────────────────────────────────. echo "\nsanitize_emails():\n"; pr_assert_equals( '', PR_Core_Settings::sanitize_emails( '' ), 'empty → empty' ); @@ -93,14 +93,14 @@ pr_assert_equals( 'a@test.com, b@test.com', PR_Core_Settings::sanitize_emails( ' pr_assert_equals( 'good@test.com', PR_Core_Settings::sanitize_emails( 'good@test.com, bad-email' ), 'mixed → only valid kept' ); pr_assert_equals( 'trimmed@test.com', PR_Core_Settings::sanitize_emails( ' trimmed@test.com ' ), 'whitespace trimmed around valid email' ); -// ── reschedule_cron(): no-op when same value ──────────────────────────── +// ── reschedule_cron(): no-op when same value ────────────────────────────. echo "\nreschedule_cron() — no-op when value unchanged:\n"; $GLOBALS['pr_core_cron_calls'] = []; PR_Core_Settings::reschedule_cron( 'weekly', 'weekly' ); pr_assert( empty( $GLOBALS['pr_core_cron_calls'] ), 'no cron calls when old === new' ); -// ── reschedule_cron(): clears + reschedules on change ─────────────────── +// ── reschedule_cron(): clears + reschedules on change ───────────────────. echo "\nreschedule_cron() — reschedules when value changes:\n"; $GLOBALS['pr_core_cron_calls'] = []; @@ -126,6 +126,6 @@ $clear_idx = array_search( 'clear', $actions, true ); $schedule_idx = array_search( 'schedule', $actions, true ); pr_assert( $clear_idx < $schedule_idx, 'clear fires before schedule' ); -// ── Summary ───────────────────────────────────────────────────────────── +// ── Summary ─────────────────────────────────────────────────────────────. exit( pr_test_summary() ); diff --git a/uninstall.php b/uninstall.php index 41496fa..ea12ebc 100644 --- a/uninstall.php +++ b/uninstall.php @@ -8,6 +8,7 @@ * peptide_category taxonomy terms) is preserved. * * @see ARCHITECTURE.md — §2.9 Uninstall specification. + * @package Peptide_Repo_Core */ if ( ! defined( 'WP_UNINSTALL_PLUGIN' ) ) { @@ -26,7 +27,8 @@ $tables = array( foreach ( $tables as $table ) { // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery, WordPress.DB.DirectDatabaseQuery.SchemaChange - $wpdb->query( "DROP TABLE IF EXISTS {$table}" ); + // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared,WordPress.DB.DirectDatabaseQuery.SchemaChange,WordPress.DB.DirectDatabaseQuery.DirectQuery + $wpdb->query( "DROP TABLE IF EXISTS {$table}" ); // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared } /* ── 2. Delete only peptide posts authored through PR Core's UI ───────── */ @@ -44,7 +46,7 @@ foreach ( $tables as $table ) { // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery $post_ids = $wpdb->get_col( $wpdb->prepare( - "SELECT p.ID FROM {$wpdb->posts} p + "SELECT p.ID FROM {$wpdb->posts} p // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared INNER JOIN {$wpdb->postmeta} pm ON p.ID = pm.post_id AND pm.meta_key = %s AND pm.meta_value = %s WHERE p.post_type = %s", @@ -54,7 +56,7 @@ $post_ids = $wpdb->get_col( ) ); -foreach ( $post_ids as $post_id ) { +foreach ( $post_ids as $post_id ) { // phpcs:ignore WordPress.WP.GlobalVariablesOverride.Prohibited wp_delete_post( (int) $post_id, true ); } @@ -72,8 +74,9 @@ foreach ( $post_ids as $post_id ) { /* ── 4. Delete all pr_core_ prefixed options ──────────────────────────── */ // phpcs:ignore WordPress.DB.DirectDatabaseQuery.DirectQuery +// phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared,WordPress.DB.DirectDatabaseQuery.NoCaching $wpdb->query( - "DELETE FROM {$wpdb->options} WHERE option_name LIKE 'pr\_core\_%'" + "DELETE FROM {$wpdb->options} WHERE option_name LIKE 'pr\_core\_%'" // phpcs:ignore WordPress.DB.PreparedSQL.InterpolatedNotPrepared ); /* ── 4b. Delete v0.6.0 schema-input meta keys from all peptide posts ──── */ @@ -106,7 +109,7 @@ foreach ( $schema_meta_keys as $meta_key ) { $editable_roles = wp_roles()->roles; foreach ( array_keys( $editable_roles ) as $role_name ) { - $role = get_role( $role_name ); + $role = get_role( $role_name ); // phpcs:ignore WordPress.WP.GlobalVariablesOverride.Prohibited if ( $role && $role->has_cap( 'manage_peptide_content' ) ) { $role->remove_cap( 'manage_peptide_content' ); }