feat(try-it-out): validation should consider readOnly

Author: mathis-m

src/core/utils.js

@@ -428,15 +428,19 @@ function validateValueBySchema(value, schema, requiredByParam, bypassRequiredChe
   let maxItems = schema.get("maxItems")
   let minItems = schema.get("minItems")
   let pattern = schema.get("pattern")
+  const isReadOnly = schema.get("readOnly", false)
+
+  let isRequired = isReadOnly
+    ? false
+    : requiredByParam || requiredBySchema === true
 
-  const schemaRequiresValue = requiredByParam || requiredBySchema === true
   const hasValue = value !== undefined && value !== null
-  const isValidEmpty = !schemaRequiresValue && !hasValue
+  const isValidEmpty = !isRequired && !hasValue
 
   const needsExplicitConstraintValidation = hasValue && type === "array"
 
   const requiresFurtherValidation =
-    schemaRequiresValue
+    isRequired
     || needsExplicitConstraintValidation
     || !isValidEmpty
 
@@ -471,7 +475,7 @@ function validateValueBySchema(value, schema, requiredByParam, bypassRequiredChe
 
   const passedAnyCheck = allChecks.some(v => !!v)
 
-  if (schemaRequiresValue && !passedAnyCheck && !bypassRequiredCheck) {
+  if (isRequired && !passedAnyCheck && !bypassRequiredCheck) {
     errors.push("Required field is not provided")
     return errors
   }
@@ -491,7 +495,8 @@ function validateValueBySchema(value, schema, requiredByParam, bypassRequiredChe
     }
     if(schema && schema.has("required") && isFunc(requiredBySchema.isList) && requiredBySchema.isList()) {
       requiredBySchema.forEach(key => {
-        if(objectVal[key] === undefined) {
+        const propIsReadOnly = schema.getIn(["properties", key, "readOnly"], false)
+        if (!propIsReadOnly && objectVal[key] === undefined) {
           errors.push({ propKey: key, error: "Required property not found" })
         }
       })