Merge pull request #301 from sandyharvie/fix-type-conversion-bug-in-r…

…edisToLua fix type conversion bug in redisToLua
alicebob · Oct 31, 2022 · 1bcf722 · 1bcf722
2 parents 1d7ae5f + eb795a4
commit 1bcf722
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/cmd_scripting_test.go b/cmd_scripting_test.go
@@ -421,6 +421,16 @@ func TestCmdEvalReply(t *testing.T) {
 		),
 	)
 
+	mustOK(t, c,
+		"EVAL", `return redis.call("XGROUP", "CREATE", KEYS[1], ARGV[1], "$", "MKSTREAM")`,
+		"1", "stream", "group",
+	)
+	mustDo(t, c,
+		"EVAL", `return redis.call("XPENDING", KEYS[1], ARGV[1], "-", "+", 1, ARGV[2])`,
+		"1", "stream", "group", "consumer",
+		proto.Array(),
+	)
+
 	mustDo(t, c,
 		"EVAL", `return {err="broken"}`, "0",
 		proto.Error("broken"),

diff --git a/lua.go b/lua.go
@@ -217,6 +217,8 @@ func redisToLua(l *lua.LState, res []interface{}) *lua.LTable {
 			v = lua.LFalse
 		} else {
 			switch et := e.(type) {
+			case int:
+				v = lua.LNumber(et)
 			case int64:
 				v = lua.LNumber(et)
 			case []uint8: