From c91267235c0e787d180c6c4388ff88c6dc99dd56 Mon Sep 17 00:00:00 2001
From: Mike Pennisi <mike@mikepennisi.com>
Date: Tue, 21 Apr 2015 13:15:19 -0400
Subject: [PATCH] Extend assertion error messages

When invoked without a custom assertion message, `assert.sameValue` and
`assert.notSameValue` automatically create a message that describes the
actual and expected values.

Extend both assertion methods to also include this information in cases
where a custom message has been specified.
---
 harness/assert.js | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/harness/assert.js b/harness/assert.js
index 6fafa2df5f..35abaf8f6d 100644
--- a/harness/assert.js
+++ b/harness/assert.js
@@ -25,8 +25,13 @@ assert.sameValue = function (actual, expected, message) {
     }
 
     if (message === undefined) {
-        message = 'Expected SameValue(' + String(actual) + ', ' + String(expected) + ') to be true';
+        message = '';
+    } else {
+        message += ' ';
     }
+
+    message += 'Expected SameValue(' + String(actual) + ', ' + String(expected) + ') to be true';
+
     $ERROR(message);
 };
 
@@ -36,8 +41,13 @@ assert.notSameValue = function (actual, unexpected, message) {
     }
 
     if (message === undefined) {
-        message = 'Expected SameValue(' + String(actual) + ', ' + String(unexpected) + ') to be false';
+        message = '';
+    } else {
+        message += ' ';
     }
+
+    message += 'Expected SameValue(' + String(actual) + ', ' + String(unexpected) + ') to be false';
+
     $ERROR(message);
 };