diff options
author | Waleed El-Geresy <waleed.el-geresy@arm.com> | 2018-06-14 16:15:33 +0100 |
---|---|---|
committer | setrofim <setrofim@gmail.com> | 2018-06-18 20:46:56 +0100 |
commit | 578d2d3a16f455da6bf96f7cf59600c3f92af8ed (patch) | |
tree | aa395013f517787c5d689f582860c8cddac8c4f2 /wa/utils | |
parent | 88731fe4895fd442719d1d0f08f6a1ca12360148 (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.py | 6 |
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': |