fast-json-stringify 2.2.8 → 2.2.9

package/bench.js

@@ -58,11 +58,6 @@ const obj = {
 
 const multiArray = []
 
-const JSTR = require('json-strify')
-const JSTRStringify = JSTR(schemaCJS)
-const JSTRArray = JSTR(arraySchemaCJS)
-const JSTRInstance = JSTR()
-
 const CJS = require('compile-json-stringify')
 const CJSStringify = CJS(schemaCJS)
 const CJSStringifyArray = CJS(arraySchemaCJS)
@@ -90,9 +85,6 @@ for (i = 0; i < 1000; i++) {
 suite.add('FJS creation', function () {
   FJS(schema)
 })
-suite.add('JSTR creation', function () {
-  JSTR(schemaCJS)
-})
 suite.add('CJS creation', function () {
   CJS(schemaCJS)
 })
@@ -105,10 +97,6 @@ suite.add('fast-json-stringify array', function () {
   stringifyArray(multiArray)
 })
 
-suite.add('json-strify array', function () {
-  JSTRArray(multiArray)
-})
-
 suite.add('compile-json-stringify array', function () {
   CJSStringifyArray(multiArray)
 })
@@ -121,10 +109,6 @@ suite.add('fast-json-stringify long string', function () {
   stringifyString(str)
 })
 
-suite.add('json-strify long string', function () {
-  JSTRInstance(str)
-})
-
 suite.add('compile-json-stringify long string', function () {
   CJSStringifyString(str)
 })
@@ -137,10 +121,6 @@ suite.add('fast-json-stringify short string', function () {
   stringifyString('hello world')
 })
 
-suite.add('json-strify short string', function () {
-  JSTRInstance('hello world')
-})
-
 suite.add('compile-json-stringify short string', function () {
   CJSStringifyString('hello world')
 })
@@ -153,10 +133,6 @@ suite.add('fast-json-stringify obj', function () {
   stringify(obj)
 })
 
-suite.add('json-strify obj', function () {
-  JSTRStringify(obj)
-})
-
 suite.add('compile-json-stringify obj', function () {
   CJSStringify(obj)
 })

package/index.js

@@ -937,7 +937,7 @@ function buildObject (location, code, name) {
   return code
 }
 
-function buildArray (location, code, name) {
+function buildArray (location, code, name, key = null) {
   var schema = location.schema
   code += `
     function ${name} (obj) {
@@ -988,6 +988,14 @@ function buildArray (location, code, name) {
     result = nested(laterCode, name, '[i]', mergeLocation(location, { schema: schema.items }), undefined, true)
   }
 
+  if (key) {
+    code += `
+    if(!Array.isArray(obj)) {
+      throw new TypeError(\`Property '${key}' should be of type array, received '$\{obj}' instead.\`)
+    }
+    `
+  }
+
   code += `
     var l = obj.length
     var w = l - 1
@@ -1131,7 +1139,7 @@ function nested (laterCode, name, key, location, subKey, isArray) {
       break
     case 'array':
       funcName = asFuncName('$arr' + name + key + subKey) // eslint-disable-line
-      laterCode = buildArray(location, laterCode, funcName)
+      laterCode = buildArray(location, laterCode, funcName, key)
       code += `
         json += ${funcName}(obj${accessor})
       `

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fast-json-stringify",
-  "version": "2.2.8",
+  "version": "2.2.9",
   "description": "Stringify your JSON at max speed",
   "main": "index.js",
   "types": "index.d.ts",
@@ -33,7 +33,6 @@
     "benchmark": "^2.1.4",
     "compile-json-stringify": "^0.1.2",
     "is-my-json-valid": "^2.20.0",
-    "json-strify": "^0.1.7",
     "long": "^4.0.0",
     "moment": "^2.24.0",
     "pre-commit": "^1.2.2",

package/test/typesArray.test.js

@@ -417,3 +417,21 @@ test('object that is simultaneously a string and a json switched', (t) => {
   const valueObj = stringify({ simultaneously: { foo: likeObjectId } })
   t.is(valueObj, '{"simultaneously":{"foo":"hello"}}')
 })
+
+test('should throw an error when type is array and object is null', (t) => {
+  t.plan(1)
+  const schema = {
+    type: 'object',
+    properties: {
+      arr: {
+        type: 'array',
+        items: {
+          type: 'number'
+        }
+      }
+    }
+  }
+
+  const stringify = build(schema)
+  t.throws(() => stringify({ arr: null }), new TypeError('Property \'arr\' should be of type array, received \'null\' instead.'))
+})