v0.3.4-pre.002

crates/ksp-store-postgres-lib/unit_tests/migration.rs

@@ -1,5 +1,5 @@
 // file: crates/ksp-store-postgres-lib/unit_tests/migration.rs
-// version: 5
+// version: 6
 
 fn applied(version: i64, name: &str, checksum: &str) -> super::AppliedMigration {
     return super::AppliedMigration { checksum: checksum.to_owned(), name: name.to_owned(), version };
@@ -17,7 +17,7 @@ fn embedded(version: i64, name: &'static str, resources: &'static [crate::Schema
 
 #[test]
 fn pre_003_fix_001_embedded_registry_keeps_v000_checksum_and_uses_resource_owned_v001() {
-    assert_eq!(super::EMBEDDED_MIGRATIONS.len(), 2);
+    assert!(super::EMBEDDED_MIGRATIONS.len() >= 2);
     let v000 = &super::EMBEDDED_MIGRATIONS[0];
     assert_eq!(v000.version, 0);
     assert_eq!(v000.name, "bootstrap");
@@ -32,7 +32,29 @@ fn pre_003_fix_001_embedded_registry_keeps_v000_checksum_and_uses_resource_owned
     assert_eq!(v001.resources.len(), 40);
     assert_eq!(super::migration_checksum(v001), "31488cda2f08f3f46c4cdbdbb6c18c243662fada02eac4487040c8735d72cc51");
     assert!(super::validate_embedded_registry(super::EMBEDDED_MIGRATIONS).is_ok());
-    assert_eq!(crate::current_migration_version(), 1);
+    return;
+}
+
+#[test]
+fn pre_002_v002_registry_is_additive_minimal_and_keeps_v000_v001_checksums_stable() {
+    assert_eq!(super::EMBEDDED_MIGRATIONS.len(), 3);
+    let v000 = &super::EMBEDDED_MIGRATIONS[0];
+    let v001 = &super::EMBEDDED_MIGRATIONS[1];
+    let v002 = &super::EMBEDDED_MIGRATIONS[2];
+    assert_eq!(super::migration_checksum(v000), "d29068b8c13b9dc0cc9ef6aaadd0fa12d41e0fe4c56541a1118c4bfc846a1450");
+    assert_eq!(super::migration_checksum(v001), "31488cda2f08f3f46c4cdbdbb6c18c243662fada02eac4487040c8735d72cc51");
+    assert_eq!(v002.version, 2);
+    assert_eq!(v002.name, "raw_account_state");
+    assert_eq!(v002.hook, super::MigrationHook::None);
+    assert_eq!(v002.resources.len(), 5);
+    assert!(super::validate_embedded_registry(super::EMBEDDED_MIGRATIONS).is_ok());
+    assert_eq!(crate::current_migration_version(), 2);
+    let full = [
+        applied(0, v000.name, super::migration_checksum(v000).as_str()),
+        applied(1, v001.name, super::migration_checksum(v001).as_str()),
+        applied(2, v002.name, super::migration_checksum(v002).as_str()),
+    ];
+    assert_eq!(super::validate_history(&full, super::EMBEDDED_MIGRATIONS).ok(), std::option::Option::Some(3));
     return;
 }
 

crates/ksp-store-postgres-lib/unit_tests/schema.rs

@@ -1,5 +1,5 @@
 // file: crates/ksp-store-postgres-lib/unit_tests/schema.rs
-// version: 4
+// version: 5
 
 fn actual_column(name: &str, udt_name: &str, nullable: bool) -> super::ActualColumn {
     return super::ActualColumn {
@@ -48,6 +48,37 @@ fn pre_003_fix_001_v001_resources_are_split_and_idempotent_by_object_family() {
     return;
 }
 
+#[test]
+fn pre_002_v002_resources_are_exactly_two_tables_plus_base_pk_fk_without_indexes_or_domain_checks() {
+    assert_eq!(crate::V002_RESOURCES.len(), 5);
+    let ids = crate::V002_RESOURCES.iter().map(|resource| return resource.id).collect::<std::vec::Vec<_>>();
+    assert_eq!(ids[..2], ["tables/001_ksp_raw_account_states.sql", "tables/002_ksp_raw_account_observations.sql"]);
+    assert_eq!(
+        ids[2..],
+        [
+            "constraints/001_pk_ksp_raw_account_states.sql",
+            "constraints/002_pk_ksp_raw_account_observations.sql",
+            "constraints/003_fk_ksp_raw_account_observations_state.sql",
+        ]
+    );
+    let sql = crate::V002_RESOURCES.iter().map(|resource| return resource.sql).collect::<std::vec::Vec<_>>().concat();
+    for required in [
+        "CREATE TABLE IF NOT EXISTS ksp_raw_account_states",
+        "CREATE TABLE IF NOT EXISTS ksp_raw_account_observations",
+        "slot NUMERIC(20, 0) NOT NULL",
+        "write_version NUMERIC(20, 0) NULL",
+        "PRIMARY KEY (pubkey, slot, state_hash)",
+        "PRIMARY KEY (observation_key)",
+        "FOREIGN KEY (account_pubkey, account_slot, account_state_hash) REFERENCES ksp_raw_account_states(pubkey, slot, state_hash) ON DELETE RESTRICT",
+    ] {
+        assert!(sql.contains(required), "V002 pre.002 physical foundation is missing: {required}");
+    }
+    for forbidden in ["CREATE INDEX", " CHECK ", "ON DELETE CASCADE", "BIGSERIAL", "slot BIGINT", "data TEXT"] {
+        assert!(!sql.contains(forbidden), "pre.002 advanced V002 beyond the minimal table/PK/FK scope: {forbidden}");
+    }
+    return;
+}
+
 #[test]
 fn pre_003_fix_001_external_extra_columns_are_accepted_only_when_they_cannot_block_ksp_inserts() {
     let nullable = actual_column("external_nullable", "text", true);