fix(tests): fix: tests which call HTTP endpoints directly with the example parameters

Author: stainless-app[bot]

tests/test_client.py

@@ -23,9 +23,7 @@
 
 from supermemory import Supermemory, AsyncSupermemory, APIResponseValidationError
 from supermemory._types import Omit
-from supermemory._utils import maybe_transform
 from supermemory._models import BaseModel, FinalRequestOptions
-from supermemory._constants import RAW_RESPONSE_HEADER
 from supermemory._exceptions import APIStatusError, APITimeoutError, SupermemoryError, APIResponseValidationError
 from supermemory._base_client import (
     DEFAULT_TIMEOUT,
@@ -35,7 +33,6 @@
     DefaultAsyncHttpxClient,
     make_request_options,
 )
-from supermemory.types.memory_add_params import MemoryAddParams
 
 from .utils import update_env
 
@@ -725,42 +722,25 @@ def test_parse_retry_after_header(self, remaining_retries: int, retry_after: str
 
     @mock.patch("supermemory._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
-    def test_retrying_timeout_errors_doesnt_leak(self, respx_mock: MockRouter) -> None:
+    def test_retrying_timeout_errors_doesnt_leak(self, respx_mock: MockRouter, client: Supermemory) -> None:
         respx_mock.post("/v3/memories").mock(side_effect=httpx.TimeoutException("Test timeout error"))
 
         with pytest.raises(APITimeoutError):
-            self.client.post(
-                "/v3/memories",
-                body=cast(
-                    object,
-                    maybe_transform(
-                        dict(content="This is a detailed article about machine learning concepts..."), MemoryAddParams
-                    ),
-                ),
-                cast_to=httpx.Response,
-                options={"headers": {RAW_RESPONSE_HEADER: "stream"}},
-            )
+            client.memories.with_streaming_response.add(
+                content="This is a detailed article about machine learning concepts..."
+            ).__enter__()
 
         assert _get_open_connections(self.client) == 0
 
     @mock.patch("supermemory._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
-    def test_retrying_status_errors_doesnt_leak(self, respx_mock: MockRouter) -> None:
+    def test_retrying_status_errors_doesnt_leak(self, respx_mock: MockRouter, client: Supermemory) -> None:
         respx_mock.post("/v3/memories").mock(return_value=httpx.Response(500))
 
         with pytest.raises(APIStatusError):
-            self.client.post(
-                "/v3/memories",
-                body=cast(
-                    object,
-                    maybe_transform(
-                        dict(content="This is a detailed article about machine learning concepts..."), MemoryAddParams
-                    ),
-                ),
-                cast_to=httpx.Response,
-                options={"headers": {RAW_RESPONSE_HEADER: "stream"}},
-            )
-
+            client.memories.with_streaming_response.add(
+                content="This is a detailed article about machine learning concepts..."
+            ).__enter__()
         assert _get_open_connections(self.client) == 0
 
     @pytest.mark.parametrize("failures_before_success", [0, 2, 4])
@@ -1568,42 +1548,29 @@ async def test_parse_retry_after_header(self, remaining_retries: int, retry_afte
 
     @mock.patch("supermemory._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
-    async def test_retrying_timeout_errors_doesnt_leak(self, respx_mock: MockRouter) -> None:
+    async def test_retrying_timeout_errors_doesnt_leak(
+        self, respx_mock: MockRouter, async_client: AsyncSupermemory
+    ) -> None:
         respx_mock.post("/v3/memories").mock(side_effect=httpx.TimeoutException("Test timeout error"))
 
         with pytest.raises(APITimeoutError):
-            await self.client.post(
-                "/v3/memories",
-                body=cast(
-                    object,
-                    maybe_transform(
-                        dict(content="This is a detailed article about machine learning concepts..."), MemoryAddParams
-                    ),
-                ),
-                cast_to=httpx.Response,
-                options={"headers": {RAW_RESPONSE_HEADER: "stream"}},
-            )
+            await async_client.memories.with_streaming_response.add(
+                content="This is a detailed article about machine learning concepts..."
+            ).__aenter__()
 
         assert _get_open_connections(self.client) == 0
 
     @mock.patch("supermemory._base_client.BaseClient._calculate_retry_timeout", _low_retry_timeout)
     @pytest.mark.respx(base_url=base_url)
-    async def test_retrying_status_errors_doesnt_leak(self, respx_mock: MockRouter) -> None:
+    async def test_retrying_status_errors_doesnt_leak(
+        self, respx_mock: MockRouter, async_client: AsyncSupermemory
+    ) -> None:
         respx_mock.post("/v3/memories").mock(return_value=httpx.Response(500))
 
         with pytest.raises(APIStatusError):
-            await self.client.post(
-                "/v3/memories",
-                body=cast(
-                    object,
-                    maybe_transform(
-                        dict(content="This is a detailed article about machine learning concepts..."), MemoryAddParams
-                    ),
-                ),
-                cast_to=httpx.Response,
-                options={"headers": {RAW_RESPONSE_HEADER: "stream"}},
-            )
-
+            await async_client.memories.with_streaming_response.add(
+                content="This is a detailed article about machine learning concepts..."
+            ).__aenter__()
         assert _get_open_connections(self.client) == 0
 
     @pytest.mark.parametrize("failures_before_success", [0, 2, 4])