Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 7, 2025

What

  • add focused tests for entrypoints, loader utilities, and server helpers to close previously uncovered paths
  • update existing test doubles to support async client shutdown during sample-data runs
  • bump the project version and lockfile to 2.0.8 to reflect the new test suite

Why

  • ensure the suite achieves the requested 95% coverage target across critical modules

Affects

  • adds new tests in tests/ and adjusts existing loader/server fixtures; increments package metadata

Testing

  • uv run coverage run -m pytest

Documentation

  • none needed

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

@github-actions
Copy link

github-actions bot commented Oct 7, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/loader
   __init__.py1821194%139, 209, 323–325, 327, 416–435
   qdrant.py198299%338–343
   samples.py41393%55–57
mcp_plex/loader/pipeline
   enrichment.py5207586%234, 255–261, 314, 319, 350, 352, 359, 361, 461, 482, 502, 510–512, 515–519, 522–524, 532–537, 545, 596–597, 621, 679–680, 696, 723, 730, 732, 736, 788, 806, 818, 847, 853, 856–864, 890, 893–895, 912–914, 934–937, 941, 955–959, 981, 983, 986–988
   ingestion.py1551392%82, 133–153, 188, 225, 241, 334, 340, 358
   orchestrator.py95595%60, 136, 165, 188–189
   persistence.py130993%123, 165–166, 174, 185, 189–191, 240
mcp_plex/server
   __init__.py3511297%201, 263, 281–282, 290, 365, 459, 489, 508, 515, 534, 647
   cli.py62198%73
   config.py82199%59
mcp_plex/server/tools
   media_library.py289698%501, 811–815
TOTAL270013895% 

Tests Skipped Failures Errors Time
194 0 💤 0 ❌ 0 🔥 1m 18s ⏱️

@Teagan42 Teagan42 merged commit 0886058 into main Oct 7, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/achieve-95%-test-coverage branch October 7, 2025 10:48
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