ignore missing coverage

Author: rmorshea

src/py/reactpy/reactpy/_console/ast_utils.py

@@ -37,7 +37,7 @@ def rewrite_changed_nodes(
             ):
                 nodes_to_unparse.append(current_node)
                 break
-        else:  # pragma: no cover
+        else:  # nocov
             msg = "Failed to change code"
             raise RuntimeError(msg)
 

src/py/reactpy/reactpy/_console/rewrite_camel_case_props.py

@@ -23,7 +23,7 @@
 @click.argument("paths", nargs=-1, type=click.Path(exists=True))
 def rewrite_camel_case_props(paths: list[str]) -> None:
     """Rewrite camelCase props to snake_case"""
-    if sys.version_info < (3, 9):  # pragma: no cover
+    if sys.version_info < (3, 9):  # nocov
         msg = "This command requires Python>=3.9"
         raise RuntimeError(msg)
 

src/py/reactpy/reactpy/_console/rewrite_keys.py

@@ -45,7 +45,7 @@ def rewrite_keys(paths: list[str]) -> None:
     just above its changes. As such it requires manual intervention to put those
     comments back in their original location.
     """
-    if sys.version_info < (3, 9):  # pragma: no cover
+    if sys.version_info < (3, 9):  # nocov
         msg = "This command requires Python>=3.9"
         raise RuntimeError(msg)
 

src/py/reactpy/reactpy/_option.py

@@ -119,7 +119,7 @@ def __repr__(self) -> str:
         return f"Option({self._name}={self.current!r})"
 
 
-class DeprecatedOption(Option[_O]):  # pragma: no cover
+class DeprecatedOption(Option[_O]):  # nocov
     def __init__(self, message: str, *args: Any, **kwargs: Any) -> None:
         self._deprecation_message = message
         super().__init__(*args, **kwargs)

src/py/reactpy/reactpy/backend/_common.py

@@ -53,7 +53,7 @@ async def serve_development_asgi(
         # Since we aren't using the uvicorn's `run()` API, we can't guarantee uvicorn's
         # order of operations. So we need to make sure `shutdown()` always has an initialized
         # list of `self.servers` to use.
-        if not hasattr(server, "servers"):  # pragma: no cover
+        if not hasattr(server, "servers"):  # nocov
             server.servers = []
         await asyncio.wait_for(server.shutdown(), timeout=3)
 

src/py/reactpy/reactpy/backend/default.py

@@ -16,7 +16,7 @@ def configure(
     app: Any, component: RootComponentConstructor, options: None = None
 ) -> None:
     """Configure the given app instance to display the given component"""
-    if options is not None:  # pragma: no cover
+    if options is not None:  # nocov
         msg = "Default implementation cannot be configured with options"
         raise ValueError(msg)
     return _default_implementation().configure(app, component)
@@ -27,7 +27,7 @@ def create_development_app() -> Any:
     return _default_implementation().create_development_app()
 
 
-def Options(*args: Any, **kwargs: Any) -> NoReturn:  # pragma: no cover
+def Options(*args: Any, **kwargs: Any) -> NoReturn:  # nocov
     """Create configuration options"""
     msg = "Default implementation has no options."
     raise ValueError(msg)
@@ -57,7 +57,7 @@ def _default_implementation() -> BackendImplementation[Any]:
 
     try:
         implementation = next(all_implementations())
-    except StopIteration:  # pragma: no cover
+    except StopIteration:  # nocov
         logger.debug("Backend implementation import failed", exc_info=exc_info())
         msg = "No built-in server implementation installed."
         raise RuntimeError(msg) from None

src/py/reactpy/reactpy/backend/flask.py

@@ -111,7 +111,7 @@ def run_server() -> None:
         # the thread should eventually join
         thread.join(timeout=3)
         # just double check it happened
-        if thread.is_alive():  # pragma: no cover
+        if thread.is_alive():  # nocov
             msg = "Failed to shutdown server."
             raise RuntimeError(msg)
 
@@ -129,7 +129,7 @@ def use_request() -> Request:
 def use_connection() -> Connection[_FlaskCarrier]:
     """Get the current :class:`Connection`"""
     conn = _use_connection()
-    if not isinstance(conn.carrier, _FlaskCarrier):  # pragma: no cover
+    if not isinstance(conn.carrier, _FlaskCarrier):  # nocov
         msg = f"Connection has unexpected carrier {conn.carrier}. Are you running with a Flask server?"
         raise TypeError(msg)
     return conn
@@ -152,7 +152,7 @@ def _setup_common_routes(
     options: Options,
 ) -> None:
     cors_options = options.cors
-    if cors_options:  # pragma: no cover
+    if cors_options:  # nocov
         cors_params = cors_options if isinstance(cors_options, dict) else {}
         CORS(api_blueprint, **cors_params)
 
@@ -258,7 +258,7 @@ async def main() -> None:
     dispatch_thread_info = cast(_DispatcherThreadInfo, dispatch_thread_info_ref.current)
 
     if dispatch_thread_info is None:
-        raise RuntimeError("Failed to create dispatcher thread")
+        raise RuntimeError("Failed to create dispatcher thread")  # nocov
 
     stop = ThreadEvent()
 
@@ -274,7 +274,7 @@ def run_send() -> None:
             dispatch_thread_info.dispatch_loop.call_soon_threadsafe(
                 dispatch_thread_info.async_recv_queue.put_nowait, value
             )
-    finally:  # pragma: no cover
+    finally:  # nocov
         dispatch_thread_info.dispatch_loop.call_soon_threadsafe(
             dispatch_thread_info.dispatch_future.cancel
         )

src/py/reactpy/reactpy/backend/hooks.py

@@ -13,9 +13,9 @@
 def use_connection() -> Connection[Any]:
     """Get the current :class:`~reactpy.backend.types.Connection`."""
     conn = use_context(ConnectionContext)
-    if conn is None:
+    if conn is None:  # nocov
         msg = "No backend established a connection."
-        raise RuntimeError(msg)  # pragma: no cover
+        raise RuntimeError(msg)
     return conn
 
 

src/py/reactpy/reactpy/backend/sanic.py

@@ -80,7 +80,7 @@ def use_websocket() -> WebSocketConnection:
 def use_connection() -> Connection[_SanicCarrier]:
     """Get the current :class:`Connection`"""
     conn = _use_connection()
-    if not isinstance(conn.carrier, _SanicCarrier):  # pragma: no cover
+    if not isinstance(conn.carrier, _SanicCarrier):  # nocov
         msg = f"Connection has unexpected carrier {conn.carrier}. Are you running with a Sanic server?"
         raise TypeError(msg)
     return conn
@@ -103,7 +103,7 @@ def _setup_common_routes(
     options: Options,
 ) -> None:
     cors_options = options.cors
-    if cors_options:  # pragma: no cover
+    if cors_options:  # nocov
         cors_params = cors_options if isinstance(cors_options, dict) else {}
         CORS(api_blueprint, **cors_params)
 
@@ -159,7 +159,7 @@ async def model_stream(
     ) -> None:
         asgi_app = getattr(request.app, "_asgi_app", None)
         scope = asgi_app.transport.scope if asgi_app else {}
-        if not scope:  # pragma: no cover
+        if not scope:  # nocov
             logger.warning("No scope. Sanic may not be running with an ASGI server")
 
         send, recv = _make_send_recv_callbacks(socket)

src/py/reactpy/reactpy/backend/starlette.py

@@ -76,7 +76,7 @@ def use_websocket() -> WebSocket:
 
 def use_connection() -> Connection[WebSocket]:
     conn = _use_connection()
-    if not isinstance(conn.carrier, WebSocket):  # pragma: no cover
+    if not isinstance(conn.carrier, WebSocket):  # nocov
         msg = f"Connection has unexpected carrier {conn.carrier}. Are you running with a Flask server?"
         raise TypeError(msg)
     return conn
@@ -95,7 +95,7 @@ class Options(CommonOptions):
 
 def _setup_common_routes(options: Options, app: Starlette) -> None:
     cors_options = options.cors
-    if cors_options:  # pragma: no cover
+    if cors_options:  # nocov
         cors_params = (
             cors_options if isinstance(cors_options, dict) else {"allow_origins": ["*"]}
         )

src/py/reactpy/reactpy/backend/tornado.py

@@ -98,7 +98,7 @@ def use_request() -> HTTPServerRequest:
 
 def use_connection() -> Connection[HTTPServerRequest]:
     conn = _use_connection()
-    if not isinstance(conn.carrier, HTTPServerRequest):  # pragma: no cover
+    if not isinstance(conn.carrier, HTTPServerRequest):  # nocov
         msg = f"Connection has unexpected carrier {conn.carrier}. Are you running with a Flask server?"
         raise TypeError(msg)
     return conn

src/py/reactpy/reactpy/backend/utils.py

@@ -79,11 +79,11 @@ def all_implementations() -> Iterator[BackendImplementation[Any]]:
         try:
             relative_import_name = f"{__name__.rsplit('.', 1)[0]}.{name}"
             module = import_module(relative_import_name)
-        except ImportError:  # pragma: no cover
+        except ImportError:  # nocov
             logger.debug(f"Failed to import {name!r}", exc_info=True)
             continue
 
-        if not isinstance(module, BackendImplementation):  # pragma: no cover
+        if not isinstance(module, BackendImplementation):  # nocov
             msg = f"{module.__name__!r} is an invalid implementation"
             raise TypeError(msg)
 

src/py/reactpy/reactpy/core/_f_back.py

@@ -7,10 +7,10 @@
 def f_module_name(index: int = 0) -> str:
     frame = f_back(index + 1)
     if frame is None:
-        return ""  # pragma: no cover
+        return ""  # nocov
     name = frame.f_globals.get("__name__", "")
     if not isinstance(name, str):
-        raise TypeError("Expected module name to be a string")
+        raise TypeError("Expected module name to be a string")  # nocov
     return name
 
 
@@ -21,4 +21,4 @@ def f_back(index: int = 0) -> FrameType | None:
             return frame
         frame = frame.f_back
         index -= 1
-    return None  # pragma: no cover
+    return None  # nocov

src/py/reactpy/reactpy/core/hooks.py

@@ -248,11 +248,11 @@ def use_context(context: Context[_Type]) -> _Type:
     if provider is None:
         # same assertions but with normal exceptions
         if not isinstance(context, FunctionType):
-            raise TypeError(f"{context} is not a Context")
+            raise TypeError(f"{context} is not a Context")  # nocov
         if context.__kwdefaults__ is None:
-            raise TypeError(f"{context} has no 'value' kwarg")
+            raise TypeError(f"{context} has no 'value' kwarg")  # nocov
         if "value" not in context.__kwdefaults__:
-            raise TypeError(f"{context} has no 'value' kwarg")
+            raise TypeError(f"{context} has no 'value' kwarg")  # nocov
         return cast(_Type, context.__kwdefaults__["value"])
 
     return provider._value

src/py/reactpy/reactpy/core/layout.py

@@ -45,7 +45,7 @@ class Layout:
         "_model_states_by_life_cycle_state_id",
     ]
 
-    if not hasattr(abc.ABC, "__weakref__"):  # pragma: no cover
+    if not hasattr(abc.ABC, "__weakref__"):  # nocov
         __slots__.append("__weakref__")
 
     def __init__(self, root: ComponentType) -> None:
@@ -197,7 +197,7 @@ def _render_model(
     ) -> None:
         try:
             new_state.model.current = {"tagName": raw_model["tagName"]}
-        except Exception as e:  # pragma: no cover
+        except Exception as e:  # nocov
             msg = f"Expected a VDOM element dict, not {raw_model}"
             raise ValueError(msg) from e
         if "key" in raw_model:
@@ -595,13 +595,13 @@ def is_component_state(self) -> bool:
     def parent(self) -> _ModelState:
         parent = self._parent_ref()
         if parent is None:
-            raise RuntimeError("detached model state")
+            raise RuntimeError("detached model state")  # nocov
         return parent
 
     def append_child(self, child: Any) -> None:
         self.model.current["children"].append(child)
 
-    def __repr__(self) -> str:  # pragma: no cover
+    def __repr__(self) -> str:  # nocov
         return f"ModelState({ {s: getattr(self, s, None) for s in self.__slots__} })"
 
 

src/py/reactpy/reactpy/core/types.py

@@ -28,7 +28,7 @@ class State(NamedTuple, Generic[_Type]):
         value: _Type
         set_value: Callable[[_Type | Callable[[_Type], _Type]], None]
 
-else:  # pragma: no cover
+else:  # nocov
     State = namedtuple("State", ("value", "set_value"))
 
 

src/py/reactpy/reactpy/core/vdom.py

@@ -162,7 +162,7 @@ def vdom(
             (subject to change) specifies javascript that, when evaluated returns a
             React component.
     """
