better handle flags

Signed-off-by: tison <wander4096@gmail.com>

tests/gocase/integration/cli/cli_test.go

@@ -23,7 +23,6 @@ import (
 	"bufio"
 	"bytes"
 	"context"
-	"flag"
 	"fmt"
 	"io"
 	"math"
@@ -39,16 +38,14 @@ import (
 
 const defaultByteBufLen = math.MaxUint16
 
-var cliPath = flag.String("cliPath", "redis-cli", "path to redis-cli")
-
 type interactiveCli struct {
 	c *exec.Cmd
 	r *bufio.Reader
 	w *bufio.Writer
 }
 
 func createInteractiveCli(t *testing.T, srv *util.KvrocksServer) *interactiveCli {
-	c := exec.Command(*cliPath)
+	c := exec.Command(util.CLIPath())
 	c.Args = append(c.Args, "-h", srv.Host(), "-p", fmt.Sprintf("%d", srv.Port()))
 	w, err := c.StdinPipe()
 	require.NoError(t, err)
@@ -120,7 +117,7 @@ func (res *result) Failed() {
 }
 
 func runCli(t *testing.T, srv *util.KvrocksServer, in io.Reader, args ...string) *result {
-	c := exec.Command(*cliPath)
+	c := exec.Command(util.CLIPath())
 	c.Stdin = in
 	c.Args = append(c.Args, "-h", srv.Host(), "-p", fmt.Sprintf("%d", srv.Port()))
 	c.Args = append(c.Args, args...)

tests/gocase/util/flags.go

@@ -0,0 +1,36 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one
+ * or more contributor license agreements.  See the NOTICE file
+ * distributed with this work for additional information
+ * regarding copyright ownership.  The ASF licenses this file
+ * to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance
+ * with the License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing,
+ * software distributed under the License is distributed on an
+ * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+ * KIND, either express or implied.  See the License for the
+ * specific language governing permissions and limitations
+ * under the License.
+ */
+
+package util
+
+import "flag"
+
+var binPath = flag.String("binPath", "", "directory including kvrocks build files")
+var workspace = flag.String("workspace", "", "directory of cases workspace")
+var deleteOnExit = flag.Bool("deleteOnExit", false, "whether to delete workspace on exit")
+var cliPath = flag.String("cliPath", "redis-cli", "path to redis-cli")
+var tlsEnable = flag.Bool("tlsEnable", true, "enable TLS-related test cases")
+
+func CLIPath() string {
+	return *cliPath
+}
+
+func TLSEnable() bool {
+	return *tlsEnable
+}

tests/gocase/util/server.go

@@ -22,7 +22,6 @@ package util
 import (
 	"context"
 	"crypto/tls"
-	"flag"
 	"fmt"
 	"net"
 	"os"
@@ -39,10 +38,6 @@ import (
 	"golang.org/x/exp/slices"
 )
 
-var binPath = flag.String("binPath", "", "directory including kvrocks build files")
-var workspace = flag.String("workspace", "", "directory of cases workspace")
-var deleteOnExit = flag.Bool("deleteOnExit", false, "whether to delete workspace on exit")
-
 type KvrocksServer struct {
 	t   testing.TB
 	cmd *exec.Cmd

tests/gocase/util/tls.go

@@ -22,17 +22,10 @@ package util
 import (
 	"crypto/tls"
 	"crypto/x509"
-	"flag"
 	"os"
 	"path/filepath"
 )
 
-var tlsEnable = flag.Bool("tlsEnable", true, "enable TLS-related test cases")
-
-func TLSEnable() bool {
-	return *tlsEnable
-}
-
 func DefaultTLSConfig() (*tls.Config, error) {
 	dir := filepath.Join(*workspace, "..", "tls", "cert")