Adds new module API test to test the scripting engine module API

This commit adds a simple test to test the new module API that allows to
implement new scripting engines as modules.

Signed-off-by: Ricardo Dias <[email protected]>

src/modules/Makefile

@@ -23,7 +23,7 @@ endif
 all: helloworld.so hellotype.so helloblock.so hellocluster.so hellotimer.so hellodict.so hellohook.so helloacl.so helloscripting.so
 
 .c.xo:
-	$(CC) -I. $(CFLAGS) $(SHOBJ_CFLAGS) -fPIC -c $< -o $@
+	$(CC) -I.. -I. $(CFLAGS) $(SHOBJ_CFLAGS) -fPIC -c $< -o $@
 
 helloworld.xo: ../valkeymodule.h
 

src/modules/helloscripting.c

@@ -1,4 +1,4 @@
-#include "../valkeymodule.h"
+#include "valkeymodule.h"
 
 #include <string.h>
 #include <ctype.h>

tests/modules/Makefile

@@ -64,7 +64,8 @@ TEST_MODULES = \
     moduleauthtwo.so \
     rdbloadsave.so \
     crash.so \
-    cluster.so
+    cluster.so \
+    helloscripting.so
 
 .PHONY: all
 

tests/modules/helloscripting.c

@@ -0,0 +1 @@
+../../src/modules/helloscripting.c

tests/unit/moduleapi/scriptingengine.tcl

@@ -0,0 +1,30 @@
+set testmodule [file normalize tests/modules/helloscripting.so]
+
+set HELLO_PROGRAM "#!hello name=mylib\nFUNCTION foo\nARGS 0\nRETURN\nFUNCTION bar\nCONSTI 432\nRETURN"
+
+start_server {tags {"modules"} overrides {{save ""}}} {
+    r module load $testmodule
+
+    r function load $HELLO_PROGRAM
+
+    test {Call scripting engine function: calling foo works} {
+        set result [r fcall foo 0 134]
+        assert_equal $result 134
+    }
+
+    test {Call scripting engine function: calling bar works} {
+        set result [r fcall bar 0]
+        assert_equal $result 432
+    }
+
+    test {Replace function library and call functions} {
+        set result [r function load replace "#!hello name=mylib\nFUNCTION foo\nARGS 0\nRETURN\nFUNCTION bar\nCONSTI 500\nRETURN"]
+        assert_equal $result "mylib"
+
+        set result [r fcall foo 0 132]
+        assert_equal $result 132
+
+        set result [r fcall bar 0]
+        assert_equal $result 500
+    }
+}