Add deprecated warnings to TestClient on use of timeout argument (#2840)

Co-authored-by: Marcelo Trylesinski <marcelotryle@gmail.com>

Author: iudeen

starlette/testclient.py

@@ -7,6 +7,7 @@
 import math
 import sys
 import typing
+import warnings
 from concurrent.futures import Future
 from types import GeneratorType
 from urllib.parse import unquote, urljoin
@@ -426,6 +427,12 @@ def request(  # type: ignore[override]
         timeout: httpx._types.TimeoutTypes | httpx._client.UseClientDefault = httpx._client.USE_CLIENT_DEFAULT,
         extensions: dict[str, typing.Any] | None = None,
     ) -> httpx.Response:
+        if timeout is not httpx.USE_CLIENT_DEFAULT:
+            warnings.warn(
+                "You should not use the 'timeout' argument with the TestClient. "
+                "See https://github.com/encode/starlette/issues/1108 for more information.",
+                DeprecationWarning,
+            )
         url = self._merge_url(url)
         return super().request(
             method,

tests/test_testclient.py

@@ -422,3 +422,9 @@ async def app(scope: Scope, receive: Receive, send: Send) -> None:
     with client.websocket_connect("/hello-world", params={"foo": "bar"}) as websocket:
         data = websocket.receive_bytes()
         assert data == b"/hello-world"
+
+
+def test_timeout_deprecation() -> None:
+    with pytest.deprecated_call(match="You should not use the 'timeout' argument with the TestClient."):
+        client = TestClient(mock_service)
+        client.get("/", timeout=1)