fix cutting off arrays with more than 100 items in anyOf and oneOf (#237)

* fix cutting off arrays with more than 100 items in anyOf and oneOf

* fix cutting off arrays with more than 100 items in anyOf and oneOf

* fix cutting off arrays with more than 100 items in anyOf and oneOf

Author: johakr

index.js

@@ -1079,7 +1079,7 @@ function nested (laterCode, name, key, schema, externalSchema, fullSchema, subKe
         schema.anyOf.forEach((s, index) => {
           var nestedResult = nested(laterCode, name, key, s, externalSchema, fullSchema, subKey !== '' ? subKey : 'i' + index, isArray)
           code += `
-            ${index === 0 ? 'if' : 'else if'}(ajv.validate(${require('util').inspect(s, { depth: null })}, obj${accessor}))
+            ${index === 0 ? 'if' : 'else if'}(ajv.validate(${require('util').inspect(s, { depth: null, maxArrayLength: null })}, obj${accessor}))
               ${nestedResult.code}
           `
           laterCode = nestedResult.laterCode
@@ -1092,7 +1092,7 @@ function nested (laterCode, name, key, schema, externalSchema, fullSchema, subKe
         schema.oneOf.forEach((s, index) => {
           var nestedResult = nested(laterCode, name, key, s, externalSchema, fullSchema, subKey !== '' ? subKey : 'i' + index, isArray)
           code += `
-            ${index === 0 ? 'if' : 'else if'}(ajv.validate(${require('util').inspect(s, { depth: null })}, obj${accessor}))
+            ${index === 0 ? 'if' : 'else if'}(ajv.validate(${require('util').inspect(s, { depth: null, maxArrayLength: null })}, obj${accessor}))
               ${nestedResult.code}
           `
           laterCode = nestedResult.laterCode

test/anyof.test.js

@@ -435,3 +435,29 @@ test('anyOf looks for all of the array items', (t) => {
     t.fail()
   }
 })
+
+test('anyOf with enum with more than 100 entries', (t) => {
+  t.plan(1)
+
+  const schema = {
+    title: 'type array that may have any of declared items',
+    type: 'array',
+    items: {
+      anyOf: [
+        {
+          type: 'string',
+          enum: ['EUR', 'USD', ...(new Set([...new Array(200)].map(() => Math.random().toString(36).substr(2, 3)))).values()]
+        },
+        { type: 'null' }
+      ]
+    }
+  }
+  const stringify = build(schema)
+
+  try {
+    const value = stringify(['EUR', 'USD', null])
+    t.is(value, '["EUR","USD",null]')
+  } catch (e) {
+    t.fail()
+  }
+})

test/oneof.test.js

@@ -397,3 +397,29 @@ test('oneOf and $ref: multiple levels should throw at build.', (t) => {
     t.fail(e)
   }
 })
+
+test('oneOf with enum with more than 100 entries', (t) => {
+  t.plan(1)
+
+  const schema = {
+    title: 'type array that may have one of declared items',
+    type: 'array',
+    items: {
+      oneOf: [
+        {
+          type: 'string',
+          enum: ['EUR', 'USD', ...(new Set([...new Array(200)].map(() => Math.random().toString(36).substr(2, 3)))).values()]
+        },
+        { type: 'null' }
+      ]
+    }
+  }
+  const stringify = build(schema)
+
+  try {
+    const value = stringify(['EUR', 'USD', null])
+    t.is(value, '["EUR","USD",null]')
+  } catch (e) {
+    t.fail()
+  }
+})