Merge pull request gcanti#228 from gcanti/227

null Maybes should stringify to null, fix gcanti#227

CHANGELOG.md

@@ -12,6 +12,11 @@
 **Note**: Gaps between patch versions are faulty/broken releases.
 **Note**: A feature tagged as Experimental is in a high state of flux, you're at risk of it changing without notice.
 
+# v3.2.6
+
+- **Bug Fix**
+  - null Maybes should stringify to null, fix #227 (@gcanti)
+
 # v3.2.5
 
 - **Polish**

lib/create.js

@@ -6,7 +6,7 @@ var stringify = require('./stringify');
 // creates an instance of a type, handling the optional new operator
 module.exports = function create(type, value, path) {
   if (isType(type)) {
-    return !type.meta.identity && typeof value === 'object' ? new type(value, path): type(value, path);
+    return !type.meta.identity && typeof value === 'object' && value !== null ? new type(value, path): type(value, path);
   }
 
   if (process.env.NODE_ENV !== 'production') {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "tcomb",
-  "version": "3.2.5",
+  "version": "3.2.6",
   "description": "Type checking and DDD for JavaScript",
   "main": "index.js",
   "typings": "index.d.ts",

test/maybe.js

@@ -91,6 +91,12 @@ describe('t.maybe(type, [name])', function () {
       assert.ok(typeof T(undefined) === 'undefined');
     });
 
+    it('should play well with JSON.stringify (#227)', function () {
+      var thing = t.struct({ foo: t.maybe(t.struct({ bar: t.String })) });
+      assert.strictEqual(JSON.stringify(thing({foo: null})), '{"foo":null}');
+      assert.strictEqual(JSON.stringify(thing({foo: undefined})), '{}');
+    });
+
   });
 
   describe('is(x)', function () {