Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 6, 2025

What

  • move the enrichment helper unit cases from the loader suite into a dedicated tests/test_enrichment_helpers.py
  • update the relocated tests to import helpers from mcp_plex.loader.pipeline.enrichment and remove the coverage from tests/test_loader_unit.py
  • bump the project version metadata to 0.26.77 and refresh uv.lock

Why

  • keeps the loader unit suite focused on loader behaviors while grouping enrichment-specific checks together
  • aligns the tests with the helpers' module location

Affects

  • test modules covering loader enrichment helpers
  • project version metadata and lockfile

Testing

  • uv run pytest

Documentation

  • n/a

https://chatgpt.com/codex/tasks/task_e_68e3502123d8832895e1cd970c4c54b4

@github-actions
Copy link

github-actions bot commented Oct 6, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/loader
   __init__.py3951895%135–142, 172, 258–263, 616, 629, 748–750, 814–831, 1173
mcp_plex/loader/pipeline
   __init__.py15753%57–62, 68
   channels.py73297%19–20
   enrichment.py4306086%248, 253, 284, 286, 293, 297, 376, 397, 417, 425–427, 434–437, 440–442, 450, 507, 563, 586, 590, 592, 618, 636, 648, 677, 683, 686–694, 720, 723–725, 742–744, 764–767, 771, 785–791, 813, 815, 818–820
   ingestion.py81890%67, 98–108, 129, 157, 179
   orchestrator.py85594%48, 112, 141, 164–165
   persistence.py124993%112, 154–155, 165, 170, 174–176, 214
mcp_plex/server
   __init__.py6142995%43–44, 119–120, 148, 252, 256, 277–280, 297, 362, 365, 402, 420–421, 458, 1109, 1131–1137, 1173, 1191, 1196, 1214, 1338, 1375
   __main__.py440%3–8
   config.py48785%50, 52–55, 65, 76
TOTAL207114993% 

Tests Skipped Failures Errors Time
120 0 💤 0 ❌ 0 🔥 53.003s ⏱️

@Teagan42 Teagan42 merged commit 22dac1c into main Oct 6, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/create-enrichment-test-module-and-relocate-tests branch October 6, 2025 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants