typescript tests

mobxjs · Sep 18, 2019 · c93c329 · c93c329
1 parent 9a51779
commit c93c329
Show file tree

Hide file tree

Showing 4 changed files with 80 additions and 11 deletions.
diff --git a/lib/es/serializr.js b/lib/es/serializr.js
@@ -446,6 +446,7 @@ function serializeWithSchema(schema, obj) {
         if (propDef === false)
             return
         var jsonValue = propDef.serializer(obj[key], key, obj);
+        console.warn({schema, obj, key, value: obj[key], jsonValue});
         if (jsonValue === SKIP){
             return
         }
@@ -1084,6 +1085,38 @@ function object(modelSchema, additionalArgs) {
     return result
 }
 
+/**
+ * Optional indicates that this model property shouldn't be serialized if it isn't present.
+ *
+ * @example
+ * createModelSchema(Todo, {
+ *     title: optional(primitive()),
+ * });
+ *
+ * console.dir(serialize(new Todo()));
+ * // {}
+ *
+ * @param {PropSchema} propSchema propSchema to (de)serialize the contents of this field
+ * @returns {PropSchema}
+ */
+function optional(name, propSchema) {
+    propSchema = (!propSchema || propSchema === true)  ? _defaultPrimitiveProp : propSchema;
+    invariant(isPropSchema(propSchema), "expected prop schema as second argument");
+    const propSerializer = propSchema.serializer;
+    invariant(typeof propSerializer === "function", "expected prop schema to have a callable serializer");
+    function serializer(...args) {
+        const result = propSerializer(...args);
+        if (result === undefined) {
+            return SKIP
+        }
+        return result
+    }
+    return {
+        ...propSchema,
+        serializer,
+    }
+}
+
 function createDefaultRefLookup(modelSchema) {
     return function resolve(uuid, cb, context) {
         context.rootContext.await(modelSchema, uuid, cb);
@@ -1434,9 +1467,4 @@ function raw(additionalArgs) {
     return result
 }
 
-/*
- * ## Managing model schemas
- */
-// ~ deprecated
-
-export { SKIP, alias, cancelDeserialize, object as child, createModelSchema, createSimpleSchema, custom, date, deserialize, getDefaultModelSchema, identifier, list, map, mapAsArray, object, primitive, raw, reference as ref, reference, serializable, serialize, serializeAll, setDefaultModelSchema, update };
+export { SKIP, alias, cancelDeserialize, object as child, createModelSchema, createSimpleSchema, custom, date, deserialize, getDefaultModelSchema, identifier, list, map, mapAsArray, object, optional, primitive, raw, reference as ref, reference, serializable, serialize, serializeAll, setDefaultModelSchema, update };
diff --git a/lib/serializr.js b/lib/serializr.js
@@ -453,6 +453,7 @@
             if (propDef === false)
                 return
             var jsonValue = propDef.serializer(obj[key], key, obj);
+            console.warn({schema, obj, key, value: obj[key], jsonValue});
             if (jsonValue === SKIP){
                 return
             }
@@ -1091,6 +1092,38 @@
         return result
     }
 
+    /**
+     * Optional indicates that this model property shouldn't be serialized if it isn't present.
+     *
+     * @example
+     * createModelSchema(Todo, {
+     *     title: optional(primitive()),
+     * });
+     *
+     * console.dir(serialize(new Todo()));
+     * // {}
+     *
+     * @param {PropSchema} propSchema propSchema to (de)serialize the contents of this field
+     * @returns {PropSchema}
+     */
+    function optional(name, propSchema) {
+        propSchema = (!propSchema || propSchema === true)  ? _defaultPrimitiveProp : propSchema;
+        invariant(isPropSchema(propSchema), "expected prop schema as second argument");
+        const propSerializer = propSchema.serializer;
+        invariant(typeof propSerializer === "function", "expected prop schema to have a callable serializer");
+        function serializer(...args) {
+            const result = propSerializer(...args);
+            if (result === undefined) {
+                return SKIP
+            }
+            return result
+        }
+        return {
+            ...propSchema,
+            serializer,
+        }
+    }
+
     function createDefaultRefLookup(modelSchema) {
         return function resolve(uuid, cb, context) {
             context.rootContext.await(modelSchema, uuid, cb);
@@ -1441,11 +1474,6 @@
         return result
     }
 
-    /*
-     * ## Managing model schemas
-     */
-    // ~ deprecated
-
     exports.SKIP = SKIP;
     exports.alias = alias;
     exports.cancelDeserialize = cancelDeserialize;
@@ -1461,6 +1489,7 @@
     exports.map = map;
     exports.mapAsArray = mapAsArray;
     exports.object = object;
+    exports.optional = optional;
     exports.primitive = primitive;
     exports.raw = raw;
     exports.ref = reference;

diff --git a/src/core/serialize.js b/src/core/serialize.js
@@ -59,6 +59,7 @@ export function serializeWithSchema(schema, obj) {
         if (propDef === false)
             return
         var jsonValue = propDef.serializer(obj[key], key, obj)
+        console.warn({schema, obj, key, value: obj[key], jsonValue})
         if (jsonValue === SKIP){
             return
         }

diff --git a/test/typescript/ts.ts b/test/typescript/ts.ts
@@ -6,6 +6,7 @@ import {
     map,
     mapAsArray,
     object,
+    optional,
     identifier,
     reference,
     primitive,
@@ -74,6 +75,9 @@ test("typescript class with constructor params", t => {
         @serializable(alias("identifier", identifier()))
         public id: string;
 
+        @serializable(alias("desc", optional()))
+        public description?: string;
+
         @serializable(alias("width", true))
         public width: number
 
@@ -95,13 +99,20 @@ test("typescript class with constructor params", t => {
     a.someNumber = 123;
 
     let json = serialize(a);
+    t.equal(false, json.hasOwnProperty("desc"));
+    t.equal(false, json.hasOwnProperty("description"));
     const b = deserialize(Rectangle, json);
     t.equal(a.id, b.id);
     t.equal(a.width, b.width);
     t.equal(a.height, b.height);
     t.equal(a.someNumber, b.someNumber);
     t.equal(b.getArea(), 200);
 
+    a.description = "example";
+    json = serialize(a);
+    t.equal("example", json["desc"]);
+    t.equal(false, json.hasOwnProperty("description"));
+
     t.end();
 });