Skip to content

Commit b48acb9

Browse files
Dumbrisclaude
andauthored
fix(test): increase socket E2E server startup timeout (#352)
The TestE2E_SocketPermissions test was flaky on macOS CI runners because 5 seconds wasn't enough for server startup. Other E2E tests already use 15 seconds. Align all socket E2E tests to the same timeout. Co-authored-by: Claude Code <noreply@anthropic.com>
1 parent a0cdc37 commit b48acb9

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

internal/server/socket_e2e_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ func TestE2E_TrayToCore_UnixSocket(t *testing.T) {
8888
// Wait for server to be ready
8989
require.Eventually(t, func() bool {
9090
return srv.IsReady()
91-
}, 5*time.Second, 100*time.Millisecond, "Server should become ready")
91+
}, 15*time.Second, 100*time.Millisecond, "Server should become ready")
9292

9393
// Wait for TCP address to be resolved (may take a moment with race detector)
9494
var tcpAddr string
@@ -271,7 +271,7 @@ func TestE2E_DualListener_Concurrent(t *testing.T) {
271271

272272
require.Eventually(t, func() bool {
273273
return srv.IsReady()
274-
}, 5*time.Second, 100*time.Millisecond)
274+
}, 15*time.Second, 100*time.Millisecond)
275275

276276
tcpAddr := srv.GetListenAddress()
277277
socketPath := filepath.Join(tmpDir, "mcpproxy.sock")
@@ -399,7 +399,7 @@ func TestE2E_SocketPermissions(t *testing.T) {
399399

400400
require.Eventually(t, func() bool {
401401
return srv.IsReady()
402-
}, 5*time.Second, 100*time.Millisecond)
402+
}, 15*time.Second, 100*time.Millisecond)
403403

404404
socketPath := filepath.Join(tmpDir, "mcpproxy.sock")
405405

0 commit comments

Comments
 (0)