Skip to content

test: add saga, middleware, embeddings, tools unit tests (+54) - #54

Merged
Cipher208 merged 1 commit into
masterfrom
test/unit-tests-coverage
Jul 5, 2026
Merged

Cipher208 merged 1 commit into
masterfrom
test/unit-tests-coverage

Conversation

@Cipher208

Copy link
Copy Markdown
Owner

Changes

New test files (54 tests)

  • est_saga_unit.py — persistence, idempotency, watchdog, helpers (20 tests)
  • est_middleware_unit.py — ImportanceGate, Validation, Dedup, Pipeline (10 tests)
  • est_embeddings_unit.py — hash_embedding, similarity (8 tests)
  • est_tools_unit.py — _validate_layer, _fire_hook, memory_remember, recall, forget, session, episode, graph, stats (16 tests)

Coverage improvement

  • Total: 64% → 70% (+6%)
  • shared/saga.py: 58% → 79% (+21%)
  • shared/middleware.py: 71% → 88% (+17%)
  • shared/embeddings.py: 58% → 78% (+20%)
  • mcp_server/tools_layer.py: 15% → 53% (+38%)

Tests

  • Total: 447 (was 393, +54)
  • 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!

- test_saga_unit.py: persistence, idempotency, watchdog, helpers (20 tests)
- test_middleware_unit.py: ImportanceGate, Validation, Dedup, Pipeline (10 tests)
- test_embeddings_unit.py: hash_embedding, similarity (8 tests)
- test_tools_unit.py: _validate_layer, _fire_hook, memory_remember, recall, forget, session, episode, graph, stats (16 tests)
- Coverage: 64% → 70% (+6%)
  - shared/saga.py: 58% → 79%
  - shared/middleware.py: 71% → 88%
  - shared/embeddings.py: 58% → 78%
  - mcp_server/tools_layer.py: 15% → 53%
- Total: 447 tests (was 393, +54)
@Cipher208
Cipher208 force-pushed the test/unit-tests-coverage branch from 4420d80 to f288f88 Compare July 5, 2026 21:25
@Cipher208
Cipher208 merged commit 8f36741 into master Jul 5, 2026
18 checks passed
@Cipher208
Cipher208 deleted the test/unit-tests-coverage branch July 5, 2026 21:28
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