commit 93159a225083f134bace7d279cd84e46d5606350
author Daisuke Nojiri <dnojiri@chromium.org> Fri Sep 23 13:09:41 2016 -0700
committer chrome-bot <chrome-bot@chromium.org> Wed Oct 05 17:11:30 2016 -0700
tree 3f8a56fd55ff712e61a715d214760d491b4d2d35
parent c3f8de399bc089f973f6766e5dcd06922d5639cd

test: Make TEST_EQ print either 'comment' or 'desc'

Currently, test macros print out 'desc' regardless 'comment' is specified
or not. This patch makes TEST_EQ print 'desc' only if 'comment' is not
supplied.

BUG=none
BRANCH=none
TEST=make runtests

Change-Id: I9cc3c9a9561534352ae0315dfea983f2c212b909
Signed-off-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/388859
Reviewed-by: Randall Spangler <rspangler@chromium.org>

tests/test_common.c

diff --git a/tests/test_common.c b/tests/test_common.c
index dae485f..0a76ccd 100644
--- a/tests/test_common.c
+++ b/tests/test_common.c
@@ -22,12 +22,12 @@
 	    const char *preamble, const char *desc, const char *comment)
 {
   if (result == expected) {
-    fprintf(stderr, "%s: %s, %s ... " COL_GREEN "PASSED\n" COL_STOP,
-	    preamble, desc, comment);
+    fprintf(stderr, "%s: %s ... " COL_GREEN "PASSED\n" COL_STOP,
+	    preamble, comment ? comment : desc);
     return 1;
   } else {
-    fprintf(stderr, "%s: %s, %s ... " COL_RED "FAILED\n" COL_STOP,
-	    preamble, desc, comment);
+    fprintf(stderr, "%s: %s ... " COL_RED "FAILED\n" COL_STOP,
+	    preamble, comment ? comment : desc);
     fprintf(stderr, "  Expected: 0x%x (%d), got: 0x%x (%d)\n",
 	    expected, expected, result, result);
     gTestSuccess = 0;