Skip to content

Commit 262c17b

Browse files
committed
Bound the bare-import subprocess test with the suite's standard timeout
Every other subprocess.run in the suite passes timeout=20 so a hung child fails the test instead of stalling the run until the CI job timeout; match that convention here. No-Verification-Needed: test-only change
1 parent 90b6ba0 commit 262c17b

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

tests/test_types.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -521,11 +521,14 @@ def test_bare_import_mcp_binds_the_types_submodule():
521521
has already imported `mcp.types`, and reloading `mcp` here would rebind classes that other
522522
tests hold references to.
523523
"""
524+
# A regression hangs forever, so the bound only has to beat never (matches the suite's
525+
# other subprocess.run calls).
524526
result = subprocess.run(
525527
[sys.executable, "-c", "import mcp; print(mcp.types.Tool.__name__)"],
526528
capture_output=True,
527529
text=True,
528530
check=False,
531+
timeout=20,
529532
)
530533
assert result.returncode == 0, result.stderr
531534
assert result.stdout == snapshot("Tool\n")

0 commit comments

Comments
 (0)