vboot: have consistent output for testing macros
Extract print_passed() and print_failed() from test_eq(), and use them
consistently in all testing functions in test_common.h.
BRANCH=none
BUG=none
TEST=make runtests
Change-Id: I1d4dbf13c9ae1a8130ea7af2733044d0556fa974
Signed-off-by: Yu-Ping Wu <yupingso@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2217547
Reviewed-by: Joel Kitching <kitching@chromium.org>
diff --git a/tests/test_common.c b/tests/test_common.c
index 21740b6..7e994f3 100644
--- a/tests/test_common.c
+++ b/tests/test_common.c
@@ -17,16 +17,28 @@
int gTestAbortArmed = 0;
jmp_buf gTestJmpEnv;
+static void print_passed(const char *preamble, const char *desc,
+ const char *comment)
+{
+ fprintf(stderr, "%s: %s ... " COL_GREEN "PASSED\n" COL_STOP,
+ preamble, comment ? comment : desc);
+}
+
+static void print_failed(const char *preamble, const char *desc,
+ const char *comment)
+{
+ fprintf(stderr, "%s: %s ... " COL_RED "FAILED\n" COL_STOP,
+ preamble, comment ? comment : desc);
+}
+
int test_eq(int result, int expected,
const char *preamble, const char *desc, const char *comment)
{
if (result == expected) {
- fprintf(stderr, "%s: %s ... " COL_GREEN "PASSED\n" COL_STOP,
- preamble, comment ? comment : desc);
+ print_passed(preamble, desc, comment);
return 1;
} else {
- fprintf(stderr, "%s: %s ... " COL_RED "FAILED\n" COL_STOP,
- preamble, comment ? comment : desc);
+ print_failed(preamble, desc, comment);
fprintf(stderr, " Expected: %#x (%d), got: %#x (%d)\n",
expected, expected, result, result);
gTestSuccess = 0;
@@ -38,12 +50,10 @@
const char *preamble, const char *desc, const char *comment)
{
if (result != not_expected) {
- fprintf(stderr, "%s: %s, %s ... " COL_GREEN "PASSED\n" COL_STOP,
- preamble, desc, comment);
+ print_passed(preamble, desc, comment);
return 1;
} else {
- fprintf(stderr, "%s: %s, %s ... " COL_RED "FAILED\n" COL_STOP,
- preamble, desc, comment);
+ print_failed(preamble, desc, comment);
fprintf(stderr, " Didn't expect %#x (%d), but got it.\n",
not_expected, not_expected);
gTestSuccess = 0;
@@ -55,13 +65,11 @@
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);
+ print_passed(preamble, desc, comment);
return 1;
} else {
- fprintf(stderr, "%s: %s, %s ... " COL_RED "FAILED\n" COL_STOP,
- preamble, desc, comment);
- fprintf(stderr, " Expected: 0x%lx, got: 0x%lx\n",
+ print_failed(preamble, desc, comment);
+ fprintf(stderr, " Expected: %#lx, got: %#lx\n",
(long)expected, (long)result);
gTestSuccess = 0;
return 0;
@@ -72,13 +80,11 @@
const char *preamble, const char *desc, const char *comment)
{
if (result != not_expected) {
- fprintf(stderr, "%s: %s, %s ... " COL_GREEN "PASSED\n" COL_STOP,
- preamble, desc, comment);
+ print_passed(preamble, desc, comment);
return 1;
} else {
- fprintf(stderr, "%s: %s, %s ... " COL_RED "FAILED\n" COL_STOP,
- preamble, desc, comment);
- fprintf(stderr, " Didn't expect 0x%lx, but got it\n",
+ print_failed(preamble, desc, comment);
+ fprintf(stderr, " Didn't expect %#lx, but got it\n",
(long)not_expected);
gTestSuccess = 0;
return 0;
@@ -89,17 +95,15 @@
const char *preamble, const char *desc, const char *comment)
{
if (!result || !expected) {
- fprintf(stderr, "%s: %s, %s ... " COL_RED "FAILED\n" COL_STOP,
- preamble, desc, comment);
+ print_failed(preamble, desc, comment);
fprintf(stderr, " String compare with NULL\n");
gTestSuccess = 0;
return 0;
} else if (!strcmp(result, expected)) {
- fprintf(stderr, "%s: %s, %s ... " COL_GREEN "PASSED\n" COL_STOP,
- preamble, desc, comment);
+ print_passed(preamble, desc, comment);
return 1;
} else {
- fprintf(stderr, "%s " COL_RED "FAILED\n" COL_STOP, comment);
+ print_failed(preamble, desc, comment);
fprintf(stderr, " Expected: \"%s\", got: \"%s\"\n",
expected, result);
gTestSuccess = 0;
@@ -111,18 +115,17 @@
const char *preamble, const char *desc, const char *comment)
{
if (!result || !not_expected) {
- fprintf(stderr, "%s: %s, %s ... " COL_RED "FAILED\n" COL_STOP,
- preamble, desc, comment);
+ print_failed(preamble, desc, comment);
fprintf(stderr, " String compare with NULL\n");
gTestSuccess = 0;
return 0;
} else if (strcmp(result, not_expected)) {
+ print_passed(preamble, desc, comment);
fprintf(stderr, "%s: %s, %s ... " COL_GREEN "PASSED\n" COL_STOP,
preamble, desc, comment);
return 1;
} else {
- fprintf(stderr, "%s: %s, %s ... " COL_RED "FAILED\n" COL_STOP,
- preamble, desc, comment);
+ print_failed(preamble, desc, comment);
fprintf(stderr, " Didn't expect: \"%s\", but got it\n",
not_expected);
gTestSuccess = 0;
@@ -134,11 +137,9 @@
const char *preamble, const char *desc, const char *comment)
{
if (result == 0) {
- fprintf(stderr, "%s: %s ... " COL_GREEN "PASSED\n" COL_STOP,
- preamble, comment ? comment : desc);
+ print_passed(preamble, desc, comment);
} else {
- fprintf(stderr, "%s: %s ... " COL_RED "FAILED\n" COL_STOP,
- preamble, comment ? comment : desc);
+ print_failed(preamble, desc, comment);
fprintf(stderr, " Expected SUCCESS, got: %#x (%d)\n",
result, result);
gTestSuccess = 0;
@@ -150,11 +151,9 @@
const char *preamble, const char *desc, const char *comment)
{
if (result) {
- fprintf(stderr, "%s: %s, %s ... " COL_GREEN "PASSED\n" COL_STOP,
- preamble, desc, comment);
+ print_passed(preamble, desc, comment);
} else {
- fprintf(stderr, "%s: %s, %s ... " COL_RED "FAILED\n" COL_STOP,
- preamble, desc, comment);
+ print_failed(preamble, desc, comment);
fprintf(stderr, " Expected TRUE, got 0\n");
gTestSuccess = 0;
}
@@ -165,12 +164,10 @@
const char *preamble, const char *desc, const char *comment)
{
if (!result) {
- fprintf(stderr, "%s: %s, %s ... " COL_GREEN "PASSED\n" COL_STOP,
- preamble, desc, comment);
+ print_passed(preamble, desc, comment);
} else {
- fprintf(stderr, "%s: %s, %s ... " COL_RED "FAILED\n" COL_STOP,
- preamble, desc, comment);
- fprintf(stderr, " Expected FALSE, got: 0x%lx\n",
+ print_failed(preamble, desc, comment);
+ fprintf(stderr, " Expected FALSE, got: %#lx\n",
(long)result);
gTestSuccess = 0;
}
@@ -181,11 +178,9 @@
const char *preamble, const char *desc, const char *comment)
{
if (aborted) {
- fprintf(stderr, "%s: %s ... " COL_GREEN "PASSED\n" COL_STOP,
- preamble, comment ? comment : desc);
+ print_passed(preamble, desc, comment);
} else {
- fprintf(stderr, "%s: %s ... " COL_RED "FAILED\n" COL_STOP,
- preamble, comment ? comment : desc);
+ print_failed(preamble, desc, comment);
fprintf(stderr, " Expected ABORT, but did not get it\n");
gTestSuccess = 0;
}