Skip to content

test(tools): add full coverage for all 25 tools and 24 hooks - #56

Merged
Cipher208 merged 1 commit into
masterfrom
test/full-tool-coverage
Jul 5, 2026
Merged

test(tools): add full coverage for all 25 tools and 24 hooks#56
Cipher208 merged 1 commit into
masterfrom
test/full-tool-coverage

Conversation

@Cipher208

Copy link
Copy Markdown
Owner

Changes

All 25 MCP tools now tested

tools_layer.py (17 tools):

  • memory_remember, recall, forget, session_start/end, episode_save/recall
  • graph_add/query, stats, context_inject
  • session_list, episode_list, episode_get, graph_nodes, graph_edges, context

tools_ops.py (8 tools):

  • memory_api_key, backup, saga, data, sync_replica, cleanup, lucidity_purge, search_rrf

Hook dispatch verification (9 key hooks verified)

  • message_received, emotion_trigger, importance_gate
  • state_delta, consolidation, error_occurred, decision_made
  • retrieval_router, auto_context

Total

  • 474 tests (was 458, +16)
  • All pass, lint clean, format clean

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

tools_layer.py (17 tools):
- memory_remember, recall, forget, session_start/end, episode_save/recall
- graph_add/query, stats, context_inject
- session_list, episode_list, episode_get, graph_nodes, graph_edges, context

tools_ops.py (8 tools):
- memory_api_key, backup, saga, data, sync_replica, cleanup, lucidity_purge, search_rrf

Hook dispatch verification:
- message_received, emotion_trigger, importance_gate, state_delta
- consolidation, error_occurred, decision_made, retrieval_router, auto_context

Total: 474 tests (was 458, +16)
@Cipher208
Cipher208 force-pushed the test/full-tool-coverage branch from cd0c905 to 445728d Compare July 5, 2026 22:23
@Cipher208
Cipher208 merged commit b0d7e7c into master Jul 5, 2026
18 checks passed
@Cipher208
Cipher208 deleted the test/full-tool-coverage branch July 5, 2026 22:26
Cipher208 pushed a commit that referenced this pull request Jul 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants