Improve Wokwi workflow: better logging, error handling, and CommonJS compatibility

Co-authored-by: softhack007 <[email protected]>

Author: Copilot

.github/workflows/wokwi-test.yml

@@ -67,29 +67,44 @@ jobs:
       - name: Start Wokwi simulator in background
         run: |
           cd test/wokwi
-          # Start simulator in background
-          ./run-simulator.sh &
+          # Create log directory
+          mkdir -p logs
+          
+          # Start simulator in background with a 180 second timeout
+          WOKWI_TIMEOUT=180 ./run-simulator.sh > logs/wokwi.log 2>&1 &
           WOKWI_PID=$!
           echo "WOKWI_PID=$WOKWI_PID" >> $GITHUB_ENV
+          echo "Started Wokwi simulator with PID $WOKWI_PID"
           
           # Wait for simulator to start and web server to be ready
           echo "Waiting for WLED web server to be ready..."
-          timeout=60
+          max_wait=120
           elapsed=0
-          while [ $elapsed -lt $timeout ]; do
-            if curl -s http://localhost:8080 > /dev/null 2>&1; then
-              echo "Web server is ready!"
+          while [ $elapsed -lt $max_wait ]; do
+            if curl -s -f http://localhost:8080 > /dev/null 2>&1; then
+              echo "Web server is ready after $elapsed seconds!"
               break
             fi
-            echo "Waiting... ($elapsed seconds)"
-            sleep 2
-            elapsed=$((elapsed + 2))
+            if ! kill -0 $WOKWI_PID 2>/dev/null; then
+              echo "Error: Wokwi simulator process died"
+              echo "Last 50 lines of Wokwi log:"
+              tail -50 logs/wokwi.log || true
+              exit 1
+            fi
+            echo "Still waiting... ($elapsed seconds)"
+            sleep 5
+            elapsed=$((elapsed + 5))
           done
           
-          if [ $elapsed -ge $timeout ]; then
-            echo "Error: Web server did not start within $timeout seconds"
+          if [ $elapsed -ge $max_wait ]; then
+            echo "Error: Web server did not start within $max_wait seconds"
+            echo "Last 50 lines of Wokwi log:"
+            tail -50 logs/wokwi.log || true
+            kill $WOKWI_PID || true
             exit 1
           fi
+          
+          echo "WLED is ready for testing!"
 
       - name: Run Playwright tests
         run: npm run test:wokwi
@@ -117,6 +132,7 @@ jobs:
         with:
           name: wokwi-test-results
           path: |
+            test/wokwi/logs/
             test-results/
             playwright-report/
           retention-days: 7

.gitignore

@@ -38,4 +38,5 @@ _codeql_detected_source_root
 /test/wokwi/firmware.bin
 /test/wokwi/firmware.elf
 /test/wokwi/.wokwi/
+/test/wokwi/logs/
 

playwright.config.js

@@ -1,10 +1,10 @@
-import { defineConfig, devices } from '@playwright/test';
+const { defineConfig, devices } = require('@playwright/test');
 
 /**
  * Playwright configuration for WLED-MM Wokwi testing
  * See https://playwright.dev/docs/test-configuration.
  */
-export default defineConfig({
+module.exports = defineConfig({
   testDir: './test/playwright',
 
   /* Run tests in files in parallel */

test/playwright/wokwi-basic.spec.js

@@ -1,4 +1,4 @@
-import { test, expect } from '@playwright/test';
+const { test, expect } = require('@playwright/test');
 
 /**
  * Basic WLED-MM Web Interface Tests

test/wokwi/run-simulator.sh

@@ -18,7 +18,9 @@ fi
 
 echo "Starting Wokwi simulator..."
 echo "Timeout: ${WOKWI_TIMEOUT} seconds"
+echo "Web server will be available at http://localhost:8080"
 
-# Run wokwi-cli with timeout
+# Run wokwi-cli with timeout (in milliseconds)
 # The simulator will forward port 80 to localhost:8080
+# Note: wokwi-cli runs in foreground, so this needs to be backgrounded or run in a separate process
 wokwi-cli --timeout ${WOKWI_TIMEOUT}000 diagram.json