diff options
| author | Denis Laxalde <denis.laxalde@logilab.fr> | 2019-07-26 16:13:02 +0200 |
|---|---|---|
| committer | Denis Laxalde <denis.laxalde@logilab.fr> | 2019-07-26 16:13:02 +0200 |
| commit | 131b6bd58b7aaed997576fe363da2c80fdcb0a67 (patch) | |
| tree | 84e18d013b9a822b55cf5dd83d119789f29c0caa /logilab | |
| parent | 0dc34dc2792b7c846b6d683cc251f05f065940e6 (diff) | |
| download | logilab-common-131b6bd58b7aaed997576fe363da2c80fdcb0a67.tar.gz | |
Raise a ValueError in case of invalid unit in textutils.apply_units()
The original KeyError comes from 57e242dffe89 which no justification. I
believe it was meant to be a ValueError as in other errors in this
function. Also, _ensure_correctly_typed() in cubicweb/web/formfields.py
will catch ValueError.
Adding a test for this.
Diffstat (limited to 'logilab')
| -rw-r--r-- | logilab/common/textutils.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/logilab/common/textutils.py b/logilab/common/textutils.py index 356b1a8..95a5e00 100644 --- a/logilab/common/textutils.py +++ b/logilab/common/textutils.py @@ -371,8 +371,8 @@ def apply_units(string, units, inter=None, final=float, blank_reg=_BLANK_RE, try: value *= units[unit.lower()] except KeyError: - raise KeyError('invalid unit %s. valid units are %s' % - (unit, units.keys())) + raise ValueError('invalid unit %s. valid units are %s' % + (unit, units.keys())) values.append(value) return final(sum(values)) |