-
Notifications
You must be signed in to change notification settings - Fork 2
bug(memory): graph store tests fail after PR #2465 — migration 056 not applied in test DB #2468
Copy link
Copy link
Closed
Labels
P1High ROI, low complexity — do next sprintHigh ROI, low complexity — do next sprintbugSomething isn't workingSomething isn't workingmemoryzeph-memory crate (SQLite)zeph-memory crate (SQLite)
Description
Summary
After merging PR #2465 (Kumiho belief revision + D-MEM), multiple zeph-memory graph tests fail with:
SqliteError { code: 1, message: "no such column: superseded_by" }
Affected tests
graph::store::tests::test_bfs_empty_graphgraph::store::tests::test_bfs_diamondgraph::store::tests::test_invalidated_edges_excluded_from_bfsgraph::store::tests::edges_for_entity_includes_edge_typesemantic::tests::graph::recall_graph_returns_facts_for_known_entity(and 4 more)
Root cause
Migration 056_kumiho_dmem.sql adds superseded_by column to graph_edges. Tests that create an in-memory SQLite DB via GraphStore::new_in_memory() or similar do not run migration 056, so the column is absent.
Repro
cargo nextest run -p zeph-memory -E 'test(test_bfs_empty_graph)'Fix sketch
Ensure GraphStore test setup (or the in-memory DB fixture) runs all migrations including 056 before test queries execute. Check graph::store::tests setup function.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
P1High ROI, low complexity — do next sprintHigh ROI, low complexity — do next sprintbugSomething isn't workingSomething isn't workingmemoryzeph-memory crate (SQLite)zeph-memory crate (SQLite)