Commit b56cf723

Stainless Bot <107565488+stainless-bot@users.noreply.github.com>
2023-12-18 22:06:05
chore(internal): fix binary response tests (#983)
1 parent 7367256
Changed files (2)
tests
api_resources
tests/api_resources/audio/test_speech.py
@@ -39,8 +39,7 @@ class TestSpeech:
     @pytest.mark.respx(base_url=base_url)
     def test_method_create_with_all_params(self, client: OpenAI, respx_mock: MockRouter) -> None:
         respx_mock.post("/audio/speech").mock(return_value=httpx.Response(200, json={"foo": "bar"}))
-        speech = respx_mock.post("/audio/speech").mock(return_value=httpx.Response(200, json={"foo": "bar"}))
-        client.audio.speech.create(
+        speech = client.audio.speech.create(
             input="string",
             model="string",
             voice="alloy",
@@ -89,8 +88,7 @@ class TestAsyncSpeech:
     @pytest.mark.respx(base_url=base_url)
     async def test_method_create_with_all_params(self, client: AsyncOpenAI, respx_mock: MockRouter) -> None:
         respx_mock.post("/audio/speech").mock(return_value=httpx.Response(200, json={"foo": "bar"}))
-        speech = respx_mock.post("/audio/speech").mock(return_value=httpx.Response(200, json={"foo": "bar"}))
-        await client.audio.speech.create(
+        speech = await client.audio.speech.create(
             input="string",
             model="string",
             voice="alloy",
tests/api_resources/test_files.py
@@ -95,22 +95,20 @@ class TestFiles:
         file = response.parse()
         assert_matches_type(FileDeleted, file, path=["response"])
 
-    @pytest.mark.skip(reason="mocked response isn't working yet")
     @parametrize
     @pytest.mark.respx(base_url=base_url)
     def test_method_content(self, client: OpenAI, respx_mock: MockRouter) -> None:
-        respx_mock.get("/files/{file_id}/content").mock(return_value=httpx.Response(200, json={"foo": "bar"}))
+        respx_mock.get("/files/string/content").mock(return_value=httpx.Response(200, json={"foo": "bar"}))
         file = client.files.content(
             "string",
         )
         assert isinstance(file, BinaryResponseContent)
         assert file.json() == {"foo": "bar"}
 
-    @pytest.mark.skip(reason="mocked response isn't working yet")
     @parametrize
     @pytest.mark.respx(base_url=base_url)
     def test_raw_response_content(self, client: OpenAI, respx_mock: MockRouter) -> None:
-        respx_mock.get("/files/{file_id}/content").mock(return_value=httpx.Response(200, json={"foo": "bar"}))
+        respx_mock.get("/files/string/content").mock(return_value=httpx.Response(200, json={"foo": "bar"}))
         response = client.files.with_raw_response.content(
             "string",
         )
@@ -212,22 +210,20 @@ class TestAsyncFiles:
         file = response.parse()
         assert_matches_type(FileDeleted, file, path=["response"])
 
-    @pytest.mark.skip(reason="mocked response isn't working yet")
     @parametrize
     @pytest.mark.respx(base_url=base_url)
     async def test_method_content(self, client: AsyncOpenAI, respx_mock: MockRouter) -> None:
-        respx_mock.get("/files/{file_id}/content").mock(return_value=httpx.Response(200, json={"foo": "bar"}))
+        respx_mock.get("/files/string/content").mock(return_value=httpx.Response(200, json={"foo": "bar"}))
         file = await client.files.content(
             "string",
         )
         assert isinstance(file, BinaryResponseContent)
         assert file.json() == {"foo": "bar"}
 
-    @pytest.mark.skip(reason="mocked response isn't working yet")
     @parametrize
     @pytest.mark.respx(base_url=base_url)
     async def test_raw_response_content(self, client: AsyncOpenAI, respx_mock: MockRouter) -> None:
-        respx_mock.get("/files/{file_id}/content").mock(return_value=httpx.Response(200, json={"foo": "bar"}))
+        respx_mock.get("/files/string/content").mock(return_value=httpx.Response(200, json={"foo": "bar"}))
         response = await client.files.with_raw_response.content(
             "string",
         )