From 9ae43d49312c3bc4e494ead4dc89193e6570fe11 Mon Sep 17 00:00:00 2001 From: Prateek Date: Thu, 19 Feb 2026 03:45:17 +0530 Subject: [PATCH] fix: update onboarding test to use new default terminal port (14801) The onboarding integration test had port 3003 hardcoded for the WebSocket health check. Updated to read from DIRECT_TERMINAL_PORT env var with 14801 as the default, matching the new port defaults. Co-Authored-By: Claude Opus 4.6 --- tests/integration/onboarding-test.sh | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tests/integration/onboarding-test.sh b/tests/integration/onboarding-test.sh index bb44359a7..1516e8bc7 100755 --- a/tests/integration/onboarding-test.sh +++ b/tests/integration/onboarding-test.sh @@ -131,10 +131,12 @@ end_step "Step 7: Dashboard API responding" start_step "Step 8: Verify WebSocket servers" # Check if direct terminal WebSocket server is running (required for terminal feature) -echo " Checking WebSocket server on port 3003..." +# Default port is 14801 (14800 range chosen to avoid conflicts with dev tools) +DIRECT_TERMINAL_PORT="${DIRECT_TERMINAL_PORT:-14801}" +echo " Checking WebSocket server on port $DIRECT_TERMINAL_PORT..." max_retries=10 for i in $(seq 1 $max_retries); do - if curl -sf http://localhost:3003/health > /dev/null 2>&1; then + if curl -sf "http://localhost:$DIRECT_TERMINAL_PORT/health" > /dev/null 2>&1; then echo " ✓ WebSocket server responding" break fi