admin/CC-Switch
1
0
Fork 0
mirror of https://github.com/farion1231/cc-switch.git synced 2026-08-04 19:45:34 +08:00
Code Issues Packages Projects Releases Wiki Activity

test(pi): decouple portable import coverage from restore

Browse Source
This commit is contained in:
SaladDay
2026-08-03 09:26:23 +00:00
parent 9d71e355da
commit 9ab526dabf
1 changed files with 23 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src-tauri/src/services/pi_catalog.rs
+23 -7
View File
@@ -429,12 +429,13 @@ impl PiCatalogCoordinator {
// fail-before-write error rather than a partially compensated publish. // fail-before-write error rather than a partially compensated publish.
let previous_defaults = read_pi_native_defaults()?; let previous_defaults = read_pi_native_defaults()?;
// Portable restore replaces provider rows but deliberately preserves // A portable import can leave device-local exact-key claims behind
// the device-local exact-key ledger. Claims whose providers disappeared // (for example, when the restore implementation preserves local-only
// must be released before publishing a runtime, otherwise the manifest // tables). Claims whose providers disappeared must be released before
// and provider aggregate sets can never converge. Their native values // publishing a runtime, otherwise the manifest and provider aggregate
// remain untouched and become user-owned; without a retained provider // sets can never converge. Their native values remain untouched and
// aggregate there is no safe expected value with which to delete them. // become user-owned; without a retained provider aggregate there is no
// safe expected value with which to delete them.
let orphaned_claims = manifest let orphaned_claims = manifest
.values() .values()
.filter(|projection| !providers.contains_key(&projection.provider_id)) .filter(|projection| !providers.contains_key(&projection.provider_id))
@@ -1703,9 +1704,14 @@ mod tests {
target.get_provider_aggregate(PI_APP, "local")?.is_none(), target.get_provider_aggregate(PI_APP, "local")?.is_none(),
"portable provider rows must be replaced" "portable provider rows must be replaced"
); );
// Canonical restore may preserve this device-local table, while the
// legacy importer rebuilds it empty. Seed the post-import state
// explicitly so this Pi-layer test continues to certify the invariant
// without making Canonical Restore a prerequisite of Pi support.
target.claim_pi_projection_key("local", "local-native")?;
assert!( assert!(
target.get_pi_projection("local")?.is_some(), target.get_pi_projection("local")?.is_some(),
"the device-local claim is intentionally preserved by restore" "the reconciliation input must contain an orphan device-local claim"
); );
let state = AppState::new(target.clone()); let state = AppState::new(target.clone());
@@ -1752,6 +1758,16 @@ mod tests {
#[test] #[test]
fn portable_import_of_empty_pi_catalog_releases_all_device_claims() -> Result<(), AppError> { fn portable_import_of_empty_pi_catalog_releases_all_device_claims() -> Result<(), AppError> {
let empty_source = Database::memory()?; let empty_source = Database::memory()?;
{
let conn = crate::database::lock_conn!(empty_source.conn);
conn.execute(
"INSERT INTO providers
(id, app_type, name, settings_config, meta, is_current)
VALUES ('non-pi-sentinel', 'claude', 'Non-Pi sentinel', '{}', '{}', 0)",
[],
)
.map_err(|error| AppError::Database(error.to_string()))?;
}
let exported = empty_source.export_sql_string()?; let exported = empty_source.export_sql_string()?;
let target = Arc::new(Database::memory()?); let target = Arc::new(Database::memory()?);
insert_portable_pi_provider(&target, "local")?; insert_portable_pi_provider(&target, "local")?;