Merge branch 'philippe/pyTypeFactory-cleanup' into philippe/full-js-and-python-stacks

Author: philippedistributive

python/pythonmonkey/cli/pmjs.py

@@ -146,7 +146,7 @@
 }
 """, evalOpts);
 
-def repl():
+async def repl():
     """
     Start a REPL to evaluate JavaScript code in the extra-module environment. Multi-line statements and
     readline history are supported. ^C support is sketchy. Exit the REPL with ^D or ".quit".
@@ -178,7 +178,7 @@ def quit():
         """
         Quit the REPL. Repl saved by atexit handler.
         """
-        sys.exit(0)
+        globalThis.python.exit(); # need for python.exit.code in require.py
 
     def sigint_handler(signum, frame):
         """
@@ -235,6 +235,7 @@ def sigint_handler(signum, frame):
     #
     while got_sigint < 2:
         try:
+            await asyncio.sleep(0)
             inner_loop = False
             if (statement == ""):
                 statement = input('> ')[readline_skip_chars:]
@@ -259,6 +260,7 @@ def sigint_handler(signum, frame):
                 # SIGINT is received, so we have to patch things up so that the next-entered line is
                 # treated as the input at the top of the loop.
                 while (got_sigint == 0):
+                    await asyncio.sleep(0)
                     inner_loop = True
                     lineBuffer = input('... ')
                     more = lineBuffer[readline_skip_chars:]
@@ -360,8 +362,13 @@ async def runJS():
             await pm.wait() # blocks until all asynchronous calls finish
         asyncio.run(runJS())
     elif (enterRepl or forceRepl):
-        globalInitModule.initReplLibs()
-        repl()
+        async def runREPL():
+            globalInitModule.initReplLibs()
+            await repl()
+            await pm.wait()
+        asyncio.run(runREPL())
+
+    globalThis.python.exit(); # need for python.exit.code in require.py
 
 if __name__ == "__main__":
     main()

python/pythonmonkey/require.py

@@ -71,8 +71,14 @@
 
 globalThis.python.exit = pm.eval("""'use strict';
 (exit) => function pythonExitWrapper(exitCode) {
+  if (typeof exitCode === 'undefined')
+    exitCode = pythonExitWrapper.code;
+  if (typeof exitCode == 'undefined')
+    exitCode = 0n;
   if (typeof exitCode === 'number')
     exitCode = BigInt(Math.floor(exitCode));
+  if (typeof exitCode !== 'bigint')
+    exitCode = 1n;
   exit(exitCode);
 }
 """, evalOpts)(sys.exit);

tests/js/set-timeout.simple

@@ -0,0 +1,32 @@
+/**
+ * @file        set-timeout.simple
+ *              Simple test which ensures that setTimeout() fires, and that it fires at about the right
+ *              time.
+ * @author      Wes Garland, [email protected]
+ * @date        March 2024
+ */
+
+const time = python.eval('__import__("time").time');
+const start = time();
+
+/**
+ * - must fire later than 100ms
+ * - must fire before 10s
+ * - 10s is well before 100s but very CI-load-tolerant; the idea is not to check for accurancy, but
+ *  rather ensure we haven't mixed up seconds and milliseconds somewhere.
+ */
+function check100ms()
+{
+  end = time();
+
+  if (end - start < 0.1)
+    throw new Error('timer fired too soon');
+  if (end - start > 10)
+    throw new Error('timer fired too late');
+
+  console.log('done - timer fired after ', (end-start) * 1000, 'ms');
+  python.exit.code = 0;
+}
+
+python.exit.code = 2;
+setTimeout(check100ms, 100);