Improve test coverage (#18)

rhendric · blakeembrey · commit 8f55d174a19c · 2018-04-07T19:33:02.000+09:00
diff --git a/javascript-stringify.js b/javascript-stringify.js
@@ -195,18 +195,11 @@
     '[object Boolean]': function (boolean) {
       return 'new Boolean(' + boolean + ')';
     },
-    '[object Uint8Array]': function (array, indent) {
-      return 'new Uint8Array(' + stringifyArray(array) + ')';
-    },
     '[object Set]': function (array, indent, next) {
-      if (typeof Array.from === 'function') {
-        return 'new Set(' + stringify(Array.from(array), indent, next) + ')';
-      } else return undefined;
+      return 'new Set(' + stringify(Array.from(array), indent, next) + ')';
     },
     '[object Map]': function (array, indent, next) {
-      if (typeof Array.from === 'function') {
-        return 'new Map(' + stringify(Array.from(array), indent, next) + ')';
-      } else return undefined;
+      return 'new Map(' + stringify(Array.from(array), indent, next) + ')';
     },
     '[object RegExp]': String,
     '[object Function]': String,
diff --git a/test.js b/test.js
@@ -99,6 +99,10 @@ describe('javascript-stringify', function () {
       describe('Error', function () {
         it('should stringify', test(new Error('test'), "new Error('test')"));
       });
+
+      describe('unknown native type', function () {
+        it('should be omitted', test({ k: process }, '{}'));
+      });
     });
 
     describe('ES6', function () {
@@ -183,6 +187,18 @@ describe('javascript-stringify', function () {
 
       expect(result).to.equal('(function(){var x={a:{}};x.b=x.a;return x;}())');
     });
+
+    it('should restore repeated values with indentation', function () {
+      var obj = {}
+      var child = {};
+
+      obj.a = child;
+      obj.b = child;
+
+      var result = stringify(obj, null, 2, { references: true })
+
+      expect(result).to.equal('(function () {\nvar x = {\n  a: {}\n};\nx.b = x.a;\nreturn x;\n}())');
+    });
   });
 
   describe('custom spaces', function () {
@@ -299,5 +315,10 @@ describe('javascript-stringify', function () {
       'should get part of the object',
       test(obj, '{a:{b:{}}}', null, { maxDepth: 2 })
     );
+
+    it(
+      'should get part of the object when tracking references',
+      test(obj, '{a:{b:{}}}', null, { maxDepth: 2, references: true })
+    );
   });
 });
