anyOf like empty schema

Author: nikitarui92

lib/index.js

@@ -34,7 +34,7 @@ function serialize(resource, schema, options = {}) {
 }
 
 function findObjectDef(propertyDef) {
-  const { type, anyOf, properties } = propertyDef;
+  const { type, properties } = propertyDef;
 
   if (type === 'object' && properties) {
     return propertyDef;
@@ -44,20 +44,6 @@ function findObjectDef(propertyDef) {
     return findObjectDef(propertyDef.items);
   }
 
-  if (anyOf) {
-    const props = anyOf
-      .map(findObjectDef)
-      .filter(Boolean)
-      .map(item => item.properties)
-      .reduce((acc, curr) => ({ ...acc, ...curr }), {});
-    if (Object.keys(props).length > 0) {
-      return {
-        type: 'object',
-        properties: props,
-      };
-    }
-  }
-
   return null;
 }
 
@@ -66,6 +52,9 @@ function isObjectLike(value) {
 }
 
 function toJSON(value) {
+  if (Array.isArray(value)) {
+    return value.map(toJSON);
+  }
   return value && typeof value.toJSON === 'function' ? value.toJSON() : value;
 }
 

tests/all.test.js

@@ -1,7 +1,6 @@
 const Sequelize = require('sequelize');
 const { createModel, DUMMY_VALUES } = require('./init');
 
-const sequelize = new Sequelize('sqlite::memory:');
 const serialize = require('../lib/index');
 
 describe('Serializers', () => {
@@ -133,7 +132,9 @@ describe('Serializers', () => {
 
     expect(serialize(instanceB, schema)).toEqual({
       a: 'x',
-      modelA: { b: 777, c: true },
+      modelA: {
+        a: 'x', b: 777, c: true, id: null,
+      },
     });
   });
 
@@ -183,21 +184,25 @@ describe('Serializers', () => {
     };
 
     const instanceA = new ModelA(DUMMY_VALUES);
-    const instanceB1 = new ModelB(DUMMY_VALUES);
+    const instanceB1 = new ModelB({ a: 'x' });
     const instanceB2 = new ModelB(DUMMY_VALUES);
     instanceA.modelsB = [instanceB1, instanceB2];
 
     expect(serialize(instanceA, schema)).toEqual({
       a: 'x',
       modelsB: [
-        { b: 777, c: true },
-        { b: 777, c: true },
+        {
+          a: 'x', id: null,
+        },
+        {
+          a: 'x', b: 777, c: true, id: null,
+        },
       ],
     });
   });
 
   it('Object schema without properties', () => {
-    const TestModel = createModel();
+    const TestModel = createModel('TestModel', { a: Sequelize.DataTypes.JSONB });
 
     const schema = {
       type: 'object',
@@ -216,8 +221,14 @@ describe('Serializers', () => {
       additionalProperties: false,
     };
 
-    const instance = new TestModel({ ...DUMMY_VALUES, a: {} });
-    expect(serialize(instance, schema)).toEqual({ a: {} });
+    const instance = new TestModel({ ...DUMMY_VALUES, a: DUMMY_VALUES });
+    expect(serialize(instance, schema)).toEqual({
+      a: {
+        a: 'x',
+        b: 777,
+        c: true,
+      },
+    });
   });
 
   it('Resource as empty array', () => {
@@ -242,15 +253,8 @@ describe('Serializers', () => {
     });
   });
 
-  it('anyOf with conflicting subproperties', () => {
-    class ModelA extends Sequelize.Model { }
-
-    ModelA.init(
-      {
-        a: Sequelize.DataTypes.JSONB,
-      },
-      { sequelize },
-    );
+  it('anyOf with conflicting nested properties', () => {
+    const ModelA = createModel('ModelA', { a: Sequelize.DataTypes.JSONB });
 
     const schema = {
       type: 'object',

tests/init.js

@@ -1,26 +1,21 @@
 const Sequelize = require('sequelize');
 
-const sequelize = new Sequelize('sqlite::memory:');
+const sequelize = new Sequelize('sqlite::memory:', { logging: false });
 
 const DUMMY_VALUES = {
   a: 'x',
   b: 777,
   c: true,
 };
 
-function createModel() {
-  class TestModel extends Sequelize.Model {}
-
-  TestModel.init(
+function createModel(modelName = 'TestModel', props = {}) {
+  return sequelize.define(modelName,
     {
       a: Sequelize.DataTypes.STRING,
       b: Sequelize.DataTypes.INTEGER,
       c: Sequelize.DataTypes.BOOLEAN,
-    },
-    { sequelize },
-  );
-
-  return TestModel;
+      ...props,
+    });
 }
 
 module.exports = {