linux-kernel - [PATCH -next] selftests: Add color to keywords of results report

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <20221103115828.49391-1-zhaogongyi@huawei.com>
Date:   Thu, 3 Nov 2022 19:58:28 +0800
From:   Zhao Gongyi <zhaogongyi@...wei.com>
To:     <linux-kselftest@...r.kernel.org>, <linux-kernel@...r.kernel.org>
CC:     <shuah@...nel.org>, Zhao Gongyi <zhaogongyi@...wei.com>
Subject: [PATCH -next] selftests: Add color to keywords of results report

For ksft_test_result_*()/ksft_exit_*(), we could add color to some
keywords like:

function	color
*pass*		green
*fail*		red
*skip*		yellow
*error*		magenta

And it is hoped to improve the readability of the output.

Signed-off-by: Zhao Gongyi <zhaogongyi@...wei.com>
---
 tools/testing/selftests/kselftest.h | 28 ++++++++++++++++++++--------
 1 file changed, 20 insertions(+), 8 deletions(-)

diff --git a/tools/testing/selftests/kselftest.h b/tools/testing/selftests/kselftest.h
index 33a0dbd26bd3..8c53a7520578 100644
--- a/tools/testing/selftests/kselftest.h
+++ b/tools/testing/selftests/kselftest.h
@@ -75,6 +75,18 @@
 #define KSFT_XPASS 3
 #define KSFT_SKIP  4

+#define COLOR_RED	"\033[1;31m"
+#define COLOR_GREEN	"\033[1;32m"
+#define COLOR_YELLOW	"\033[1;33m"
+#define COLOR_MAGENTA	"\033[1;35m"
+
+#define RESET		"\033[0m"
+
+#define RED(msg) COLOR_RED msg RESET
+#define GREEN(msg) COLOR_GREEN msg RESET
+#define YELLOW(msg) COLOR_YELLOW msg RESET
+#define MAGENTA(msg) COLOR_MAGENTA msg RESET
+
 /* counters */
 struct ksft_count {
 	unsigned int ksft_pass;
@@ -152,7 +164,7 @@ static inline void ksft_test_result_pass(const char *msg, ...)
 	ksft_cnt.ksft_pass++;

 	va_start(args, msg);
-	printf("ok %d ", ksft_test_num());
+	printf(GREEN("ok %d "), ksft_test_num());
 	errno = saved_errno;
 	vprintf(msg, args);
 	va_end(args);
@@ -166,7 +178,7 @@ static inline void ksft_test_result_fail(const char *msg, ...)
 	ksft_cnt.ksft_fail++;

 	va_start(args, msg);
-	printf("not ok %d ", ksft_test_num());
+	printf(RED("not ok %d "), ksft_test_num());
 	errno = saved_errno;
 	vprintf(msg, args);
 	va_end(args);
@@ -192,7 +204,7 @@ static inline void ksft_test_result_xfail(const char *msg, ...)
 	ksft_cnt.ksft_xfail++;

 	va_start(args, msg);
-	printf("ok %d # XFAIL ", ksft_test_num());
+	printf(RED("ok %d # XFAIL "), ksft_test_num());
 	errno = saved_errno;
 	vprintf(msg, args);
 	va_end(args);
@@ -206,7 +218,7 @@ static inline void ksft_test_result_skip(const char *msg, ...)
 	ksft_cnt.ksft_xskip++;

 	va_start(args, msg);
-	printf("ok %d # SKIP ", ksft_test_num());
+	printf(YELLOW("ok %d # SKIP "), ksft_test_num());
 	errno = saved_errno;
 	vprintf(msg, args);
 	va_end(args);
@@ -221,7 +233,7 @@ static inline void ksft_test_result_error(const char *msg, ...)
 	ksft_cnt.ksft_error++;

 	va_start(args, msg);
-	printf("not ok %d # error ", ksft_test_num());
+	printf(MAGENTA("not ok %d # error "), ksft_test_num());
 	errno = saved_errno;
 	vprintf(msg, args);
 	va_end(args);
@@ -266,7 +278,7 @@ static inline int ksft_exit_fail_msg(const char *msg, ...)
 	va_list args;

 	va_start(args, msg);
-	printf("Bail out! ");
+	printf(RED("Bail out! "));
 	errno = saved_errno;
 	vprintf(msg, args);
 	va_end(args);
@@ -302,9 +314,9 @@ static inline int ksft_exit_skip(const char *msg, ...)
 	 */
 	if (ksft_plan || ksft_test_num()) {
 		ksft_cnt.ksft_xskip++;
-		printf("ok %d # SKIP ", 1 + ksft_test_num());
+		printf(YELLOW("ok %d # SKIP "), 1 + ksft_test_num());
 	} else {
-		printf("1..0 # SKIP ");
+		printf(YELLOW("1..0 # SKIP "));
 	}
 	if (msg) {
 		errno = saved_errno;
--
2.17.1