Add tests

Author: Archmonger

src/js/bun.lockb

@@ -1,5 +1,5 @@
 from reactpy import config, logging, reactjs, types, web, widgets
-from reactpy._html import html
+from reactpy._html import h, html
 from reactpy.core import hooks
 from reactpy.core.component import component
 from reactpy.core.events import event
@@ -32,6 +32,7 @@
     "config",
     "create_context",
     "event",
+    "h",
     "hooks",
     "html",
     "logging",

src/reactpy/__init__.py

@@ -9,7 +9,7 @@
 from lxml import etree
 from lxml.html import fromstring
 
-from reactpy import html as _html
+from reactpy import h
 from reactpy.transforms import RequiredTransforms, attributes_to_reactjs
 from reactpy.types import Component, VdomDict
 
@@ -105,11 +105,11 @@ def string_to_reactpy(
             event handler that prevents the browser from navigating to the link. This is
             useful if you would rather have `reactpy-router` handle your URL navigation.
     """
-    if not html.strip():
-        return _html.div()
     if not isinstance(html, str):
         msg = f"Expected html to be a string, not {type(html).__name__}"
         raise TypeError(msg)
+    if not html.strip():
+        return h.div()
     if "<" not in html or ">" not in html:
         msg = "Expected html string to contain HTML tags, but no tags were found."
         raise ValueError(msg)
@@ -158,7 +158,7 @@ def _etree_to_vdom(
     attributes = attributes_to_reactjs(dict(node.items()))
 
     # Convert the lxml node to a VDOM dict
-    constructor = getattr(_html, str(node.tag))
+    constructor = getattr(h, str(node.tag))
     el = constructor(attributes, children)
 
     # Perform necessary transformations on the VDOM attributes to meet VDOM spec
@@ -241,13 +241,13 @@ def component_to_vdom(component: Component) -> VdomDict:
     """Convert the first render of a component into a VDOM dictionary"""
     result = component.render()
 
+    if result is None:
+        return h.fragment()
     if isinstance(result, dict):
         return result
     if hasattr(result, "render"):
         return component_to_vdom(cast(Component, result))
-    elif isinstance(result, str):
-        return _html.div(result)
-    return _html.div()
+    return h.div(result) if isinstance(result, str) else h.div()
 
 
 def _react_attribute_to_html(key: str, value: Any) -> tuple[str, str]:

src/reactpy/utils.py

@@ -92,6 +92,23 @@ def test_string_to_reactpy(case):
     assert utils.string_to_reactpy(case["source"]) == case["model"]
 
 
+@pytest.mark.parametrize("source", ["", "   ", "\n\t  "])
+def test_string_to_reactpy_empty_source(source):
+    assert utils.string_to_reactpy(source) == html.div()
+
+
+@pytest.mark.parametrize("source", [123, None, object()])
+def test_string_to_reactpy_non_string_source(source):
+    with pytest.raises(TypeError):
+        utils.string_to_reactpy(source)  # type: ignore[arg-type]
+
+
+@pytest.mark.parametrize("source", ["no tags", "plain text", "just words"])
+def test_string_to_reactpy_missing_tags(source):
+    with pytest.raises(ValueError):
+        utils.string_to_reactpy(source)
+
+
 @pytest.mark.parametrize(
     "case",
     [