wrote test function for _is_node_field_valid()

src/cript/nodes/util/__init__.py

@@ -286,7 +286,7 @@ def _is_node_field_valid(node_type_list: list) -> bool:
     """
 
     # TODO consider having exception handling for the dict
-    if isinstance(node_type_list, list) and len(node_type_list) == 1 and isinstance(node_type_list[0], str):
+    if isinstance(node_type_list, list) and len(node_type_list) == 1 and isinstance(node_type_list[0], str) and node_type_list[0]:
         return True
     else:
         return False

tests/nodes/test_utils.py

@@ -0,0 +1,18 @@
+from cript.nodes.util import _is_node_field_valid
+
+
+def test_is_node_field_valid() -> None:
+    """
+    test the `_is_node_field_valid()` function to be sure it does the node type check correctly
+
+    checks both in places it should be valid and invalid
+    """
+    assert _is_node_field_valid(node_type_list=["Project"]) is True
+
+    assert _is_node_field_valid(node_type_list=["Project", "Material"]) is False
+
+    assert _is_node_field_valid(node_type_list=[""]) is False
+
+    assert _is_node_field_valid(node_type_list="Project") is False
+
+    assert _is_node_field_valid(node_type_list=[]) is False