From c815b9f1b1760750f91a5778d61e4baa689e4a48 Mon Sep 17 00:00:00 2001 From: Snider Date: Sat, 21 Mar 2026 17:48:48 +0000 Subject: [PATCH] fix(test): use port 0 instead of hardcoded 9101 in TCP warning test Port 9101 conflicts with running core-agent serve. OS-assigned port avoids the conflict and still tests the 0.0.0.0 warning. Co-Authored-By: Virgil --- pkg/mcp/transport_tcp_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/pkg/mcp/transport_tcp_test.go b/pkg/mcp/transport_tcp_test.go index 2afeda1..109a617 100644 --- a/pkg/mcp/transport_tcp_test.go +++ b/pkg/mcp/transport_tcp_test.go @@ -37,8 +37,8 @@ func TestNewTCPTransport_Warning(t *testing.T) { old := setDiagWriter(&buf) defer setDiagWriter(old) - // Trigger warning - tr, err := NewTCPTransport("0.0.0.0:9101") + // Trigger warning — use port 0 (OS assigns free port) + tr, err := NewTCPTransport("0.0.0.0:0") if err != nil { t.Fatalf("Failed to create transport: %v", err) }