test: bootstrap live_tests conftest + fix requires-python - #2
Open
neoalienson wants to merge 1 commit into
Open
Conversation
- Add live_tests/conftest.py with the autouse `throttle` fixture (default 1.5s, override via LIVE_TEST_THROTTLE_S) from hkopenai/live-tests-template. The fixture is the prerequisite for any follow-up live integration test in this repo. - Bump requires-python from >=3.7 to >=3.10 in pyproject.toml -- fastmcp 3.x / 4.0.0b1 no longer resolves against 3.7, and hk-finance-mcp-server, hk-climate-mcp-server and hk-tech-mcp-server already use >=3.10.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
live_tests/conftest.pywith the autousethrottlefixture (default 1.5s, override viaLIVE_TEST_THROTTLE_S) from the shared hkopenai/live-tests-template repo. The fixture is the prerequisite for any follow-up live integration test in this repo.requires-pythonfrom>=3.7to>=3.10inpyproject.toml. The pin was stale and causeduv syncto fail becausefastmcp3.x / 4.0.0b1 no longer resolves against 3.7; this aligns withhk-finance-mcp-server,hk-climate-mcp-serverandhk-tech-mcp-serverwhich already use>=3.10.Why no live test in this PR
Each tool has its own upstream endpoint and schema, so the actual test bodies belong in per-tool follow-up PRs (see hk-tech-mcp-server #1 and hk-transportation-mcp-server #3 for prior art). This PR just gets the throttle fixture and the python-version fix in place so those follow-ups can drop tests in without touching shared files.
The live tests opt in via the
RUN_LIVE_TESTS=trueenv var, and the autouse throttle fixture only fires when those tests actually run; the defaultuv run pytestinvocation is unaffected.Test plan
uv run pytest-- no live test files exist yet, so the conftest has no effect; unit tests run as beforeRUN_LIVE_TESTS=true uv run pytest-- once follow-up PRs land, those live tests will run with the 1.5s throttle