Revert "Move GITHUB_ACTIONS out of testing module"

This reverts commit 5171a1a.

Author: Archmonger

src/reactpy/executors/pyscript/utils.py

@@ -17,8 +17,9 @@
 
 import reactpy
 from reactpy.config import REACTPY_DEBUG, REACTPY_PATH_PREFIX, REACTPY_WEB_MODULES_DIR
+from reactpy.testing.common import GITHUB_ACTIONS
 from reactpy.types import VdomDict
-from reactpy.utils import GITHUB_ACTIONS, reactpy_to_string
+from reactpy.utils import reactpy_to_string
 
 if TYPE_CHECKING:
     from collections.abc import Sequence

src/reactpy/testing/backend.py

@@ -6,23 +6,21 @@
 from collections.abc import Callable
 from contextlib import AsyncExitStack
 from types import TracebackType
-from typing import Any
+from typing import TYPE_CHECKING, Any
 from urllib.parse import urlencode, urlunparse
 
 import uvicorn
 
-from reactpy.core.component import component
-from reactpy.core.hooks import use_callback, use_effect, use_state
-from reactpy.executors.asgi.middleware import ReactPyMiddleware
-from reactpy.executors.asgi.standalone import ReactPy
-from reactpy.executors.asgi.types import AsgiApp
 from reactpy.testing.logs import (
     LogAssertionError,
     capture_reactpy_logs,
     list_logged_exceptions,
 )
-from reactpy.types import ComponentConstructor
-from reactpy.utils import Ref
+
+if TYPE_CHECKING:
+    from reactpy.executors.asgi.types import AsgiApp
+    from reactpy.types import ComponentConstructor
+    from reactpy.utils import Ref
 
 
 class BackendFixture:
@@ -48,6 +46,9 @@ def __init__(
         port: int | None = None,
         **reactpy_config: Any,
     ) -> None:
+        from reactpy.executors.asgi.middleware import ReactPyMiddleware
+        from reactpy.executors.asgi.standalone import ReactPy
+
         self.host = host
         self.port = port or 0
         self.mount = mount_to_hotswap
