fix: reenable tests and dev server leaving stray server on exit in electron

Author: abose

package.json

@@ -29,8 +29,8 @@
     "build": "npm run eslint  && npm run _copy-node-src-dist && npm run _build-nonopt && npm run _make-debug-dist && npm run _build-optimized && npm run _copy-dist-to-test",
     "serve": "http-server . -p 8080 -c-1",
     "_serve-tauri": "http-server . -p 8081 -c-1",
-    "test-electron": "npm run _serve-tauri & sleep 2 && ./src-electron/node_modules/.bin/electron src-electron/main.js",
-    "test-electron-ci": "npm run _serve-tauri & sleep 2 && ./src-electron/node_modules/.bin/electron src-electron/main.js --quit-when-done",
+    "test-electron": "./scripts/run-with-server.sh ./src-electron/node_modules/.bin/electron src-electron/main.js",
+    "test-electron-ci": "./scripts/run-with-server.sh ./src-electron/node_modules/.bin/electron src-electron/main.js --quit-when-done",
     "bumpPatchVersion": "npm --no-git-tag-version version patch",
     "bumpPatchVersionWithGitTag": "npm version patch",
     "release": "npm run build && npm run bumpPatchVersionWithGitTag"

scripts/run-with-server.sh

@@ -0,0 +1,34 @@
+#!/bin/bash
+# Starts http-server, runs command, ensures cleanup on exit
+
+PORT=8081
+SERVER_PID=""
+
+cleanup() {
+    if [ -n "$SERVER_PID" ] && kill -0 "$SERVER_PID" 2>/dev/null; then
+        echo "Stopping http-server (PID: $SERVER_PID)..."
+        kill "$SERVER_PID" 2>/dev/null
+        wait "$SERVER_PID" 2>/dev/null
+    fi
+    # Safety net: kill any orphaned process on the port
+    fuser -k "$PORT/tcp" 2>/dev/null || true
+}
+
+trap cleanup EXIT INT TERM
+
+# Kill any existing server on the port first
+fuser -k "$PORT/tcp" 2>/dev/null || true
+sleep 1
+
+# Start http-server in background
+http-server . -p "$PORT" -c-1 &
+SERVER_PID=$!
+echo "Started http-server on port $PORT (PID: $SERVER_PID)"
+
+sleep 2
+
+# Run the provided command
+"$@"
+EXIT_CODE=$?
+
+exit $EXIT_CODE

test/index.html

@@ -149,13 +149,13 @@
     </script>
 
     <script src="testInit.js"></script>
-<!--    <script src="test-node.browser.js"></script>-->
-<!--    <script src="test-dir.browser.js"></script>-->
-<!--    <script src="test-file.browser.js"></script>-->
-<!--    <script src="test-copy.browser.js"></script>-->
-<!--    <script src="test.worker.js"></script>-->
+    <script src="test-node.browser.js"></script>
+    <script src="test-dir.browser.js"></script>
+    <script src="test-file.browser.js"></script>
+    <script src="test-copy.browser.js"></script>
+    <script src="test.worker.js"></script>
     <script src="test-getPlatformPath-api.browser.js"></script>
-<!--    <script src="test-watcher.browser.js"></script>-->
+    <script src="test-watcher.browser.js"></script>
     <script class="mocha-exec">
         window.virtualTestPath = '/test-phoenix-fs';