expose only location instead of Object value (#574)

Author: Uzlopak

index.js

@@ -734,8 +734,12 @@ function buildMultiTypeSerializer (location, input) {
       }
     }
   })
+  let schemaRef = location.getSchemaRef()
+  if (schemaRef.startsWith(rootSchemaId)) {
+    schemaRef = schemaRef.replace(rootSchemaId, '')
+  }
   code += `
-    else throw new Error(\`The value $\{JSON.stringify(${input})} does not match schema definition.\`)
+    else throw new TypeError(\`The value of '${schemaRef}' does not match schema definition.\`)
   `
 
   return code
@@ -854,8 +858,13 @@ function buildValue (location, input) {
       `
     }
 
+    let schemaRef = location.getSchemaRef()
+    if (schemaRef.startsWith(rootSchemaId)) {
+      schemaRef = schemaRef.replace(rootSchemaId, '')
+    }
+
     code += `
-      else throw new Error(\`The value $\{JSON.stringify(${input})} does not match schema definition.\`)
+      else throw new TypeError(\`The value of '${schemaRef}' does not match schema definition.\`)
     `
     return code
   }

test/any.test.js

@@ -152,3 +152,80 @@ test('empty schema on anyOf', (t) => {
   t.equal(stringify({ kind: 'Foo', value: true }), '{"kind":"Foo","value":true}')
   t.equal(stringify({ kind: 'Foo', value: 'hello' }), '{"kind":"Foo","value":"hello"}')
 })
+
+test('should throw a TypeError with the path to the key of the invalid value /1', (t) => {
+  t.plan(1)
+
+  // any on Foo codepath.
+  const schema = {
+    anyOf: [
+      {
+        type: 'object',
+        properties: {
+          kind: {
+            type: 'string',
+            enum: ['Foo']
+          },
+          value: {}
+        }
+      },
+      {
+        type: 'object',
+        properties: {
+          kind: {
+            type: 'string',
+            enum: ['Bar']
+          },
+          value: {
+            type: 'number'
+          }
+        }
+      }
+    ]
+  }
+
+  const stringify = build(schema)
+
+  t.throws(() => stringify({ kind: 'Baz', value: 1 }), new TypeError('The value of \'#\' does not match schema definition.'))
+})
+
+test('should throw a TypeError with the path to the key of the invalid value /2', (t) => {
+  t.plan(1)
+
+  // any on Foo codepath.
+  const schema = {
+    type: 'object',
+    properties: {
+      data: {
+        anyOf: [
+          {
+            type: 'object',
+            properties: {
+              kind: {
+                type: 'string',
+                enum: ['Foo']
+              },
+              value: {}
+            }
+          },
+          {
+            type: 'object',
+            properties: {
+              kind: {
+                type: 'string',
+                enum: ['Bar']
+              },
+              value: {
+                type: 'number'
+              }
+            }
+          }
+        ]
+      }
+    }
+  }
+
+  const stringify = build(schema)
+
+  t.throws(() => stringify({ data: { kind: 'Baz', value: 1 } }), new TypeError('The value of \'#/properties/data\' does not match schema definition.'))
+})

test/array.test.js

@@ -283,7 +283,6 @@ test('array items is a list of schema and additionalItems is false /1', (t) => {
   }
 
   const stringify = build(schema)
-
   t.throws(() => stringify({ foo: ['foo', 'bar'] }), new Error('Item at 1 does not match schema definition.'))
 })
 

test/multi-type-serializer.test.js

@@ -0,0 +1,19 @@
+'use strict'
+
+const test = require('tap').test
+const build = require('..')
+
+test('should throw a TypeError with the path to the key of the invalid value', (t) => {
+  t.plan(1)
+  const schema = {
+    type: 'object',
+    properties: {
+      num: {
+        type: ['number']
+      }
+    }
+  }
+
+  const stringify = build(schema)
+  t.throws(() => stringify({ num: { bla: 123 } }), new TypeError('The value of \'#/properties/num\' does not match schema definition.'))
+})