-    if kwargs:  # pragma: no cover
+    if kwargs:  # nocov
         if "key" in kwargs:
             if attributes_and_children:
                 maybe_attributes, *children = attributes_and_children

src/py/reactpy/reactpy/testing/backend.py

@@ -133,7 +133,7 @@ async def stop_server() -> None:
 
         try:
             await asyncio.wait_for(started.wait(), timeout=self.timeout)
-        except Exception:  # pragma: no cover
+        except Exception:  # nocov
             # see if we can await the future for a more helpful error
             await asyncio.wait_for(server_future, timeout=self.timeout)
             raise
@@ -151,7 +151,7 @@ async def __aexit__(
         self.mount(None)  # reset the view
 
         logged_errors = self.list_logged_exceptions(del_log_records=False)
-        if logged_errors:  # pragma: no cover
+        if logged_errors:  # nocov
             msg = "Unexpected logged exception"
             raise LogAssertionError(msg) from logged_errors[0]
 

src/py/reactpy/reactpy/testing/common.py

@@ -66,7 +66,7 @@ async def until(
             result = await self._func(*self._args, **self._kwargs)
             if condition(result):
                 break
-            elif (time.time() - started_at) > timeout:  # pragma: no cover
+            elif (time.time() - started_at) > timeout:  # nocov
                 msg = f"Expected {description} after {timeout} seconds - last value was {result!r}"
                 raise TimeoutError(msg)
 

src/py/reactpy/reactpy/testing/display.py

@@ -56,9 +56,9 @@ async def goto(
 
     async def root_element(self) -> ElementHandle:
         element = await self.page.wait_for_selector("#app", state="attached")
-        if element is None:
+        if element is None:  # nocov
             msg = "Root element not attached"
-            raise RuntimeError(msg)  # pragma: no cover
+            raise RuntimeError(msg)
         return element
 
     async def __aenter__(self) -> DisplayFixture:

src/py/reactpy/reactpy/testing/logs.py

@@ -61,7 +61,7 @@ def assert_reactpy_did_log(
                     )
                 ):
                     break
-            else:  # pragma: no cover
+            else:  # nocov
                 _raise_log_message_error(
                     "Could not find a log record matching the given",
                     match_message,

src/py/reactpy/reactpy/utils.py

@@ -95,7 +95,7 @@ def html_to_vdom(
             If ``True``, raise an exception if the HTML does not perfectly follow HTML5
             syntax.
     """
-    if not isinstance(html, str):  # pragma: no cover
+    if not isinstance(html, str):  # nocov
         msg = f"Expected html to be a string, not {type(html).__name__}"
         raise TypeError(msg)
 
@@ -112,7 +112,7 @@ def html_to_vdom(
         )
     except etree.XMLSyntaxError as e:
         if not strict:
-            raise e  # pragma: no cover
+            raise e  # nocov
         msg = "An error has occurred while parsing the HTML.\n\nThis HTML may be malformatted, or may not perfectly adhere to HTML5.\nIf you believe the exception above was due to something intentional, you can disable the strict parameter on html_to_vdom().\nOtherwise, repair your broken HTML and try again."
         raise HTMLParseError(msg) from e
 
@@ -136,7 +136,7 @@ def _etree_to_vdom(
             dictionary which will be replaced by ``new``. For example, you could use a
             transform function to add highlighting to a ``<code/>`` block.
     """
-    if not isinstance(node, etree._Element):  # pragma: no cover
+    if not isinstance(node, etree._Element):  # nocov
         msg = f"Expected node to be a etree._Element, not {type(node).__name__}"
         raise TypeError(msg)
 
@@ -291,7 +291,7 @@ def _vdom_attr_to_html_str(key: str, value: Any) -> tuple[str, str]:
     ):
         key = _CAMEL_CASE_SUB_PATTERN.sub("-", key)
 
-    if callable(value):
+    if callable(value):  # nocov
         raise TypeError(f"Cannot convert callable attribute {key}={value} to HTML")
 
     # Again, we lower the attribute name only to normalize - HTML is case-insensitive:

src/py/reactpy/reactpy/widgets.py

@@ -92,7 +92,7 @@ def __call__(self, value: str) -> _CastTo:
 
 def hotswap(
     update_on_change: bool = False,
-) -> tuple[_MountFunc, ComponentConstructor]:  # pragma: no cover
+) -> tuple[_MountFunc, ComponentConstructor]:  # nocov
     warn(
         "The 'hotswap' function is deprecated and will be removed in a future release",
         DeprecationWarning,

src/py/reactpy/tests/conftest.py

@@ -57,7 +57,7 @@ async def browser(pytestconfig: Config):
 
 @pytest.fixture(scope="session")
 def event_loop():
-    if os.name == "nt":  # pragma: no cover
+    if os.name == "nt":  # nocov
         asyncio.set_event_loop_policy(asyncio.WindowsProactorEventLoopPolicy())
     with open_event_loop() as loop:
         yield loop