Skip to content

bug(memory): graph store tests fail after PR #2465 — migration 056 not applied in test DB #2468

@bug-ops

Description

@bug-ops

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_graph
  • graph::store::tests::test_bfs_diamond
  • graph::store::tests::test_invalidated_edges_excluded_from_bfs
  • graph::store::tests::edges_for_entity_includes_edge_type
  • semantic::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.

Metadata

Metadata

Assignees

Labels

P1High ROI, low complexity — do next sprintbugSomething isn't workingmemoryzeph-memory crate (SQLite)

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions