Merge pull request #368 from TingluoHuang/master

Ignore 'redis.set_repl()' call

Author: alicebob

cmd_scripting_test.go

@@ -495,9 +495,17 @@ func TestCmdEvalAuth(t *testing.T) {
 func TestLuaReplicate(t *testing.T) {
 	_, c := runWithClient(t)
 
-	mustNil(t, c,
-		"EVAL", "redis.replicate_commands()", "0",
-	)
+	t.Run("replicate_commands", func(t *testing.T) {
+		mustNil(t, c,
+			"EVAL", "redis.replicate_commands()", "0",
+		)
+	})
+
+	t.Run("set_repl", func(t *testing.T) {
+		mustNil(t, c,
+			"EVAL", "redis.set_repl(redis.REPL_NONE)", "0",
+		)
+	})
 }
 
 func TestLuaTX(t *testing.T) {

integration/script_test.go

@@ -416,6 +416,12 @@ func TestScriptReplicate(t *testing.T) {
 			"EVAL", `redis.replicate_commands();`, "0",
 		)
 	})
+
+	testRaw(t, func(c *client) {
+		c.Do(
+			"EVAL", `redis.set_repl(redis.REPL_NONE);`, "0",
+		)
+	})
 }
 
 func TestScriptTx(t *testing.T) {

lua.go

@@ -161,6 +161,15 @@ func mkLua(srv *server.Server, c *server.Peer, sha string) (map[string]lua.LGFun
 			// ignored
 			return 1
 		},
+		"set_repl": func(l *lua.LState) int {
+			top := l.GetTop()
+			if top != 1 {
+				l.Error(lua.LString("wrong number of arguments"), 1)
+				return 0
+			}
+			// ignored
+			return 1
+		},
 	}, luaRedisConstants
 }