Merge branch 'feat/assert-to-val-error' into 'master'

feat[node]: validation error instead of assertion

See merge request utilities/ntc!15

ntc/__init__.py

@@ -2,4 +2,4 @@
 from .leaf import *
 from .node import *
 
-__version__ = "0.7.0"
+__version__ = "0.8.0"

ntc/node.py

@@ -6,7 +6,15 @@
 
 import yaml
 
-from ntc.errors import MissingRequired, NodeFrozenError, NodeReassignment, SaveError, SchemaError, SchemaFrozenError
+from ntc.errors import (
+    MissingRequired,
+    NodeFrozenError,
+    NodeReassignment,
+    SaveError,
+    SchemaError,
+    SchemaFrozenError,
+    ValidationError,
+)
 from ntc.utils import add_yaml_str_representer, import_module, merge_cfg_module
 
 from .leaf import CfgLeaf
@@ -153,7 +161,10 @@ def validate(self) -> None:
         """
         validators = [CfgNode.validate_required] + self._validators
         for validator in validators:
-            validator(self)
+            try:
+                validator(self)
+            except AssertionError as exc:
+                raise ValidationError from exc
 
     def run_hooks(self) -> None:
         """

tests/test_post_load.py

@@ -3,6 +3,7 @@
 import pytest
 
 from ntc import CN, SchemaFrozenError
+from ntc.errors import ValidationError
 
 DATA_DIR = Path(__file__).parent / "data" / "post_load"
 
@@ -14,12 +15,12 @@ def test_transform():
 
 
 def test_validate():
-    with pytest.raises(AssertionError):
+    with pytest.raises(ValidationError):
         CN.load(DATA_DIR / "validate_changes.py")
 
 
 def test_bad_validate():
-    with pytest.raises(AssertionError):
+    with pytest.raises(ValidationError):
         CN.load(DATA_DIR / "bad_validate.py")