Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 7, 2025

What

  • extend the play media test to cover a non-zero offset
  • verify the fake Plex client receives the expected millisecond offset and the tool response echoes the seconds value

Why

  • ensure the play media tool forwards offsets correctly and returns the requested offset to callers

Affects

  • tests

Testing

  • uv run pytest tests/test_server.py::test_play_media_with_alias

Documentation

  • not needed

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

@github-actions
Copy link

github-actions bot commented Oct 7, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex/loader
   __init__.py3861895%80–87, 161, 251–256, 633, 748–750, 752, 838–857, 1222
mcp_plex/loader/pipeline
   __init__.py16756%57–62, 68
   enrichment.py4406086%245, 250, 281, 283, 290, 294, 398, 419, 439, 447–449, 456–459, 462–464, 472, 535, 591, 616, 620, 622, 652, 670, 682, 711, 717, 720–728, 754, 757–759, 776–778, 798–801, 805, 819–825, 847, 849, 852–854
   ingestion.py1551392%80, 129–149, 187, 224, 240, 333, 339, 359
   orchestrator.py92595%48, 126, 155, 178–179
   persistence.py129993%113, 155–156, 166, 177, 181–183, 236
mcp_plex/server
   __init__.py6453195%52–53, 133–134, 162, 231, 237–238, 304, 325–328, 345, 410, 413, 450, 468–469, 506, 1157, 1179–1185, 1221, 1239, 1244, 1262, 1386, 1423
   __main__.py440%3–8
   config.py48785%50, 52–55, 65, 76
TOTAL221015493% 

Tests Skipped Failures Errors Time
143 0 💤 0 ❌ 0 🔥 1m 8s ⏱️

@Teagan42 Teagan42 merged commit 4fd2d24 into main Oct 7, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/add-tests-for-play_media-offset-handling branch October 7, 2025 03:39
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