Merge pull request node-modules#12 from node-modules/strict-json

feat(JSON): add strict JSON parse

README.md

@@ -185,6 +185,13 @@ var res = utils.try(function () {
 // {error: Error, value: undefined}
 ```
 
+### JSON
+
+```
+var obj = utils.strictJSONparse('"hello"');
+// will throw when JSON string is not object
+```
+
 ## benchmark
 
 * [jsperf: access log date format](http://jsperf.com/access-log-date-format)

lib/json.js

@@ -0,0 +1,19 @@
+/*!
+ * utility - lib/crypto.js
+ *
+ * MIT Licensed
+ */
+
+"use strict";
+
+/**
+ * Module dependencies.
+ */
+
+exports.strictJSONParse = function (str) {
+  var obj = JSON.parse(str);
+  if (!obj || typeof obj !== 'object') {
+    throw new Error('JSON string is not object');
+  }
+  return obj;
+};

lib/utility.js

@@ -20,6 +20,7 @@ copy(require('./function'))
 .and(require('./number'))
 .and(require('./string'))
 .and(require('./array'))
+.and(require('./json'))
 .and(require('./date'))
 .and(require('./map'))
 .and(require('./web'))

test/json.test.js

@@ -0,0 +1,43 @@
+/**!
+ * utility - test/json.test.js
+ *
+ * MIT Licensed
+ *
+ * Authors:
+ *   fengmk2 <[email protected]> (http://fengmk2.github.com)
+ */
+
+'use strict';
+
+/**
+ * Module dependencies.
+ */
+
+var utils = require('../');
+
+describe('json.test.js', function () {
+  describe('strictJSONParse()', function () {
+    it('should parse normal json ok', function () {
+      var obj = utils.strictJSONParse('{"foo": "bar"}');
+      obj.should.eql({foo: 'bar'});
+    });
+
+    it('should parse error when invalid json', function () {
+      (function () {
+        utils.strictJSONParse('{');
+      }).should.throw();
+    });
+
+    it('should parse error when non-object json', function () {
+      (function () {
+        utils.strictJSONParse('"hello"');
+      }).should.throw();
+    });
+
+    it('should parse error when null json', function () {
+      (function () {
+        utils.strictJSONParse('null');
+      }).should.throw();
+    });
+  });
+});