From bfa0fc2c989b662cea2f171ae345b0d4e358fdf1 Mon Sep 17 00:00:00 2001
From: wxiaoguang <wxiaoguang@gmail.com>
Date: Fri, 12 May 2023 22:20:29 +0800
Subject: [PATCH] Make testlogger output "TestLogger" prefix (#24683)
Make testlogger output "TestLogger" prefix instead of "Unknown Test"
when there is no running test case.
---
modules/testlogger/testlogger.go | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/modules/testlogger/testlogger.go b/modules/testlogger/testlogger.go
index bf912f41dc..cc80e86c81 100644
--- a/modules/testlogger/testlogger.go
+++ b/modules/testlogger/testlogger.go
@@ -58,7 +58,9 @@ func (w *testLoggerWriterCloser) Write(p []byte) (int, error) {
}
if t == nil || *t == nil {
- return fmt.Fprintf(os.Stdout, "??? [Unknown Test] %s\n", p)
+ // if there is no running test, the log message should be outputted to console, to avoid losing important information.
+ // the "???" prefix is used to match the "===" and "+++" in PrintCurrentTest
+ return fmt.Fprintf(os.Stdout, "??? [TestLogger] %s\n", p)
}
defer func() {