Skip to content

test(memory): add unit tests for run_consolidation_sweep orchestration #2360

@bug-ops

Description

@bug-ops

Context

run_consolidation_sweep in crates/zeph-memory/src/consolidation.rs has no unit test covering its orchestration logic. The function drives the MERGE/UPDATE decision loop, clustering, and result counters.

What to test

  • result.merges, result.updates, result.skipped counters are correctly incremented
  • embedded.len() < 2 guard exits early with zero ops
  • The Update op DB path (content replacement) executes and updates the correct row

Acceptance criteria

#[tokio::test] tests in consolidation.rs covering the above cases with an in-memory SQLite pool.

Discovered in PR #2358 tester review.

Metadata

Metadata

Assignees

Labels

P2High value, medium complexitymemoryzeph-memory crate (SQLite)

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions