|
6 | 6 | import os
|
7 | 7 | import sys
|
8 | 8 | import json
|
9 |
| -import time |
10 | 9 | import asyncio
|
11 | 10 | import inspect
|
12 |
| -import subprocess |
13 | 11 | import tracemalloc
|
14 | 12 | from typing import Any, Union, cast
|
15 |
| -from textwrap import dedent |
16 | 13 | from unittest import mock
|
17 | 14 | from typing_extensions import Literal
|
18 | 15 |
|
|
23 | 20 |
|
24 | 21 | from runwayml import RunwayML, AsyncRunwayML, APIResponseValidationError
|
25 | 22 | from runwayml._types import Omit
|
| 23 | +from runwayml._utils import asyncify |
26 | 24 | from runwayml._models import BaseModel, FinalRequestOptions
|
27 | 25 | from runwayml._exceptions import RunwayMLError, APIStatusError, APITimeoutError, APIResponseValidationError
|
28 | 26 | from runwayml._base_client import (
|
29 | 27 | DEFAULT_TIMEOUT,
|
30 | 28 | HTTPX_DEFAULT_TIMEOUT,
|
31 | 29 | BaseClient,
|
| 30 | + OtherPlatform, |
32 | 31 | DefaultHttpxClient,
|
33 | 32 | DefaultAsyncHttpxClient,
|
| 33 | + get_platform, |
34 | 34 | make_request_options,
|
35 | 35 | )
|
36 | 36 |
|
@@ -1661,50 +1661,9 @@ def retry_handler(_request: httpx.Request) -> httpx.Response:
|
1661 | 1661 |
|
1662 | 1662 | assert response.http_request.headers.get("x-stainless-retry-count") == "42"
|
1663 | 1663 |
|
1664 |
| - def test_get_platform(self) -> None: |
1665 |
| - # A previous implementation of asyncify could leave threads unterminated when |
1666 |
| - # used with nest_asyncio. |
1667 |
| - # |
1668 |
| - # Since nest_asyncio.apply() is global and cannot be un-applied, this |
1669 |
| - # test is run in a separate process to avoid affecting other tests. |
1670 |
| - test_code = dedent(""" |
1671 |
| - import asyncio |
1672 |
| - import nest_asyncio |
1673 |
| - import threading |
1674 |
| -
|
1675 |
| - from runwayml._utils import asyncify |
1676 |
| - from runwayml._base_client import get_platform |
1677 |
| -
|
1678 |
| - async def test_main() -> None: |
1679 |
| - result = await asyncify(get_platform)() |
1680 |
| - print(result) |
1681 |
| - for thread in threading.enumerate(): |
1682 |
| - print(thread.name) |
1683 |
| -
|
1684 |
| - nest_asyncio.apply() |
1685 |
| - asyncio.run(test_main()) |
1686 |
| - """) |
1687 |
| - with subprocess.Popen( |
1688 |
| - [sys.executable, "-c", test_code], |
1689 |
| - text=True, |
1690 |
| - ) as process: |
1691 |
| - timeout = 10 # seconds |
1692 |
| - |
1693 |
| - start_time = time.monotonic() |
1694 |
| - while True: |
1695 |
| - return_code = process.poll() |
1696 |
| - if return_code is not None: |
1697 |
| - if return_code != 0: |
1698 |
| - raise AssertionError("calling get_platform using asyncify resulted in a non-zero exit code") |
1699 |
| - |
1700 |
| - # success |
1701 |
| - break |
1702 |
| - |
1703 |
| - if time.monotonic() - start_time > timeout: |
1704 |
| - process.kill() |
1705 |
| - raise AssertionError("calling get_platform using asyncify resulted in a hung process") |
1706 |
| - |
1707 |
| - time.sleep(0.1) |
| 1664 | + async def test_get_platform(self) -> None: |
| 1665 | + platform = await asyncify(get_platform)() |
| 1666 | + assert isinstance(platform, (str, OtherPlatform)) |
1708 | 1667 |
|
1709 | 1668 | async def test_proxy_environment_variables(self, monkeypatch: pytest.MonkeyPatch) -> None:
|
1710 | 1669 | # Test that the proxy environment variables are set correctly
|
|
0 commit comments