Skip to content

Commit 10ce668

Browse files
adamtheturtleclaude
andcommitted
Document that client timeouts are not enforced in tests
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 parent 381c12a commit 10ce668

1 file changed

Lines changed: 9 additions & 1 deletion

File tree

tests/mock_vws/test_flask_app_usage.py

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -610,7 +610,15 @@ def test_random(
610610

611611

612612
class TestResponseDelay:
613-
"""Tests for the response delay feature."""
613+
"""Tests for the response delay feature.
614+
615+
These tests run through the ``responses`` library, which intercepts
616+
requests in-process. Because of this, the client ``timeout`` parameter
617+
is not enforced — the delay blocks but never raises
618+
``requests.exceptions.Timeout``. When running the Flask app as a real
619+
server (e.g. in Docker), the delay causes a genuinely slow HTTP
620+
response and the ``requests`` client will raise ``Timeout`` on its own.
621+
"""
614622

615623
DELAY_SECONDS = 0.5
616624

0 commit comments

Comments
 (0)