You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kvrocks.apache.org by ti...@apache.org on 2022/09/21 02:44:51 UTC

[incubator-kvrocks] branch unstable updated: test: harden quit_test and list_test (#899)

This is an automated email from the ASF dual-hosted git repository.

tison pushed a commit to branch unstable
in repository https://gitbox.apache.org/repos/asf/incubator-kvrocks.git


The following commit(s) were added to refs/heads/unstable by this push:
     new 04f25c8  test: harden quit_test and list_test (#899)
04f25c8 is described below

commit 04f25c878f8ae80265f1071b674ded0d008e6715
Author: tison <wa...@gmail.com>
AuthorDate: Wed Sep 21 10:44:46 2022 +0800

    test: harden quit_test and list_test (#899)
    
    Signed-off-by: tison <wa...@gmail.com>
---
 tests/gocase/unit/quit/quit_test.go      | 2 +-
 tests/gocase/unit/type/list/list_test.go | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/tests/gocase/unit/quit/quit_test.go b/tests/gocase/unit/quit/quit_test.go
index c2cddf8..fa2aa82 100644
--- a/tests/gocase/unit/quit/quit_test.go
+++ b/tests/gocase/unit/quit/quit_test.go
@@ -45,7 +45,7 @@ func TestPipeQuit(t *testing.T) {
 
 		require.NoError(t, c.WriteArgs("PING"))
 		r, err = c.ReadLine()
-		require.EqualError(t, err, "EOF")
+		require.Error(t, err)
 		require.Empty(t, r)
 	})
 
diff --git a/tests/gocase/unit/type/list/list_test.go b/tests/gocase/unit/type/list/list_test.go
index 26285e3..579bfab 100644
--- a/tests/gocase/unit/type/list/list_test.go
+++ b/tests/gocase/unit/type/list/list_test.go
@@ -26,6 +26,7 @@ import (
 	"strconv"
 	"strings"
 	"testing"
+	"time"
 
 	"github.com/apache/incubator-kvrocks/tests/gocase/util"
 	"github.com/go-redis/redis/v9"
@@ -93,7 +94,7 @@ func TestZipList(t *testing.T) {
 	defer srv.Close()
 	ctx := context.Background()
 	rdb := srv.NewClientWithOption(&redis.Options{
-		MaxRetries: -1,
+		ReadTimeout: 10 * time.Second,
 	})
 	defer func() { require.NoError(t, rdb.Close()) }()