more test coverage

xmidt-org · Feb 21, 2023 · 6cd21e8 · 6cd21e8
1 parent b287856
commit 6cd21e8
Showing 1 changed file with 9 additions and 9 deletions.
diff --git a/kratos_test.go b/kratos_test.go
@@ -73,7 +73,7 @@ var (
 			fmt.Println("hi")
 			return nil
 		},
-		ClientLogger: sallust.Default(),
+		ClientLogger: nil,
 		PingConfig:   PingConfig{},
 	}
 
@@ -220,15 +220,15 @@ func TestNew_NoPingMissHandler(t *testing.T) {
 	assert.Equal(err, errNilHandlePingMiss)
 }
 
-// func TestNew_ClientLoggerNotNil(t *testing.T) {
-// 	logger := sallust.Default()
-// 	clientConfigHasLogger := clientConfig
-// 	clientConfigHasLogger.ClientLogger = logger
-// 	assert := assert.New(t)
-// 	_, err := NewClient(clientConfigHasLogger)
+func TestNew_ClientLoggerNotNil(t *testing.T) {
+	logger := sallust.Default()
+	clientConfigHasLogger := clientConfig
+	clientConfigHasLogger.ClientLogger = logger
+	assert := assert.New(t)
+	_, err := NewClient(clientConfigHasLogger)
 
-// 	assert.Nil(err)
-// }
+	assert.Nil(err)
+}
 
 func Error(s string) {
 	panic("unimplemented")