Merge pull request #90 from tdebarochez/master

Fix subject.toString is not a function

Author: Phillip Clark

index.js

@@ -131,7 +131,7 @@
       return 'array';
     } else if (Object.prototype.toString.call(subject) === '[object Date]') {
       return 'date';
-    } else if (typeof subject.toString !== 'undefined' && /^\/.*\//.test(subject.toString())) {
+    } else if (typeof subject.toString === 'function' && /^\/.*\//.test(subject.toString())) {
       return 'regexp';
     }
     return 'object';

test/tests.js

@@ -532,4 +532,23 @@ describe('deep-diff', function() {
         });
     });
 
+    describe('subject.toString is not a function', function() {
+        var lhs = {
+          left: 'yes',
+          right: 'no',
+        };
+        var rhs = {
+          left: {
+            toString: true,
+          },
+          right: 'no',
+        };
+
+        it('should not throw a TypeError', function() {
+            var diff = deep.diff(lhs, rhs);
+
+            expect(diff.length).to.be(1);
+        });
+    });
+
 });