raft,rafttest: print panic arguments correctly

This commit makes the testing loggers print Panic and Fatal agruments before redirecting them to the panic() call. Previously they would be displayed in a non-human-readable way, as something like: panic: ([]interface {}) 0x1400000eb88 Signed-off-by: Pavel Kalinnikov <[email protected]>
etcd-io · Mar 9, 2023 · 574d2f1 · 574d2f1
1 parent d0fb0cd
commit 574d2f1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/node_util_test.go b/node_util_test.go
@@ -60,7 +60,7 @@ func (l *nodeTestHarness) Warningf(format string, v ...interface{}) {
 
 func (l *nodeTestHarness) Fatal(v ...interface{}) {
 	l.t.Error(v...)
-	panic(v)
+	panic(fmt.Sprint(v...))
 }
 
 func (l *nodeTestHarness) Fatalf(format string, v ...interface{}) {
@@ -70,7 +70,7 @@ func (l *nodeTestHarness) Fatalf(format string, v ...interface{}) {
 
 func (l *nodeTestHarness) Panic(v ...interface{}) {
 	l.t.Log(v...)
-	panic(v)
+	panic(fmt.Sprint(v...))
 }
 
 func (l *nodeTestHarness) Panicf(format string, v ...interface{}) {

diff --git a/rafttest/interaction_env_logger.go b/rafttest/interaction_env_logger.go
@@ -82,7 +82,7 @@ func (l *RedirectLogger) Errorf(format string, v ...interface{}) {
 
 func (l *RedirectLogger) Fatal(v ...interface{}) {
 	l.print(4, v...)
-	panic(v)
+	panic(fmt.Sprint(v...))
 }
 
 func (l *RedirectLogger) Fatalf(format string, v ...interface{}) {
@@ -92,7 +92,7 @@ func (l *RedirectLogger) Fatalf(format string, v ...interface{}) {
 
 func (l *RedirectLogger) Panic(v ...interface{}) {
 	l.print(4, v...)
-	panic(v)
+	panic(fmt.Sprint(v...))
 }
 
 func (l *RedirectLogger) Panicf(format string, v ...interface{}) {