Fix: pkg/gobash/gobash_test.go:21:11: non-constant format string in call to (*testing.common).Logf

Author: Eric-Guo

pkg/aicli/chatgpt/client_test.go

@@ -46,7 +46,7 @@ func TestClient_SendStream(t *testing.T) {
 	ctx, _ := context.WithTimeout(context.Background(), 5*time.Second)
 	reply := client.SendStream(ctx, "Which model did you use to answer the question?")
 	for content := range reply.Content {
-		fmt.Printf(content)
+		fmt.Print(content)
 	}
 	if reply.Err != nil {
 		t.Log(reply.Err)

pkg/aicli/deepseek/client_test.go

@@ -46,7 +46,7 @@ func TestClient_SendStream(t *testing.T) {
 	ctx, _ := context.WithTimeout(context.Background(), 10*time.Second)
 	answer := client.SendStream(ctx, genericRoleDescZH)
 	for content := range answer.Content {
-		fmt.Printf(content)
+		fmt.Print(content)
 	}
 	if answer.Err != nil {
 		t.Log(answer.Err)

pkg/aicli/gemini/client_test.go

@@ -41,7 +41,7 @@ func TestClient_SendStream(t *testing.T) {
 	ctx, _ := context.WithTimeout(context.Background(), 5*time.Second)
 	reply := client.SendStream(ctx, "Which model did you use to answer the question?")
 	for content := range reply.Content {
-		fmt.Printf(content)
+		fmt.Print(content)
 	}
 	if reply.Err != nil {
 		t.Log(reply.Err)

pkg/conf/parse_test.go

@@ -27,10 +27,10 @@ func Test_hideSensitiveFields(t *testing.T) {
 	keywords = append(keywords, `"dsn"`, `"password"`, `"name"`)
 	str := Show(c, keywords...)
 
-	fmt.Printf(hideSensitiveFields(str))
+	fmt.Print(hideSensitiveFields(str))
 
 	str = "\ndefault:123456@192.168.3.37:6379/0\n"
-	fmt.Printf(hideSensitiveFields(str))
+	fmt.Print(hideSensitiveFields(str))
 }
 
 // test listening for configuration file updates

pkg/gobash/gobash_test.go

@@ -18,7 +18,7 @@ func TestRun(t *testing.T) {
 		ctx, _ := context.WithTimeout(context.Background(), time.Second)
 		result := Run(ctx, cmd, args...)
 		for v := range result.StdOut { // Real-time output of logs and error messages
-			t.Logf(v)
+			t.Log(v)
 		}
 		if result.Err != nil {
 			t.Logf("execute command failed, %v", result.Err)

pkg/sgorm/mysql/mysql_test.go

@@ -1,7 +1,6 @@
 package mysql
 
 import (
-	"fmt"
 	"testing"
 	"time"
 
@@ -13,7 +12,7 @@ func TestInitMysql(t *testing.T) {
 	db, err := Init(dsn, WithEnableTrace())
 	if err != nil {
 		// ignore test error about not being able to connect to real mysql
-		t.Logf(fmt.Sprintf("connect to mysql failed, err=%v, dsn=%s", err, dsn))
+		t.Logf("connect to mysql failed, err=%v, dsn=%s", err, dsn)
 		return
 	}
 	defer Close(db)
@@ -26,7 +25,7 @@ func TestInitTidb(t *testing.T) {
 	db, err := InitTidb(dsn)
 	if err != nil {
 		// ignore test error about not being able to connect to real tidb
-		t.Logf(fmt.Sprintf("connect to mysql failed, err=%v, dsn=%s", err, dsn))
+		t.Logf("connect to mysql failed, err=%v, dsn=%s", err, dsn)
 		return
 	}
 	defer Close(db)

pkg/sgorm/postgresql/postgresql_test.go

@@ -1,7 +1,6 @@
 package postgresql
 
 import (
-	"fmt"
 	"testing"
 	"time"
 
@@ -13,7 +12,7 @@ func TestInit(t *testing.T) {
 	db, err := Init(dsn, WithEnableTrace())
 	if err != nil {
 		// ignore test error about not being able to connect to real postgresql
-		t.Logf(fmt.Sprintf("connect to postgresql failed, err=%v, dsn=%s", err, dsn))
+		t.Logf("connect to postgresql failed, err=%v, dsn=%s", err, dsn)
 		return
 	}
 	defer Close(db)

pkg/sgorm/sqlite/sqlite_test.go

@@ -1,7 +1,6 @@
 package sqlite
 
 import (
-	"fmt"
 	"testing"
 	"time"
 
@@ -13,7 +12,7 @@ func TestInit(t *testing.T) {
 	db, err := Init(dbFile)
 	if err != nil {
 		// ignore test error about not being able to connect to real sqlite
-		t.Logf(fmt.Sprintf("connect to sqlite failed, err=%v, dbFile=%s", err, dbFile))
+		t.Logf("connect to sqlite failed, err=%v, dbFile=%s", err, dbFile)
 		return
 	}
 	defer Close(db)