Skip to content

Conversation

@Teagan42
Copy link
Contributor

@Teagan42 Teagan42 commented Oct 4, 2025

Summary

  • update invalid JSON REST tests to use content so httpx no longer emits deprecation warnings
  • document in AGENTS.md that pytest warnings must be addressed and bump the package version metadata

Testing

  • uv run pytest

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

Copilot AI review requested due to automatic review settings October 4, 2025 11:39
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR resolves pytest warnings by updating test HTTP client calls to use the correct parameter name and documents the requirement to address pytest warnings in the development workflow.

  • Updated HTTP test client calls from deprecated data parameter to content parameter
  • Bumped package version from 0.26.46 to 0.26.47
  • Enhanced documentation to require addressing pytest warnings before committing

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

File Description
tests/test_server.py Updated two test functions to use content instead of data parameter for HTTP POST requests
pyproject.toml Bumped version number to 0.26.47
docker/pyproject.deps.toml Updated version number to match main pyproject.toml
AGENTS.md Added requirement to address pytest warnings in the testing workflow documentation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link

github-actions bot commented Oct 4, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
mcp_plex
   config.py33682%50, 52–55, 58
   loader.py565699%198, 253, 255–257, 273
   server.py5922895%41–42, 117–118, 146, 250, 254, 275–278, 295, 353, 367, 385–386, 423, 1074, 1096–1102, 1138, 1156, 1161, 1179, 1303, 1340
TOTAL13644097% 

Tests Skipped Failures Errors Time
91 0 💤 0 ❌ 0 🔥 50.381s ⏱️

@Teagan42 Teagan42 merged commit afb6ccc into main Oct 4, 2025
4 checks passed
@Teagan42 Teagan42 deleted the codex/run-tests-and-fix-warnings branch October 4, 2025 11:42
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