fix(stringifier): do not truncate if string length is short enough

Author: twada

strategies.js

@@ -109,8 +109,14 @@ function truncate (size) {
         return function (acc, x) {
             var orig = acc.push, ret;
             acc.push = function (str) {
-                var truncated = str.substring(0, size);
-                orig.call(acc, truncated + acc.options.snip);
+                var savings = str.length - size,
+                    truncated;
+                if (savings <= size) {
+                    orig.call(acc, str);
+                } else {
+                    truncated = str.substring(0, size);
+                    orig.call(acc, truncated + acc.options.snip);
+                }
             };
             ret = next(acc, x);
             acc.push = orig;

test/customization_test.js

@@ -32,6 +32,7 @@ describe('strategies', function () {
     beforeEach(function () {
         this.student = new Student('tom', 10, 'M');
         this.anonymous = new AnonStudent('mary', 9, 'F');
+        this.longNameStudent = new Student('the_long_name_man', 18, 'M');
     });
 
     it('always', function () {
@@ -182,7 +183,19 @@ describe('strategies', function () {
                 return true;
             })
         };
-        assert.equal(stringify(this.student, null, handlers), 'Student{name:"to..(snip),age:10,gender:"M"}');
+        assert.equal(stringify(this.longNameStudent, null, handlers), 'Student{name:"th..(snip),age:18,gender:"M"}');
+    });
+
+    it('do not truncate if string length is short enough', function () {
+        var handlers = {
+            'Student': s.object(function (kvp) {
+                if (kvp.key === 'name') {
+                    return 3;
+                }
+                return true;
+            })
+        };
+        assert.equal(stringify(this.student, null, handlers), 'Student{name:"tom",age:10,gender:"M"}');
     });
 
     it('per-property truncate bare handler', function () {
@@ -194,7 +207,7 @@ describe('strategies', function () {
                 return true;
             })
         };
-        assert.equal(stringify(this.student, null, handlers), 'Student{name:"to..(snip),age:10,gender:"M"}');
+        assert.equal(stringify(this.longNameStudent, null, handlers), 'Student{name:"th..(snip),age:18,gender:"M"}');
     });