From eb292ebccd3881c1bfe19d96d30f0e354e894de9 Mon Sep 17 00:00:00 2001
From: Andreas Pelme <andreas@pelme.se>
Date: Sat, 14 Sep 2024 15:06:45 +0200
Subject: [PATCH] Use TypeError instead of ValueError where appropriate.

---
 htpy/__init__.py         | 10 +++++-----
 tests/test_attributes.py |  8 ++++----
 tests/test_children.py   |  8 ++++----
 3 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/htpy/__init__.py b/htpy/__init__.py
index 46ee5c0..997a846 100644
--- a/htpy/__init__.py
+++ b/htpy/__init__.py
@@ -50,7 +50,7 @@ def _class_names_for_items(items: t.Any) -> t.Any:
 
 def _id_class_names_from_css_str(x: t.Any) -> dict[str, Attribute]:
     if not isinstance(x, str):
-        raise ValueError(f"id/class strings must be str. got {x}")
+        raise TypeError(f"id/class strings must be str. got {x}")
 
     if "#" in x and "." in x and x.find("#") > x.find("."):
         raise ValueError("id (#) must be specified before classes (.)")
@@ -85,7 +85,7 @@ def _kwarg_attribute_name(name: str) -> str:
 def _generate_attrs(raw_attrs: dict[str, Attribute]) -> Iterable[tuple[str, Attribute]]:
     for key, value in raw_attrs.items():
         if not isinstance(key, str):  # pyright: ignore [reportUnnecessaryIsInstance]
-            raise ValueError("Attribute key must be a string")
+            raise TypeError("Attribute key must be a string")
 
         if value is False or value is None:
             continue
@@ -99,7 +99,7 @@ def _generate_attrs(raw_attrs: dict[str, Attribute]) -> Iterable[tuple[str, Attr
 
         else:
             if not isinstance(value, str | int | _HasHtml):
-                raise ValueError(f"Attribute value must be a string or an integer , got {value!r}")
+                raise TypeError(f"Attribute value must be a string or an integer , got {value!r}")
 
             yield _force_escape(key), _force_escape(value)
 
@@ -197,7 +197,7 @@ def _iter_node_context(x: Node, context_dict: dict[Context[t.Any], t.Any]) -> It
         for child in x:
             yield from _iter_node_context(child, context_dict)
     else:
-        raise ValueError(f"{x!r} is not a valid child element")
+        raise TypeError(f"{x!r} is not a valid child element")
 
 
 @functools.lru_cache(maxsize=300)
@@ -300,7 +300,7 @@ def _validate_children(children: t.Any) -> None:
             _validate_children(child)
         return
 
-    raise ValueError(f"{children!r} is not a valid child element")
+    raise TypeError(f"{children!r} is not a valid child element")
 
 
 class Element(BaseElement):
diff --git a/tests/test_attributes.py b/tests/test_attributes.py
index 703a73f..9abdb74 100644
--- a/tests/test_attributes.py
+++ b/tests/test_attributes.py
@@ -159,7 +159,7 @@ def test_id_class_bad_format() -> None:
 
 
 def test_id_class_bad_type() -> None:
-    with pytest.raises(ValueError, match="id/class strings must be str. got {'oops': 'yes'}"):
+    with pytest.raises(TypeError, match="id/class strings must be str. got {'oops': 'yes'}"):
         div({"oops": "yes"}, {})  # type: ignore
 
 
@@ -188,8 +188,8 @@ def test_attribute_priority() -> None:
 
 @pytest.mark.parametrize("not_an_attr", [1234, b"foo", object(), object, 1, 0, None])
 def test_invalid_attribute_key(not_an_attr: t.Any) -> None:
-    with pytest.raises(ValueError, match="Attribute key must be a string"):
-        str(div({not_an_attr: "foo"}))
+    with pytest.raises(TypeError, match="Attribute key must be a string"):
+        div({not_an_attr: "foo"})
 
 
 @pytest.mark.parametrize(
@@ -197,5 +197,5 @@ def test_invalid_attribute_key(not_an_attr: t.Any) -> None:
     [12.34, b"foo", object(), object],
 )
 def test_invalid_attribute_value(not_an_attr: t.Any) -> None:
-    with pytest.raises(ValueError, match="Attribute value must be a string"):
+    with pytest.raises(TypeError, match="Attribute value must be a string"):
         div(foo=not_an_attr)
diff --git a/tests/test_children.py b/tests/test_children.py
index 2e35528..8d016c2 100644
--- a/tests/test_children.py
+++ b/tests/test_children.py
@@ -240,13 +240,13 @@ class SomeClass:
 
 @pytest.mark.parametrize("not_a_child", _invalid_children)
 def test_invalid_child_direct(not_a_child: t.Any) -> None:
-    with pytest.raises(ValueError, match="is not a valid child element"):
+    with pytest.raises(TypeError, match="is not a valid child element"):
         div[not_a_child]
 
 
 @pytest.mark.parametrize("not_a_child", _invalid_children)
 def test_invalid_child_nested_iterable(not_a_child: t.Any) -> None:
-    with pytest.raises(ValueError, match="is not a valid child element"):
+    with pytest.raises(TypeError, match="is not a valid child element"):
         div[[not_a_child]]
 
 
@@ -256,7 +256,7 @@ def test_invalid_child_lazy_callable(not_a_child: t.Any) -> None:
     Ensure proper exception is raised for lazily evaluated invalid children.
     """
     element = div[lambda: not_a_child]
-    with pytest.raises(ValueError, match="is not a valid child element"):
+    with pytest.raises(TypeError, match="is not a valid child element"):
         str(element)
 
 
@@ -270,5 +270,5 @@ def gen() -> t.Any:
         yield not_a_child
 
     element = div[gen()]
-    with pytest.raises(ValueError, match="is not a valid child element"):
+    with pytest.raises(TypeError, match="is not a valid child element"):
         str(element)