Make testlogger output "TestLogger" prefix (#24683)

wxiaoguang · web-flow · commit bfa0fc2c989b · 2023-05-12T14:20:29.000Z
Make testlogger output "TestLogger" prefix instead of "Unknown Test"
when there is no running test case.
diff --git 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() {

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,9 @@ func (w *testLoggerWriterCloser) Write(p []byte) (int, error) {`
`58`	`58`	`}`
`59`	`59`
`60`	`60`	`if t == nil \|\| *t == nil {`
`61`		`- return fmt.Fprintf(os.Stdout, "??? [Unknown Test] %s\n", p)`
	`61`	`+ // if there is no running test, the log message should be outputted to console, to avoid losing important information.`
	`62`	`+ // the "???" prefix is used to match the "===" and "+++" in PrintCurrentTest`
	`63`	`+ return fmt.Fprintf(os.Stdout, "??? [TestLogger] %s\n", p)`
`62`	`64`	`}`
`63`	`65`
`64`	`66`	`defer func() {`