feat(power-assert-formatter): treat ambiguous east asian character as fullwidth (= 2)

Author: twada

lib/default-options.js

@@ -7,7 +7,7 @@ module.exports = function defaultOptions () {
         anonymous: 'Object',
         circular: '#@Circular#',
         lineSeparator: '\n',
-        ambiguousEastAsianCharWidth: 1,
+        ambiguousEastAsianCharWidth: 2,
         renderers: [
             './built-in/file',
             './built-in/assertion',

test/power_assert_formatter_test.js

@@ -844,7 +844,36 @@ suite('power-assert-formatter', function () {
             '         false               ',
             '  '
         ]);
+    });
+
 
+    test('Ambiguous EastAsianWidth', function () {
+        var suffix1 = '…',
+            suffix2 = '☆';
+        assertPowerAssertContextFormatting(function () {
+            eval(weave('assert("※ただしイケメンに限る" + suffix1 === "○ただしイケメンに限る" + suffix2);'));
+        }, [
+            '  # /path/to/some_test.js:1',
+            '  ',
+            '  assert("※ただしイケメンに限る" + suffix1 === "○ただしイケメンに限る" + suffix2)',
+            '                                  | |       |                            | |       ',
+            '                                  | |       |                            | "☆"    ',
+            '                                  | "…"    false                        "○ただしイケメンに限る☆"',
+            '                                  "※ただしイケメンに限る…"                       ',
+            '  ',
+            '  --- [string] "○ただしイケメンに限る" + suffix2',
+            '  +++ [string] "※ただしイケメンに限る" + suffix1',
+            '  @@ -1,5 +1,5 @@',
+            '  -○',
+            '  +※',
+            '   ただしイ',
+            '  @@ -8,5 +8,5 @@',
+            '   ンに限る',
+            '  -☆',
+            '  +…',
+            '  ',
+            '  '
+        ]);
     });
 
 

test/string_width_test.js

@@ -44,4 +44,22 @@
             assert.equal(stringWidth(strWithSurrogatePair), 10);
         });
     });
+
+    suite('ambiguous EastAsianWidth', function () {
+        setup(function () {
+            this.strWithAmbiguousEastAsian = '※ただしイケメンに限る';
+        });
+        test('when set to 1', function () {
+            var widthOf = formatter.stringWidth({
+                ambiguousEastAsianCharWidth: 1
+            });
+            assert.equal(widthOf(this.strWithAmbiguousEastAsian), 21);
+        });
+        test('when set to 2', function () {
+            var widthOf = formatter.stringWidth({
+                ambiguousEastAsianCharWidth: 2
+            });
+            assert.equal(widthOf(this.strWithAmbiguousEastAsian), 22);
+        });
+    });
 }));