Merge pull request #41 from p1c2u/fix/readonly-and-write-only-on-jsonschema4-fix

readOnly and writeOnly on jsonschema4 fix

Author: p1c2u

openapi_schema_validator/validators.py

@@ -63,6 +63,13 @@ def __init__(self, *args, **kwargs):
         self.write = kwargs.pop('write', None)
         super(OAS30Validator, self).__init__(*args, **kwargs)
 
+    def evolve(self, **kwargs):
+        # jsonschema4 interface compatibility workaround
+        validator = super(OAS30Validator, self).evolve(**kwargs)
+        validator.read = self.read
+        validator.write = self.write
+        return validator
+
     def iter_errors(self, instance, _schema=None):
         if _schema is None:
             # creates a copy by value from schema to prevent mutation

tests/integration/test_validators.py

@@ -196,6 +196,9 @@ def test_required_read_only(self):
             validator.validate({"another_prop": "hello"})
         validator = OAS30Validator(schema, format_checker=oas30_format_checker,
                                    write=True)
+        with pytest.raises(ValidationError,
+                           match="Tried to write read-only property with hello"):
+            validator.validate({"some_prop": "hello"})
         assert validator.validate({"another_prop": "hello"}) is None
 
     def test_required_write_only(self):
@@ -217,6 +220,9 @@ def test_required_write_only(self):
             validator.validate({"another_prop": "hello"})
         validator = OAS30Validator(schema, format_checker=oas30_format_checker,
                                    read=True)
+        with pytest.raises(ValidationError,
+                           match="Tried to read write-only property with hello"):
+            validator.validate({"some_prop": "hello"})
         assert validator.validate({"another_prop": "hello"}) is None
 
     def test_oneof_required(self):