feat!: drop required property support

Author: ivan-tymoshenko

index.js

@@ -319,7 +319,6 @@ function buildCode (location) {
   }
 
   const schema = location.schema
-  const required = schema.required || []
 
   let code = ''
 
@@ -351,23 +350,12 @@ function buildCode (location) {
         ${addComma}
         json += ${asString} + ':' + ${JSON.stringify(JSON.stringify(defaultValue))}
       `
-    } else if (required.includes(key)) {
-      code += `
-      } else {
-        throw new Error('${sanitized} is required!')
-      `
     }
-
     code += `
       }
     `
   })
 
-  for (const requiredProperty of required) {
-    if (schema.properties && schema.properties[requiredProperty] !== undefined) continue
-    code += `if (obj['${requiredProperty}'] === undefined) throw new Error('"${requiredProperty}" is required!')\n`
-  }
-
   return code
 }
 

test/allof.test.js

@@ -60,7 +60,7 @@ test('allOf: combine pattern properties', (t) => {
 })
 
 test('object with allOf and multiple schema on the allOf', (t) => {
-  t.plan(4)
+  t.plan(2)
 
   const schema = {
     title: 'object with allOf and multiple schema on the allOf',
@@ -95,22 +95,6 @@ test('object with allOf and multiple schema on the allOf', (t) => {
   }
   const stringify = build(schema)
 
-  try {
-    stringify({
-      id: 1
-    })
-  } catch (e) {
-    t.equal(e.message, '"name" is required!')
-  }
-
-  try {
-    stringify({
-      name: 'string'
-    })
-  } catch (e) {
-    t.equal(e.message, '"id" is required!')
-  }
-
   t.equal(stringify({
     id: 1,
     name: 'string'

test/required.test.js

@@ -1088,7 +1088,7 @@ test('toJSON forwards nullable types', (t) => {
 })
 
 test('toJSON supports required types', (t) => {
-  t.plan(2)
+  t.plan(1)
 
   const stringify = build({
     title: 'object of required primitive (json) types',
@@ -1134,11 +1134,6 @@ test('toJSON supports required types', (t) => {
   }
   const expected = '{"_bool":true,"_int":42,"_null":null,"_num":3.14,"_str":"whatever"}'
   t.equal(stringify(input), expected)
-
-  const invalidInput = {
-    toJSON () { return {} }
-  }
-  t.throws(() => { stringify(invalidInput) })
 })
 
 test('use toJSON recursively', (t) => {