chore: add LOG_XXX constants for lua redis table (#6036)

Signed-off-by: Kostas Kyrimis <[email protected]>

Author: kostasrim

src/core/interpreter.cc

@@ -708,6 +708,18 @@ Interpreter::Interpreter() {
   lua_pushcfunction(lua_, RedisLogCommand);
   lua_settable(lua_, -3);
 
+  lua_pushinteger(lua_, LL_DEBUG);
+  lua_setfield(lua_, -2, "LOG_DEBUG");
+
+  lua_pushinteger(lua_, LL_VERBOSE);
+  lua_setfield(lua_, -2, "LOG_VERBOSE");
+
+  lua_pushinteger(lua_, LL_NOTICE);
+  lua_setfield(lua_, -2, "LOG_NOTICE");
+
+  lua_pushinteger(lua_, LL_WARNING);
+  lua_setfield(lua_, -2, "LOG_WARNING");
+
   /* Finally set the table as 'redis' global var. */
   lua_setglobal(lua_, "redis");
   CHECK(lua_checkstack(lua_, 64));

src/core/interpreter_test.cc

@@ -493,7 +493,7 @@ TEST_F(InterpreterTest, ReplicateCommands) {
 TEST_F(InterpreterTest, Log) {
   EXPECT_FALSE(Execute(R"(redis.log('nonsense', 'nonsense'))"));
   EXPECT_THAT(error_, testing::HasSubstr("First argument must be a number (log level)."));
-  EXPECT_TRUE(Execute(R"(redis.log(0, 'warn'))"));
+  EXPECT_TRUE(Execute(R"(redis.log(redis.LOG_WARNING, 'warn'))"));
   EXPECT_EQ("nil", ser_.res);
   EXPECT_FALSE(Execute(R"(redis.log(4))"));
   EXPECT_THAT(error_, testing::HasSubstr("requires two arguments or more"));