Merge pull request #26 from cjihrig/19

Add configuration settings, including disabling message truncating

Author: geek

lib/index.js

@@ -13,6 +13,9 @@ var internals = {
     count: 0
 };
 
+exports.settings = {
+    truncateMessages: true
+};
 
 exports.expect = function (value, prefix) {
 
@@ -269,7 +272,8 @@ internals.addMethod(['throw', 'throws'], function (/* type, message */) {
 internals.display = function (value) {
 
     var string = NodeUtil.inspect(value);
-    if (string.length <= 40) {
+
+    if (!exports.settings.truncateMessages || string.length <= 40) {
         return string;
     }
 

test/index.js

@@ -120,7 +120,7 @@ describe('expect()', function () {
         done();
     });
 
-    it('asserts on invalid condition (large array)', function (done) {
+    it('asserts on invalid condition (large array, display truncated)', function (done) {
 
         var exception = false;
         try {
@@ -134,7 +134,24 @@ describe('expect()', function () {
         done();
     });
 
-    it('asserts on invalid condition (large object)', function (done) {
+    it('asserts on invalid condition (large array, display not truncated)', function (done) {
+
+        var exception = false;
+        var origTruncate = Code.settings.truncateMessages;
+        try {
+            Code.settings.truncateMessages = false;
+            Code.expect([1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18]).to.be.a.string();
+        }
+        catch (err) {
+            Code.settings.truncateMessages = origTruncate;
+            exception = err;
+        }
+
+        Hoek.assert(exception.message === 'Expected [ 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18 ] to be a string but got \'array\'', exception);
+        done();
+    });
+
+    it('asserts on invalid condition (large object, display truncated)', function (done) {
 
         var exception = false;
         try {
@@ -148,7 +165,24 @@ describe('expect()', function () {
         done();
     });
 
-    it('asserts on invalid condition (long object values)', function (done) {
+    it('asserts on invalid condition (large object, display not truncated)', function (done) {
+
+        var exception = false;
+        var origTruncate = Code.settings.truncateMessages;
+        try {
+            Code.settings.truncateMessages = false;
+            Code.expect({ a: 1, b: 2, c: 3, d: 4, e: 5, f: 6, g: 7, h: 8, i: 9, j: 10 }).to.be.a.string();
+        }
+        catch (err) {
+            Code.settings.truncateMessages = origTruncate;
+            exception = err;
+        }
+
+        Hoek.assert(exception.message === 'Expected { a: 1, b: 2, c: 3, d: 4, e: 5, f: 6, g: 7, h: 8, i: 9, j: 10 } to be a string but got \'object\'', exception);
+        done();
+    });
+
+    it('asserts on invalid condition (long object values, display truncated)', function (done) {
 
         var exception = false;
         try {
@@ -162,7 +196,24 @@ describe('expect()', function () {
         done();
     });
 
-    it('asserts on invalid condition (long string)', function (done) {
+    it('asserts on invalid condition (long object values, display not truncated)', function (done) {
+
+        var exception = false;
+        var origTruncate = Code.settings.truncateMessages;
+        try {
+            Code.settings.truncateMessages = false;
+            Code.expect({ a: 12345678901234567890, b: 12345678901234567890 }).to.be.a.string();
+        }
+        catch (err) {
+            Code.settings.truncateMessages = origTruncate;
+            exception = err;
+        }
+
+        Hoek.assert(exception.message === 'Expected { a: 12345678901234567000, b: 12345678901234567000 } to be a string but got \'object\'', exception);
+        done();
+    });
+
+    it('asserts on invalid condition (long string, display truncated)', function (done) {
 
         var exception = false;
         try {
@@ -176,6 +227,23 @@ describe('expect()', function () {
         done();
     });
 
+    it('asserts on invalid condition (long string, display not truncated)', function (done) {
+
+        var exception = false;
+        var origTruncate = Code.settings.truncateMessages;
+        try {
+            Code.settings.truncateMessages = false;
+            Code.expect('{ a: 1, b: 2, c: 3, d: 4, e: 5, f: 6, g: 7, h: 8, i: 9, j: 10 }').to.be.a.number();
+        }
+        catch (err) {
+            Code.settings.truncateMessages = origTruncate;
+            exception = err;
+        }
+
+        Hoek.assert(exception.message === 'Expected \'{ a: 1, b: 2, c: 3, d: 4, e: 5, f: 6, g: 7, h: 8, i: 9, j: 10 }\' to be a number but got \'string\'', exception);
+        done();
+    });
+
     it('resets flags between chained assertions', function (done) {
 
         var exception = false;