suica commented on code in PR #835:
URL: https://github.com/apache/incubator-kvrocks/pull/835#discussion_r967811356


##########
tests/gocase/unit/type/list/list_test.go:
##########
@@ -84,3 +84,154 @@ func BenchmarkLTRIM(b *testing.B) {
                })
        }
 }
+
+func TestZipList(t *testing.T) {
+       srv := util.StartServer(t, map[string]string{
+               "list-max-ziplist-size": "16",
+       })
+       defer srv.Close()
+       ctx := context.Background()
+       rdb := srv.NewClient()
+       defer func() { require.NoError(t, rdb.Close()) }()
+
+       rand.Seed(0)

Review Comment:
   The main reason to have seed set here is to make the testing failure 
reproducible (although this is to predetermine the test input). 
   Otherwise, we need other machanisms to ensure failed tests can be 
reproduced, such as dumping all history commands and replay them...



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to