fix(differ): should not affect recursiveEqual

Author: RexSkz

src/utils/diff-object.ts

@@ -59,8 +59,8 @@ const diffObject = (
 
     if (keyLeft === keyRight) {
       if (getType(lhs[keyLeft]) !== getType(rhs[keyRight])) {
-        const _resultLeft = formatValue(lhs[keyLeft]).split('\n');
-        const _resultRight = formatValue(rhs[keyRight]).split('\n');
+        const _resultLeft = formatValue(lhs[keyLeft], options.maxDepth, true).split('\n');
+        const _resultRight = formatValue(rhs[keyRight], options.maxDepth, true).split('\n');
         if (options.showModifications) {
           const maxLines = Math.max(_resultLeft.length, _resultRight.length);
           for (let i = _resultLeft.length; i < maxLines; i++) {
@@ -138,17 +138,17 @@ const diffObject = (
       } else {
         if (lhs[keyLeft] !== rhs[keyRight]) {
           if (options.showModifications) {
-            linesLeft.push({ level, type: 'modify', text: `"${keyLeft}": ${formatValue(lhs[keyLeft])}` });
-            linesRight.push({ level, type: 'modify', text: `"${keyRight}": ${formatValue(rhs[keyRight])}` });
+            linesLeft.push({ level, type: 'modify', text: `"${keyLeft}": ${formatValue(lhs[keyLeft], options.maxDepth)}` });
+            linesRight.push({ level, type: 'modify', text: `"${keyRight}": ${formatValue(rhs[keyRight], options.maxDepth)}` });
           } else {
-            linesLeft.push({ level, type: 'remove', text: `"${keyLeft}": ${formatValue(lhs[keyLeft])}` });
+            linesLeft.push({ level, type: 'remove', text: `"${keyLeft}": ${formatValue(lhs[keyLeft], options.maxDepth)}` });
             linesLeft.push({ level, type: 'equal', text: '' });
             linesRight.push({ level, type: 'equal', text: '' });
-            linesRight.push({ level, type: 'add', text: `"${keyRight}": ${formatValue(rhs[keyRight])}` });
+            linesRight.push({ level, type: 'add', text: `"${keyRight}": ${formatValue(rhs[keyRight], options.maxDepth)}` });
           }
         } else {
-          linesLeft.push({ level, type: 'equal', text: `"${keyLeft}": ${formatValue(lhs[keyLeft])}` });
-          linesRight.push({ level, type: 'equal', text: `"${keyRight}": ${formatValue(rhs[keyRight])}` });
+          linesLeft.push({ level, type: 'equal', text: `"${keyLeft}": ${formatValue(lhs[keyLeft], options.maxDepth)}` });
+          linesRight.push({ level, type: 'equal', text: `"${keyRight}": ${formatValue(rhs[keyRight], options.maxDepth)}` });
         }
       }
     } else if (keysLeft.length && keysRight.length) {

src/utils/format-value.spec.ts

@@ -14,8 +14,13 @@ describe('Utility function: formatValue', () => {
   });
 
   it('should work for array & object', () => {
-    expect(formatValue([1, 2, '3'])).toBe('[\n 1,\n 2,\n "3"\n]');
-    expect(formatValue({ a: 1, b: ['2', true] })).toBe('{\n "a": 1,\n "b": [\n  "2",\n  true\n ]\n}');
+    expect(formatValue([1, 2, '3'])).toBe('[1,2,"3"]');
+    expect(formatValue({ a: 1, b: ['2', true] })).toBe('{"a":1,"b":["2",true]}');
+  });
+
+  it('should work for array & object with pretty', () => {
+    expect(formatValue([1, 2, '3'], undefined, true)).toBe('[\n 1,\n 2,\n "3"\n]');
+    expect(formatValue({ a: 1, b: ['2', true] }, undefined, true)).toBe('{\n "a": 1,\n "b": [\n  "2",\n  true\n ]\n}');
   });
 
   it('should escape the characters when necessary for better display', () => {

src/utils/format-value.ts

@@ -1,11 +1,11 @@
 import stringify from './stringify';
 
-const formatValue = (value: any, depth = Infinity) => {
+const formatValue = (value: any, depth = Infinity, pretty = false) => {
   if (Number.isNaN(value) || value === null) {
     return 'null';
   }
   if (Array.isArray(value) || typeof value === 'object') {
-    return stringify(value, null, 1, depth);
+    return stringify(value, null, pretty ? 1 : null, depth);
   }
   if (typeof value === 'string') {
     return stringify(value);