Merge pull request #37 from SimenB/fastSafeStringify-undefined

If a value is undefined, it shouldn't be passed to fastSafeStringify

Author: mcollina

index.js

@@ -237,7 +237,8 @@ function additionalProperty (schema, externalSchema, fullSchema) {
   let code = ''
   if (ap === true) {
     return `
-        json += $asString(keys[i]) + ':' + fastSafeStringify(obj[keys[i]]) + ','
+        if (obj[keys[i]] !== undefined)
+          json += $asString(keys[i]) + ':' + fastSafeStringify(obj[keys[i]]) + ','
     `
   }
   if (ap['$ref']) {

test/additionalProperties.test.js

@@ -254,3 +254,20 @@ test('nested additionalProperties set to true', (t) => {
   let obj = { ap: { value: 'string', someNumber: 42 } }
   t.equal('{"ap":{"value":"string","someNumber":42}}', stringify(obj))
 })
+
+test('field passed to fastSafeStringify as undefined should be removed', (t) => {
+  t.plan(1)
+  const stringify = build({
+    title: 'nested additionalProperties=true',
+    type: 'object',
+    properties: {
+      ap: {
+        type: 'object',
+        additionalProperties: true
+      }
+    }
+  })
+
+  let obj = { ap: { value: 'string', someNumber: undefined } }
+  t.equal('{"ap":{"value":"string"}}', stringify(obj))
+})