Merge pull request #339 from kayjan/feature/node-name-mandatory

Make Node name mandatory

Author: kayjan

CHANGELOG.md

@@ -7,6 +7,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ## [Unreleased]
 ### Changed:
 - Tree Search: Search relative path allows unix folder expression for leading sep symbol.
+- Node: Updated Node signature to reflect that name is mandatory.
 
 ## [0.23.0] - 2024-12-26
 ### Changed:

bigtree/node/node.py

@@ -79,7 +79,7 @@ class Node(basenode.BaseNode):
 
     """
 
-    def __init__(self, name: str = "", sep: str = "/", **kwargs: Any):
+    def __init__(self, name: str, sep: str = "/", **kwargs: Any):
         self.name = name
         self._sep = sep
         super().__init__(**kwargs)

tests/node/test_node.py

@@ -48,7 +48,7 @@ def tearDown(self):
 
     def test_empty_node_name_error(self):
         with pytest.raises(exceptions.TreeError) as exc_info:
-            node.Node()
+            node.Node("")
         assert str(exc_info.value) == Constants.ERROR_NODE_NAME
 
     def test_set_parent(self):