add mocha .actual / .expected string support for all assertion values

so that diffs work
thyms · Jul 30, 2012 · 05132d3 · 05132d3
1 parent 9feffef
commit 05132d3
Showing 1 changed file with 13 additions and 10 deletions.
diff --git a/lib/should.js b/lib/should.js
@@ -119,16 +119,19 @@ Assertion.prototype = {
 
   assert: function(expr, msg, negatedMsg, expected){
     var msg = this.negate ? negatedMsg : msg
-      , ok = this.negate ? !expr : expr;
-
-    if (!ok) {
-      throw new AssertionError({
-          message: msg
-        , actual: this.obj
-        , expected: expected
-        , stackStartFunction: this.assert
-      });
-    }
+      , ok = this.negate ? !expr : expr
+      , obj = this.obj;
+
+    if (ok) return;
+    if ('string' != typeof obj) obj = i(obj);
+    if ('string' != typeof expected) expected = i(expected);
+
+    throw new AssertionError({
+        message: msg
+      , actual: obj
+      , expected: expected
+      , stackStartFunction: this.assert
+    });
   },
 
   /**