From 03dfde2b5f81f06639ec8697cdd32ad50d8509fd Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Sun, 14 Feb 2021 13:17:22 +0100 Subject: [PATCH] patch 8.2.2512: Vim9: compiling error test sometimes fails MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Problem: Vim9: compiling error test sometimes fails. Solution: use WaitForAssert() instead of sleeping for a bit. (Dominique Pellé, closes #7837) --- src/testdir/term_util.vim | 4 ++++ src/testdir/test_vim9_func.vim | 14 ++------------ src/testdir/test_vim9_script.vim | 2 +- src/version.c | 2 ++ 4 files changed, 9 insertions(+), 13 deletions(-) diff --git a/src/testdir/term_util.vim b/src/testdir/term_util.vim index cfc850ba0..8541ef353 100644 --- a/src/testdir/term_util.vim +++ b/src/testdir/term_util.vim @@ -179,5 +179,9 @@ func Run_shell_in_terminal(options) return buf endfunc +" Return concatenated lines in terminal. +func Term_getlines(buf, lines) + return join(map(a:lines, 'term_getline(a:buf, v:val)'), '') +endfunc " vim: shiftwidth=2 sts=2 expandtab diff --git a/src/testdir/test_vim9_func.vim b/src/testdir/test_vim9_func.vim index 4bcd974e5..c24d32f93 100644 --- a/src/testdir/test_vim9_func.vim +++ b/src/testdir/test_vim9_func.vim @@ -31,18 +31,8 @@ def TestCompilingError() call writefile(lines, 'XTest_compile_error') var buf = RunVimInTerminal('-S XTest_compile_error', {rows: 10, wait_for_ruler: 0}) - var text = '' - for loop in range(100) - text = '' - for i in range(1, 9) - text ..= term_getline(buf, i) - endfor - if text =~ 'Variable not found: nothing' - break - endif - sleep 20m - endfor - assert_match('Error detected while compiling command line.*Fails.*Variable not found: nothing', text) + call WaitForAssert(() => assert_match('Error detected while compiling command line.*Fails.*Variable not found: nothing', + Term_getlines(buf, range(1, 9)))) # clean up call StopVimInTerminal(buf) diff --git a/src/testdir/test_vim9_script.vim b/src/testdir/test_vim9_script.vim index 85eea13fd..996c357c5 100644 --- a/src/testdir/test_vim9_script.vim +++ b/src/testdir/test_vim9_script.vim @@ -755,7 +755,7 @@ def Test_throw_vimscript() enddef def Test_error_in_nested_function() - # an error in a nested :function aborts executin in the calling :def function + # an error in a nested :function aborts executing in the calling :def function var lines =<< trim END vim9script def Func() diff --git a/src/version.c b/src/version.c index c91182234..dabc2af70 100644 --- a/src/version.c +++ b/src/version.c @@ -750,6 +750,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 2512, /**/ 2511, /**/ -- 2.40.0