[incubator-kvrocks] branch unstable updated: Increase blocking timeout to harden the blocking pop test case (#1128)

[tison]

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 5467b02b Increase blocking timeout to harden the blocking pop test 
case (#1128)
5467b02b is described below

commit 5467b02b5bf1f0d0e19382e9d410571b033f384f
Author: hulk <hulk.webs...@gmail.com>
AuthorDate: Sat Nov 26 17:02:46 2022 +0800

    Increase blocking timeout to harden the blocking pop test case (#1128)
    
    Currently, the blocking pop with the empty argument may fail if took a long 
time(more than 1s) to execute the RPUSH command, then the previous brpop 
command would get the empty bulk string(`$-1\r\n`) instead of the element array.
---
 tests/gocase/unit/type/list/list_test.go | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/gocase/unit/type/list/list_test.go 
b/tests/gocase/unit/type/list/list_test.go
index 72833f03..d8859d0b 100644
--- a/tests/gocase/unit/type/list/list_test.go
+++ b/tests/gocase/unit/type/list/list_test.go
@@ -450,7 +450,7 @@ func TestList(t *testing.T) {
                        rd := srv.NewTCPClient()
                        defer func() { require.NoError(t, rd.Close()) }()
                        require.NoError(t, rdb.Del(ctx, "blist1", 
"blist2").Err())
-                       require.NoError(t, rd.WriteArgs(popType, "blist1", 
"blist2", "1"))
+                       require.NoError(t, rd.WriteArgs(popType, "blist1", 
"blist2", "4"))
                        require.NoError(t, rdb.RPush(ctx, "blist1", 
"foo").Err())
                        rd.MustReadStrings(t, []string{"blist1", "foo"})
                        require.EqualValues(t, 0, rdb.Exists(ctx, 
"blist1").Val())