object diff: Fixed indentation of objects to be removed.

unexpectedjs · Jan 21, 2015 · c55f3f6 · c55f3f6
1 parent 32b641c
commit c55f3f6
Show file tree

Hide file tree

Showing 2 changed files with 29 additions and 0 deletions.
diff --git a/lib/types.js b/lib/types.js
@@ -200,6 +200,7 @@ module.exports = function (expect) {
                     if (conflicting) {
                         if (!(key in expected)) {
                             annotation.error('should be removed');
+                            isInlineDiff = true;
                         } else {
                             var keyDiff = diff(actual[key], expected[key]);
                             if (!keyDiff || (keyDiff && !keyDiff.inline)) {

diff --git a/test/unexpected.spec.js b/test/unexpected.spec.js
@@ -1728,6 +1728,34 @@ describe('unexpected', function () {
                    "}");
         });
 
+        it('indents removed objects correctly', function () {
+            var str = 'abcdefghijklmnopqrstuvwxyz';
+            expect(function () {
+                expect({foo: {a: str, b: str, c: str, d: str, e: str}}, 'to equal', {});
+            }, 'to throw',
+                'expected\n' +
+                '{\n' +
+                '  foo: {\n' +
+                "    a: 'abcdefghijklmnopqrstuvwxyz',\n" +
+                "    b: 'abcdefghijklmnopqrstuvwxyz',\n" +
+                "    c: 'abcdefghijklmnopqrstuvwxyz',\n" +
+                "    d: 'abcdefghijklmnopqrstuvwxyz',\n" +
+                "    e: 'abcdefghijklmnopqrstuvwxyz'\n" +
+                '  }\n' +
+                '}\n' +
+                'to equal {}\n' +
+                '\n' +
+                '{\n' +
+                '  foo: {\n' +
+                "    a: 'abcdefghijklmnopqrstuvwxyz',\n" +
+                "    b: 'abcdefghijklmnopqrstuvwxyz',\n" +
+                "    c: 'abcdefghijklmnopqrstuvwxyz',\n" +
+                "    d: 'abcdefghijklmnopqrstuvwxyz',\n" +
+                "    e: 'abcdefghijklmnopqrstuvwxyz'\n" +
+                '  } // should be removed\n' +
+                '}');
+        });
+
         describe('with a custom type', function () {
             function MysteryBox(value) {
                 this.propertyName = 'prop' + Math.floor(1000 * Math.random());