@@ -201,6 +202,10 @@ def DivTwo(self):
 
             # displaying the output now will show DivTwo
     """
+    from reactpy.core.component import component
+    from reactpy.core.hooks import use_callback, use_effect, use_state
+    from reactpy.utils import Ref
+
     constructor_ref: Ref[Callable[[], Any]] = Ref(lambda: None)
 
     if update_on_change:

src/reactpy/testing/common.py

@@ -2,23 +2,31 @@
 
 import asyncio
 import inspect
+import os
 import time
 from collections.abc import Awaitable, Callable, Coroutine
 from functools import wraps
-from typing import Any, Generic, ParamSpec, TypeVar, cast
+from typing import TYPE_CHECKING, Any, Generic, ParamSpec, TypeVar, cast
 from uuid import uuid4
 from weakref import ref
 
-from reactpy.config import REACTPY_TESTS_DEFAULT_TIMEOUT
-from reactpy.core._life_cycle_hook import HOOK_STACK, LifeCycleHook
-from reactpy.core.events import EventHandler, to_event_handler_function
-from reactpy.utils import GITHUB_ACTIONS
+if TYPE_CHECKING:
+    from reactpy.core._life_cycle_hook import LifeCycleHook
+    from reactpy.core.events import EventHandler
 
 _P = ParamSpec("_P")
 _R = TypeVar("_R")
 
 
 _DEFAULT_POLL_DELAY = 0.1
+GITHUB_ACTIONS = os.getenv("GITHUB_ACTIONS", "").lower() in {
+    "y",
+    "yes",
+    "t",
+    "true",
+    "on",
+    "1",
+}
 DEFAULT_TYPE_DELAY = 250 if GITHUB_ACTIONS else 25
 
 
@@ -47,11 +55,16 @@ async def async_func(*args: _P.args, **kwargs: _P.kwargs) -> _R:
     async def until(
         self,
         condition: Callable[[_R], bool],
-        timeout: float = REACTPY_TESTS_DEFAULT_TIMEOUT.current,
+        timeout: float | None = None,
         delay: float = _DEFAULT_POLL_DELAY,
         description: str = "condition to be true",
     ) -> None:
         """Check that the coroutines result meets a condition within the timeout"""
+        if timeout is None:
+            from reactpy.config import REACTPY_TESTS_DEFAULT_TIMEOUT
+
+            timeout = REACTPY_TESTS_DEFAULT_TIMEOUT.current
+
         started_at = time.time()
         while True:
             await asyncio.sleep(delay)
@@ -65,7 +78,7 @@ async def until(
     async def until_is(
         self,
         right: _R,
-        timeout: float = REACTPY_TESTS_DEFAULT_TIMEOUT.current,
+        timeout: float | None = None,
         delay: float = _DEFAULT_POLL_DELAY,
     ) -> None:
         """Wait until the result is identical to the given value"""
@@ -79,7 +92,7 @@ async def until_is(
     async def until_equals(
         self,
         right: _R,
-        timeout: float = REACTPY_TESTS_DEFAULT_TIMEOUT.current,
+        timeout: float | None = None,
         delay: float = _DEFAULT_POLL_DELAY,
     ) -> None:
         """Wait until the result is equal to the given value"""
@@ -132,6 +145,8 @@ def capture(self, render_function: Callable[..., Any]) -> Callable[..., Any]:
 
         @wraps(render_function)
         def wrapper(*args: Any, **kwargs: Any) -> Any:
+            from reactpy.core._life_cycle_hook import HOOK_STACK
+
             self = self_ref()
             if self is None:
                 raise RuntimeError("Hook catcher has been garbage collected")
@@ -196,6 +211,8 @@ def use(
         stop_propagation: bool = False,
         prevent_default: bool = False,
     ) -> EventHandler:
+        from reactpy.core.events import EventHandler, to_event_handler_function
+
         return EventHandler(
             to_event_handler_function(function),
             stop_propagation,

src/reactpy/testing/display.py

@@ -8,13 +8,13 @@
 
 from playwright.async_api import Browser, Page, async_playwright, expect
 
-from reactpy.config import REACTPY_TESTS_DEFAULT_TIMEOUT as DEFAULT_TIMEOUT
 from reactpy.testing.backend import BackendFixture
-from reactpy.types import RootComponentConstructor
 
 if TYPE_CHECKING:
     import pytest
 
+    from reactpy.types import RootComponentConstructor
+
 _logger = getLogger(__name__)
 
 
@@ -32,6 +32,8 @@ def __init__(
         headless: bool = False,
         timeout: float | None = None,
     ) -> None:
+        from reactpy.config import REACTPY_TESTS_DEFAULT_TIMEOUT as DEFAULT_TIMEOUT
+
         if backend:
             self.backend_is_external = True
             self.backend = backend

src/reactpy/testing/logs.py

@@ -7,8 +7,6 @@
 from traceback import format_exception
 from typing import Any, NoReturn
 
-from reactpy.logging import ROOT_LOGGER
-
 
 class LogAssertionError(AssertionError):
     """An assertion error raised in relation to log messages."""
@@ -127,6 +125,8 @@ def capture_reactpy_logs() -> Iterator[list[logging.LogRecord]]:
 
     Any logs produced in this context are cleared afterwards
     """
+    from reactpy.logging import ROOT_LOGGER
+
     original_level = ROOT_LOGGER.level
     ROOT_LOGGER.setLevel(logging.DEBUG)
     try:

src/reactpy/utils.py

@@ -1,6 +1,5 @@
 from __future__ import annotations
 
-import os
 import re
 from collections.abc import Callable, Iterable
 from importlib import import_module
@@ -17,14 +16,6 @@
 _RefValue = TypeVar("_RefValue")
 _ModelTransform = Callable[[VdomDict], Any]
 _UNDEFINED: Any = object()
-GITHUB_ACTIONS = os.getenv("GITHUB_ACTIONS", "").lower() in {
-    "y",
-    "yes",
-    "t",
-    "true",
-    "on",
-    "1",
-}
 
 
 class Ref(Generic[_RefValue]):