aboutsummaryrefslogtreecommitdiff
path: root/wa/utils
diff options
context:
space:
mode:
authorWaleed El-Geresy <waleed.el-geresy@arm.com>2018-06-14 16:15:33 +0100
committersetrofim <setrofim@gmail.com>2018-06-18 20:46:56 +0100
commit578d2d3a16f455da6bf96f7cf59600c3f92af8ed (patch)
treeaa395013f517787c5d689f582860c8cddac8c4f2 /wa/utils
parent88731fe4895fd442719d1d0f08f6a1ca12360148 (diff)
utils/serializer: Handle empty file extensions
The ValueError does not given enough information in the case of an empty file extension. Add a special case to handle files with no extension and prompt the user.
Diffstat (limited to 'wa/utils')
-rw-r--r--wa/utils/serializer.py6
1 files changed, 6 insertions, 0 deletions
diff --git a/wa/utils/serializer.py b/wa/utils/serializer.py
index 47b78340..066696d1 100644
--- a/wa/utils/serializer.py
+++ b/wa/utils/serializer.py
@@ -299,6 +299,12 @@ def load(s, fmt='json', *args, **kwargs):
def _read_pod(fh, fmt=None):
if fmt is None:
fmt = os.path.splitext(fh.name)[1].lower().strip('.')
+ if fmt == '':
+ # Special case of no given file extension
+ message = ("Could not determine format " +
+ "from file extension for \"{}\". ".format(getattr(fh, 'name', '<none>')) +
+ "Please specify it or modify the fmt parameter.")
+ raise ValueError(message)
if fmt == 'yaml':
return yaml.load(fh)
elif fmt == 'json':