Merge pull request #5985 from richtja/settings_exception_fix

Better description of config syntax error
avocado-framework · Jul 31, 2024 · 698e9d3 · 698e9d3
2 parents 34b427e + 6f39aa6
commit 698e9d3
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/avocado/core/settings.py b/avocado/core/settings.py
@@ -222,7 +222,10 @@ def _as_list(value):
             return []
 
         if isinstance(value, str):
-            return ast.literal_eval(value)
+            try:
+                return ast.literal_eval(value)
+            except SyntaxError:
+                pass
 
         if isinstance(value, list):
             return value
@@ -628,7 +631,7 @@ def merge_with_configs(self):
             value = value[0]
             try:
                 self.update_option(namespace, value, convert=True)
-            except SyntaxError:
+            except (SyntaxError, ValueError):
                 raise SyntaxError(
                     f"Syntax error in config file {path}, please check the value {value} "
                 )