refactor: refactor codebase to improve naming conventions and API usage

appleboy · appleboy · commit e81f83bc64a2 · 2025-01-18T12:14:01.000+08:00
- Update `masterName` and `slaveName` to use the `Port()` method for `masterPort` and `slavePort` respectively

Signed-off-by: appleboy &lt;appleboy.tw@gmail.com&gt;
diff --git a/redis_test.go b/redis_test.go
@@ -183,8 +183,8 @@ func TestRedisCluster(t *testing.T) {
 		Message: "foo",
 	}
 
-	masterName := fmt.Sprintf("%s:%s", hostIP, masterPort)
-	slaveName := fmt.Sprintf("%s:%s", hostIP, slavePort)
+	masterName := fmt.Sprintf("%s:%s", hostIP, masterPort.Port())
+	slaveName := fmt.Sprintf("%s:%s", hostIP, slavePort.Port())
 
 	hosts := []string{masterName, slaveName}
 

Original file line number	Diff line number	Diff line change
`@@ -183,8 +183,8 @@ func TestRedisCluster(t *testing.T) {`
`183`	`183`	`Message: "foo",`
`184`	`184`	`}`
`185`	`185`
`186`		`- masterName := fmt.Sprintf("%s:%s", hostIP, masterPort)`
`187`		`- slaveName := fmt.Sprintf("%s:%s", hostIP, slavePort)`
	`186`	`+ masterName := fmt.Sprintf("%s:%s", hostIP, masterPort.Port())`
	`187`	`+ slaveName := fmt.Sprintf("%s:%s", hostIP, slavePort.Port())`
`188`	`188`
`189`	`189`	`hosts := []string{masterName, slaveName}`
`190`	`190`