Commit: patch 9.2.0085: tests: test_clientserver.vim is flaky

1 view
Skip to first unread message

Christian Brabandt

unread,
Mar 1, 2026, 12:31:51 PM (3 days ago) Mar 1
to vim...@googlegroups.com
patch 9.2.0085: tests: test_clientserver.vim is flaky

Commit: https://github.com/vim/vim/commit/a5af4352cf60499d53cedc2435c123a448f3ab65
Author: Christian Brabandt <c...@256bit.org>
Date: Sun Mar 1 17:18:09 2026 +0000

patch 9.2.0085: tests: test_clientserver.vim is flaky

Problem: tests: test_client_server_stopinsert() is flaky.
Solution: Use remote_send() instead of remote_expr().

closes: #19539

Signed-off-by: Christian Brabandt <c...@256bit.org>

diff --git a/src/testdir/test_clientserver.vim b/src/testdir/test_clientserver.vim
index 5f2e8870a..8aa9428f2 100644
--- a/src/testdir/test_clientserver.vim
+++ b/src/testdir/test_clientserver.vim
@@ -228,12 +228,13 @@ func Test_client_server_stopinsert()
" When using valgrind it takes much longer.
call WaitForAssert({-> assert_match(name, serverlist())})

- call remote_expr(name, 'execute("stopinsert")')
+ call remote_send(name, "\<C-\>\<C-N>")

+ " Wait for the mode to change to Normal ('n')
call WaitForAssert({-> assert_equal('n', name->remote_expr("mode(1)"))})
- cal WaitForAssert({-> assert_equal('13', name->remote_expr("col('.')"))})
+ call WaitForAssert({-> assert_equal('13', name->remote_expr("col('.')"))})

- eval name->remote_send(":qa!\<CR>")
+ call remote_send(name, "\<C-\>\<C-N>:qa!\<CR>")
try
call WaitForAssert({-> assert_equal("dead", job_status(job))})
finally
diff --git a/src/version.c b/src/version.c
index 1aeec3e40..d001aa53f 100644
--- a/src/version.c
+++ b/src/version.c
@@ -734,6 +734,8 @@ static char *(features[]) =

static int included_patches[] =
{ /* Add new patch number below this line */
+/**/
+ 85,
/**/
84,
/**/
Reply all
Reply to author
Forward
0 new messages