openstack/deb-python-warlock
Code Issues Proposed changes

Merge pull request #11 from isethi/validation

Browse Source 
Add description to exception messages
This commit is contained in:
Brian Waldon 2013-09-14 08:39:07 -07:00
commit 60c667e1fd
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
warlock/model.py
View File

@ -44,8 +44,9 @@ class Model(dict):
mutation[key] = value mutation[key] = value
try: try:
self.validate(mutation) self.validate(mutation)
except exceptions.ValidationError: except exceptions.ValidationError as exc:
msg = "Unable to set '%s' to '%s'" % (key, value) msg = ("Unable to set '%s' to '%s'. Reason: %s"
% (key, value, str(exc)))
raise exceptions.InvalidOperation(msg) raise exceptions.InvalidOperation(msg)
dict.__setitem__(self, key, value) dict.__setitem__(self, key, value)
@ -57,8 +58,9 @@ class Model(dict):
del mutation[key] del mutation[key]
try: try:
self.validate(mutation) self.validate(mutation)
except exceptions.ValidationError: except exceptions.ValidationError as exc:
msg = "Unable to delete attribute '%s'" % (key) msg = ("Unable to delete attribute '%s'. Reason: %s"
% (key, str(exc)))
raise exceptions.InvalidOperation(msg) raise exceptions.InvalidOperation(msg)
dict.__delitem__(self, key) dict.__delitem__(self, key)