pbio/platform/sim_hub: Don't modify stdin on CI.

This caused a lot of headaches when the MicroPython test suite runs many instances at the same time.

Author: laurensvalk

lib/pbio/drv/bluetooth/bluetooth_simulation.c

@@ -129,6 +129,12 @@ pbio_error_t pbdrv_bluetooth_controller_initialize(pbio_os_state_t *state, pbio_
 #define STDIN_HEADER_SIZE (1)
 
 static void pbdrv_bluetooth_simulation_tick_handler() {
+    #ifdef PBDRV_CONFIG_RUN_ON_CI
+    // CI and MicroPython test suite have lots of problems with stdin. It is
+    // only needed for the REPL and interactive input, so don't bother on CI.
+    return;
+    #endif
+
     uint8_t buf[256 + STDIN_HEADER_SIZE];
 
     // This has been made non-blocking in platform.c.

lib/pbio/platform/sim_hub/platform.c

@@ -139,6 +139,9 @@ const pbdrv_motor_driver_virtual_simulation_platform_data_t
     },
 };
 
+// The 'embedded' main.
+extern void _main(void);
+
 int main(int argc, char **argv) {
 
     // Parse given program, else otherwise default to REPL.
@@ -165,6 +168,13 @@ int main(int argc, char **argv) {
         }
     }
 
+    #ifdef PBDRV_CONFIG_RUN_ON_CI
+    // On the CI modifying settings for stdin causes problems. The REPL isn't
+    // used on CI anyway.
+    _main();
+    return 0;
+    #endif
+
     // Save the original terminal settings
     struct termios term_old, term_new;
     if (tcgetattr(STDIN_FILENO, &term_old) != 0) {
@@ -197,8 +207,7 @@ int main(int argc, char **argv) {
         return 0;
     }
 
-    // Runs the 'embedded' main.
-    extern void _main(void);
+    // Simulate running embedded main.
     _main();
 
     // Restore stdin flags.

test-virtualhub.sh

@@ -18,7 +18,7 @@ PB_TEST_DIR=$"$SCRIPT_DIR/tests"
 BUILD_DIR="$BRICK_DIR/build${COVERAGE:+-coverage}"
 PBIO_DIR="$SCRIPT_DIR/lib/pbio"
 
-make -s -j $(nproc --all) -C "$BRICK_DIR"
+make -s -j $(nproc --all) -C "$BRICK_DIR" COPT=-DPBDRV_CONFIG_RUN_ON_CI
 
 export MICROPY_MICROPYTHON="$BUILD_DIR/firmware.elf"