Fixed Runtime.js to not always remove last argument

vanesca-tests/fix1.js

@@ -0,0 +1,28 @@
+
+process.stdout.write("\u001b[2J\u001b[00H") // clear screen
+const { Sandbox, SandboxCluster } = require('../v8-sandbox')
+let sandbox = new Sandbox({ memory: 1000, require: __dirname + '/functions1.js' })
+
+let code = `
+    async function test() {
+
+        // always works! (water-fall/callback embedded)
+        addNumbers(1, 2, function(result) {
+            if(result.error) console.log('error1:', result.error)
+            if(result.value) console.log('value1:', result.value)
+        })
+
+        // only works with the runtime.js fix (standard await)
+        // otherwise the last argument is removed, because it expects a callback (embedded)
+        let {error, value} = await addNumbers(1, 2)
+        if(error) console.log('error2:', error)
+        if(value) console.log('value2:', value)
+    }
+    test()
+`; // always end with semicolon
+
+(async () => {
+
+    // test 1 (runs in async)
+    await sandbox.execute({ code })
+})()

vanesca-tests/fix2.js

@@ -0,0 +1,56 @@
+
+process.stdout.write("\u001b[2J\u001b[00H") // clear screen
+const { Sandbox, SandboxCluster } = require('../v8-sandbox')
+let sandbox = new Sandbox({ memory: 1000 })
+
+let code1 = `
+    let t = 1
+    console.log(t) // works
+`; // always end with semicolon
+
+let code2 = ` // repeated executes STACK and should keep their values
+    t++
+    console.log(t)
+`;
+
+(async () => {
+
+    const RUN = 4
+    console.log('Test no.' + RUN)
+
+    // test 1 (runs in sync) (works)
+    if(RUN === 1) {
+        sandbox.execute({ code: code1 })
+        sandbox.execute({ code: code2 })
+    }
+
+    //////////////////////////////////////////
+
+    // test 2 (runs first in async/second sync) (doesn't work, even though it should!)
+    if(RUN === 2) {
+        await sandbox.execute({ code: code1 })
+        sandbox.execute({ code: code2 })
+    }
+
+    //////////////////////////////////////////
+
+    // test 3 (run in async) (doesn't work, even though it should!)
+    if(RUN === 3) {
+        await sandbox.execute({ code: code1 })
+        await sandbox.execute({ code: code2 })
+    }
+
+    //////////////////////////////////////////
+
+    // test 4 (promise result also doesn't work, even though it should!)
+    if(RUN === 4) {
+        sandbox.execute({ code: code1 }).then(() =>
+            // doesn't work, even though it should!
+            // because of a flag being reset in sandbox.js code
+            // this is fixed now.
+            setTimeout(async() => {
+               sandbox.execute({ code: code2 }) // should display '2' (after fix)
+            }, 2000)
+        )
+    }
+})()

vanesca-tests/functions1.js

@@ -0,0 +1,10 @@
+
+// Create or require your own functions and libraries (more safe)
+
+// Async (water-fall method / callback)
+defineAsync('addNumbers', ([ value1, value2 ], { respond, callback }) => {
+    respond() // always required!
+    setTimeout(() => {
+        callback({error: null, value: value1 + value2}) // send to sandbox
+    }, 1000) // 1 sec delay (for testing)
+})