97129ed457
New class PRAutoBlogger_Migrate_Remove_Cloudflare_V0100 (own file so class-activator.php stays under 300 lines). Runs from PRAutoBlogger::maybe_run_db_migrations() right after the v0.9.0 migration. Idempotent via prautoblogger_migrated_remove_cloudflare_v0100 flag option (picked up by the uninstall prefix sweep). Migrates any user on a Cloudflare image provider to Runware FLUX.1 schnell (closest equivalent, ~bash.0006/image). Detects the three shapes a legacy user can be in: registry-derived provider === 'cloudflare', image_provider option === 'cloudflare', or the legacy slug 'flux-1-schnell'. Cleans up three now-dead options: prautoblogger_cloudflare_ai_token, prautoblogger_cloudflare_account_id, prautoblogger_cf_image_via_gateway. Does NOT touch prautoblogger_ai_gateway_base_url — that option is shared with the OpenRouter AI Gateway route and must stay (live- verified test). Tests (tests/unit/Core/ActivatorRemoveCloudflareTest.php): - cloudflare user → runware schnell - non-cloudflare user preserved - runware user preserved - legacy 'flux-1-schnell' slug triggers migration - second run is a no-op (does not clobber admin manual changes) - AI Gateway URL preserved across every path ImageModelRegistryTest updated: allowed-provider set is now ['runware','openrouter']; new test_cloudflare_entries_removed_v0100 test pins the scope boundary.
103 lines
3.3 KiB
PHP
103 lines
3.3 KiB
PHP
<?php
|
|
/**
|
|
* Tests for PRAutoBlogger_Image_Model_Registry.
|
|
*
|
|
* The admin save flow (v0.8.0) collapses Provider and Image Model into a
|
|
* single dropdown and derives the provider from the model via
|
|
* Image_Model_Registry::provider_for(). This test pins that mapping down
|
|
* so a registry edit that breaks the contract fails CI.
|
|
*
|
|
* @package PRAutoBlogger\Tests\Services
|
|
*/
|
|
|
|
namespace PRAutoBlogger\Tests\Services;
|
|
|
|
use PRAutoBlogger\Tests\BaseTestCase;
|
|
|
|
class ImageModelRegistryTest extends BaseTestCase {
|
|
|
|
/**
|
|
* Every entry returned by get_models() must carry a non-empty id and
|
|
* a provider of 'runware' or 'openrouter' (Cloudflare Workers AI was
|
|
* removed as an image provider in v0.10.0). If this breaks, the
|
|
* admin save flow will silently drop the provider derivation.
|
|
*/
|
|
public function test_every_registry_entry_has_id_and_known_provider(): void {
|
|
$models = \PRAutoBlogger_Image_Model_Registry::get_models();
|
|
$this->assertNotEmpty( $models );
|
|
|
|
foreach ( $models as $model ) {
|
|
$this->assertArrayHasKey( 'id', $model );
|
|
$this->assertNotEmpty( $model['id'] );
|
|
$this->assertArrayHasKey( 'provider', $model );
|
|
$this->assertContains(
|
|
$model['provider'],
|
|
[ 'runware', 'openrouter' ],
|
|
sprintf( 'Unexpected provider %s for model %s', (string) $model['provider'], (string) $model['id'] )
|
|
);
|
|
}
|
|
}
|
|
|
|
/**
|
|
* provider_for() returns the paired provider for a known model id.
|
|
* This is the core of the v0.8.0 collapsed-dropdown contract.
|
|
*/
|
|
public function test_provider_for_returns_paired_provider_for_known_model(): void {
|
|
$this->assertSame(
|
|
'openrouter',
|
|
\PRAutoBlogger_Image_Model_Registry::provider_for( 'google/gemini-2.5-flash-image' )
|
|
);
|
|
$this->assertSame(
|
|
'runware',
|
|
\PRAutoBlogger_Image_Model_Registry::provider_for( 'runware:100@1' )
|
|
);
|
|
}
|
|
|
|
/**
|
|
* v0.10.0 scope boundary: no Cloudflare entries remain in the registry.
|
|
* The legacy `flux-1-schnell` slug (pre-Runware) must resolve to '' so
|
|
* the migration's detection path catches it. If a registry regression
|
|
* re-adds a cloudflare entry this test fails loudly.
|
|
*/
|
|
public function test_cloudflare_entries_removed_v0100(): void {
|
|
foreach ( \PRAutoBlogger_Image_Model_Registry::get_models() as $model ) {
|
|
$this->assertNotSame( 'cloudflare', $model['provider'] ?? '' );
|
|
}
|
|
$this->assertSame(
|
|
'',
|
|
\PRAutoBlogger_Image_Model_Registry::provider_for( 'flux-1-schnell' )
|
|
);
|
|
}
|
|
|
|
/**
|
|
* Runware model ids (added v0.9.0) resolve to the 'runware' provider.
|
|
* This test pins the v0.9.0 default flip so a registry regression
|
|
* that loses the Runware entries fails CI loudly.
|
|
*/
|
|
public function test_provider_for_runware_models(): void {
|
|
$this->assertSame(
|
|
'runware',
|
|
\PRAutoBlogger_Image_Model_Registry::provider_for( 'runware:100@1' )
|
|
);
|
|
$this->assertSame(
|
|
'runware',
|
|
\PRAutoBlogger_Image_Model_Registry::provider_for( 'runware:101@1' )
|
|
);
|
|
}
|
|
|
|
/**
|
|
* Unknown model ids return an empty string so callers can distinguish
|
|
* "not in registry" from a valid provider and show a settings error.
|
|
*/
|
|
public function test_provider_for_returns_empty_for_unknown_model(): void {
|
|
$this->assertSame(
|
|
'',
|
|
\PRAutoBlogger_Image_Model_Registry::provider_for( 'nonexistent/model' )
|
|
);
|
|
$this->assertSame(
|
|
'',
|
|
\PRAutoBlogger_Image_Model_Registry::provider_for( '' )
|
|
);
|
|
}
|
|
}
|