Merge pull request #19 from lengstrom/main

made ValidationError handling more verbose
GuillaumeLeclerc · Jun 29, 2022 · 53dbda2 · 53dbda2
2 parents b39083a + a598697
commit 53dbda2
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/fastargs/config.py b/fastargs/config.py
@@ -180,7 +180,11 @@ def __getitem__(self, path):
         if value is None and not param.required:
             return value
 
-        return param.validate(value)
+        try:
+            return param.validate(value)
+        except ValidationError as e:
+            print(f'Issue when typechecking argument psyh `{".".join(path)}`:')
+            raise e
 
 
     def get(self):

diff --git a/fastargs/decorators.py b/fastargs/decorators.py
@@ -25,7 +25,9 @@ def __call__(self, *args, **kwargs):
                 path = ns + path
             if alias in kwargs:  # User overrode this argument
                 continue
+
             value = config[path]
+
             if value is not None:
                 filled_args[alias] = value
 

diff --git a/fastargs/param.py b/fastargs/param.py
@@ -12,7 +12,6 @@ def __init__(self, checker, desc='', default=None, required=False,
         self.section = None
 
     def __str__(self):
-
         result = ""
 
         if self.required:
@@ -37,5 +36,6 @@ def validate(self, value):
         try:
             return self.checker.check(value)
         except Exception:
-            raise ValidationError()
+            msg = f'value `{value}` does not fit checker for `{self.checker.help()}`'
+            raise ValidationError(msg)