commit d0e3d7e1fab83606acc08036290f989c55068ee7
author Daisuke Nojiri <dnojiri@chromium.org> Tue Sep 13 16:15:02 2016 -0700
committer chrome-bot <chrome-bot@chromium.org> Sat Sep 24 16:22:34 2016 -0700
tree 2acf561cac6cb48f4507383955e1b8c33581f253
parent 42b74d2677786cf9e135873b574428bdd74bd3fb

test: Print either 'comment' or 'desc'

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

BUG=none
BRANCH=none
TEST=make runtests

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

tests/test_common.c

diff --git a/tests/test_common.c b/tests/test_common.c
index e6c32f6..dae485f 100644
--- a/tests/test_common.c
+++ b/tests/test_common.c
@@ -134,11 +134,11 @@
 	      const char *preamble, const char *desc, const char *comment)
 {
   if (result == 0) {
-    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);
   } 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 SUCCESS, got: 0x%x (%d)\n", result, result);
     gTestSuccess = 0;
   }