chore(internal): fix binary response tests (#983)

openai · Dec 18, 2023 · 89b9cd9 · 89b9cd9
1 parent 15d3aba
commit 89b9cd9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 12 deletions.
diff --git a/tests/api_resources/audio/test_speech.py b/tests/api_resources/audio/test_speech.py
@@ -39,8 +39,7 @@ def test_method_create(self, client: OpenAI, respx_mock: MockRouter) -> None:
     @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 @@ async def test_method_create(self, client: AsyncOpenAI, respx_mock: MockRouter)
     @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",

diff --git a/tests/api_resources/test_files.py b/tests/api_resources/test_files.py
@@ -95,22 +95,20 @@ def test_raw_response_delete(self, client: OpenAI) -> None:
         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 @@ async def test_raw_response_delete(self, client: AsyncOpenAI) -> None:
         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",
         )