readonly works properly just in the first validation

Given the next code:

from cerberus import Validator


v = Validator({
    'id': {'type': 'integer', 'readonly': True},
})


v.validate({'id': 1}) # False
v.validate({'id': 1}) # True
v.validate({'id': 1}) # True
v.validate({'id': 1}) # True

Just the first call validates correctly and return False. The subsequent calls to validate(...) return True. This just happen to me with property readonly. It looks like the validation works just on time.

Is this an intentional behavior of this library?

[funkyfuture]

@nicolaiarocci can you please add a bug label and assign this issue to the 1.2 milestone? i’m in fear it may be forgotten otherwise as there has been no feedback so far.

[dkellner]

It seems to be enough to reset is_normalized when a new document is coming in - all tests are passing now. The fact that this actually was not done before makes me wonder if there exist more bugs because of this. I will investigate further.

@funkyfuture What do you think about this?

diff --git a/cerberus/tests/test_validation.py b/cerberus/tests/test_validation.py
index ec97de5..2c31a74 100644
--- a/cerberus/tests/test_validation.py
+++ b/cerberus/tests/test_validation.py
@@ -85,6 +85,13 @@ def test_readonly_field_first_rule():
     assert 'read-only' in v.errors['a_readonly_number'][0]
 
 
+def test_repeated_readonly(validator):
+    # https://github.com/pyeve/cerberus/issues/311
+    validator.schema = {'id': {'readonly': True}}
+    assert_fail({'id': 0}, validator=validator)
+    assert_fail({'id': 0}, validator=validator)
+
+
 def test_readonly_field_with_default_value():
     schema = {
         'created': {
diff --git a/cerberus/validator.py b/cerberus/validator.py
index f28f74c..fd6b6eb 100644
--- a/cerberus/validator.py
+++ b/cerberus/validator.py
@@ -492,6 +492,8 @@ class Validator(object):
         self.document_error_tree = errors.DocumentErrorTree()
         self.schema_error_tree = errors.SchemaErrorTree()
         self.document = copy(document)
+        if not self.is_child:
+            self.is_normalized = False
 
         if schema is not None:
             self.schema = DefinitionSchema(self, schema)