add test for non-json serializable attrs

Author: rmorshea

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

@@ -114,7 +114,7 @@ def vdom_head_elements_to_html(head: Sequence[VdomDict] | VdomDict | str) -> str
             head = cast(VdomDict, {**head, "tagName": ""})
         return vdom_to_html(head)
     else:
-        return vdom_to_html(html._(head))
+        return vdom_to_html(html._(*head))
 
 
 @dataclass

src/py/reactpy/reactpy/core/serve.py

@@ -57,7 +57,7 @@ async def _single_outgoing_loop(
             if not REACTPY_DEBUG_MODE.current:
                 msg = (
                     "Failed to send update. More info may be available "
-                    "if you enabling debug mode by settings "
+                    "if you enabling debug mode by setting "
                     "`reactpy.config.REACTPY_DEBUG_MODE.current = True`."
                 )
                 logger.error(msg)

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

@@ -1,7 +1,6 @@
 from __future__ import annotations
 
 import json
-import logging
 from collections.abc import Mapping, Sequence
 from functools import wraps
 from typing import Any, Protocol, cast, overload
@@ -26,9 +25,6 @@
     VdomJson,
 )
 
-logger = logging.getLogger()
-
-
 VDOM_JSON_SCHEMA = {
     "$schema": "http://json-schema.org/draft-07/schema",
     "$ref": "#/definitions/element",
@@ -328,18 +324,18 @@ def _is_single_child(value: Any) -> bool:
 
 def _validate_child_key_integrity(value: Any) -> None:
     if hasattr(value, "__iter__") and not hasattr(value, "__len__"):
-        logger.error(
+        warn(
             f"Did not verify key-path integrity of children in generator {value} "
             "- pass a sequence (i.e. list of finite length) in order to verify"
         )
     else:
         for child in value:
             if isinstance(child, ComponentType) and child.key is None:
-                logger.error(f"Key not specified for child in list {child}")
+                warn(f"Key not specified for child in list {child}")
             elif isinstance(child, Mapping) and "key" not in child:
                 # remove 'children' to reduce log spam
                 child_copy = {**child, "children": _EllipsisRepr()}
-                logger.error(f"Key not specified for child in list {child_copy}")
+                warn(f"Key not specified for child in list {child_copy}")
 
 
 class _CustomVdomDictConstructor(Protocol):

src/py/reactpy/tests/test_core/test_hooks.py

@@ -1257,3 +1257,12 @@ def bad_effect():
             await layout.render()
             component_hook.latest.schedule_render()
             await layout.render()  # no error
+
+
+@pytest.mark.skipif(
+    not REACTPY_DEBUG_MODE.current,
+    reason="only check json serialization in debug mode",
+)
+def test_raise_for_non_json_attrs():
+    with pytest.raises(TypeError, match="JSON serializable"):
+        reactpy.html.div({"non_json_serializable_object": object()})