← Back to team overview

checkbox-dev team mailing list archive

[PATCH 03/11] plainbox:secure:config: use Variable.validate() in Variable.__set__()

 

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@xxxxxxxxxxxxx>
---
 plainbox/plainbox/impl/secure/config.py | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/plainbox/plainbox/impl/secure/config.py b/plainbox/plainbox/impl/secure/config.py
index 02ce661..9917157 100644
--- a/plainbox/plainbox/impl/secure/config.py
+++ b/plainbox/plainbox/impl/secure/config.py
@@ -231,10 +231,7 @@ class Variable(INameTracking):
         :raises ValidationError: if the new value is incorrect
         """
         # Check it against all validators
-        for validator in self.validator_list:
-            message = validator(self, new_value)
-            if message is not None:
-                raise ValidationError(self, new_value, message)
+        self.validate(new_value)
         # Assign it to the backing store of the instance
         instance._set_variable(self.name, new_value)
 
-- 
1.9.0



References