From 052126b77ad83b1f26da46dad8c42efacffe7568 Mon Sep 17 00:00:00 2001 From: zeertzjq Date: Mon, 18 Aug 2025 15:24:22 +0800 Subject: [PATCH] test(tui_spec): avoid race between server exit and new client (#35376) Don't start a new client to check for server exit. Check for the server socket being removed instead. --- test/functional/terminal/tui_spec.lua | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/test/functional/terminal/tui_spec.lua b/test/functional/terminal/tui_spec.lua index a298d4739c..cc1ae1d517 100644 --- a/test/functional/terminal/tui_spec.lua +++ b/test/functional/terminal/tui_spec.lua @@ -448,19 +448,15 @@ describe('TUI :connect', function() tt.feed_data(':connect! ' .. server1 .. '\013') screen2:expect({ any = vim.pesc('This is server 1^.') }) + retry(nil, nil, function() + eq(nil, vim.uv.fs_stat(server2)) + end) + local server1_session = n.connect(server1) server1_session:request('nvim_command', 'qall!') + screen2:expect({ any = vim.pesc('[Process exited 0]') }) screen2:detach() - - local screen3 = tt.setup_child_nvim({ - '--remote-ui', - '--server', - server2, - }) - screen3:expect({ any = 'Remote ui failed to start: connection refused' }) - screen3:expect({ any = vim.pesc('[Process exited 1]') }) - screen3:detach() end) end)