# HG changeset patch # User Bram Moolenaar # Date 1655829908 -7200 # Node ID 0b47ef4e19b8e7f4aa35c1755172ba50f57fc4db # Parent 6a718a4e17ce8fbc1412715789c04b99e06458b2 patch 8.2.5143: some tests fail when using valgrind Commit: https://github.com/vim/vim/commit/37bb3b111d08ce420e3cfd6ea3da3ed7428fbed1 Author: Bram Moolenaar Date: Tue Jun 21 17:40:47 2022 +0100 patch 8.2.5143: some tests fail when using valgrind Problem: Some tests fail when using valgrind. Spurious leak reports. Solution: Use WaitForAssert(). Avoid failing fork/exec. Skip tests where a job is killed when running valgrind. diff --git a/src/testdir/test_channel.vim b/src/testdir/test_channel.vim --- a/src/testdir/test_channel.vim +++ b/src/testdir/test_channel.vim @@ -1201,19 +1201,22 @@ func Test_pipe_null() call job_stop(job) endtry - let job = job_start(s:python . " test_channel_pipe.py something", - \ {'out_io': 'null', 'err_io': 'out'}) - call assert_equal("run", job_status(job)) - call job_stop(job) + " This causes spurious leak errors with valgrind. + if !RunningWithValgrind() + let job = job_start(s:python . " test_channel_pipe.py something", + \ {'out_io': 'null', 'err_io': 'out'}) + call assert_equal("run", job_status(job)) + call job_stop(job) - let job = job_start(s:python . " test_channel_pipe.py something", - \ {'in_io': 'null', 'out_io': 'null', 'err_io': 'null'}) - call assert_equal("run", job_status(job)) - call assert_equal('channel fail', string(job_getchannel(job))) - call assert_equal('fail', ch_status(job)) - call assert_equal('no process', string(test_null_job())) - call assert_equal('channel fail', string(test_null_channel())) - call job_stop(job) + let job = job_start(s:python . " test_channel_pipe.py something", + \ {'in_io': 'null', 'out_io': 'null', 'err_io': 'null'}) + call assert_equal("run", job_status(job)) + call assert_equal('channel fail', string(job_getchannel(job))) + call assert_equal('fail', ch_status(job)) + call assert_equal('no process', string(test_null_job())) + call assert_equal('channel fail', string(test_null_channel())) + call job_stop(job) + endif endfunc func Test_pipe_to_buffer_raw() @@ -1760,19 +1763,21 @@ func Test_job_start_fails() call assert_fails("call job_start('ls', \ {'err_io' : 'buffer', 'err_buf' : -1})", 'E475:') + let cmd = has('win32') ? "cmd /c dir" : "ls" + set nomodifiable - call assert_fails("call job_start('cmd /c dir', + call assert_fails("call job_start(cmd, \ {'out_io' : 'buffer', 'out_buf' :" .. bufnr() .. "})", 'E21:') - call assert_fails("call job_start('cmd /c dir', + call assert_fails("call job_start(cmd, \ {'err_io' : 'buffer', 'err_buf' :" .. bufnr() .. "})", 'E21:') set modifiable - call assert_fails("call job_start('ls', {'in_io' : 'buffer'})", 'E915:') + call assert_fails("call job_start(cmd, {'in_io' : 'buffer'})", 'E915:') edit! XXX let bnum = bufnr() enew - call assert_fails("call job_start('ls', + call assert_fails("call job_start(cmd, \ {'in_io' : 'buffer', 'in_buf' : bnum})", 'E918:') " Empty job tests @@ -1787,6 +1792,11 @@ func Test_job_start_fails() endfunc func Test_job_stop_immediately() + " With valgrind this causes spurious leak reports + if RunningWithValgrind() + return + endif + let g:job = job_start([s:python, '-c', 'import time;time.sleep(10)']) try eval g:job->job_stop() diff --git a/src/testdir/test_cscope.vim b/src/testdir/test_cscope.vim --- a/src/testdir/test_cscope.vim +++ b/src/testdir/test_cscope.vim @@ -244,10 +244,13 @@ func Test_cscopeWithCscopeConnections() call assert_equal('', a) " Test: 'csprg' option - call assert_equal('cscope', &csprg) - set csprg=doesnotexist - call assert_fails('cscope add Xcscope2.out', 'E262:') - set csprg=cscope + " Skip this with valgrind, it causes spurious leak reports + if !RunningWithValgrind() + call assert_equal('cscope', &csprg) + set csprg=doesnotexist + call assert_fails('cscope add Xcscope2.out', 'E262:') + set csprg=cscope + endif " Test: multiple cscope connections cscope add Xcscope.out diff --git a/src/testdir/test_iminsert.vim b/src/testdir/test_iminsert.vim --- a/src/testdir/test_iminsert.vim +++ b/src/testdir/test_iminsert.vim @@ -81,11 +81,9 @@ func Test_imactivatefunc_imstatusfunc_ca END call writefile(lines, 'Xscript') let buf = RunVimInTerminal('-S Xscript', {}) - call term_wait(buf) call assert_notequal('abc', term_getline(buf, 2)) call term_sendkeys(buf, "/abc\n") - call term_wait(buf) - call assert_equal('abc', term_getline(buf, 2)) + call WaitForAssert({-> assert_equal('abc', term_getline(buf, 2))}) " clean up call StopVimInTerminal(buf) diff --git a/src/testdir/test_popup.vim b/src/testdir/test_popup.vim --- a/src/testdir/test_popup.vim +++ b/src/testdir/test_popup.vim @@ -1177,13 +1177,12 @@ func Test_pum_rightleft() END call writefile(lines, 'Xtest1') let buf = RunVimInTerminal('--cmd "set rightleft" Xtest1', {}) - call term_wait(buf) call term_sendkeys(buf, "Go\") call VerifyScreenDump(buf, 'Test_pum_rightleft_01', {'rows': 8}) call term_sendkeys(buf, "\\") - call term_wait(buf) + call TermWait(buf, 30) redraw! - call assert_match('\s*miv', Screenline(5)) + call WaitForAssert({-> assert_match('\s*miv', Screenline(5))}) " Test for expanding tabs to spaces in the popup menu let lines =<< trim END @@ -1193,15 +1192,15 @@ func Test_pum_rightleft() END call writefile(lines, 'Xtest2') call term_sendkeys(buf, "\:e! Xtest2\") - call term_wait(buf) + call TermWait(buf, 30) call term_sendkeys(buf, "Goone\\") - call term_wait(buf) + call TermWait(buf, 30) redraw! call VerifyScreenDump(buf, 'Test_pum_rightleft_02', {'rows': 7}) call term_sendkeys(buf, "\") - call term_wait(buf) + call TermWait(buf, 30) redraw! - call assert_match('\s*eerht eno', Screenline(4)) + call WaitForAssert({-> assert_match('\s*eerht eno', Screenline(4))}) call StopVimInTerminal(buf) call delete('Xtest1') @@ -1218,15 +1217,15 @@ func Test_pum_scrollbar() END call writefile(lines, 'Xtest1') let buf = RunVimInTerminal('--cmd "set pumheight=2" Xtest1', {}) - call term_wait(buf) + call TermWait(buf) call term_sendkeys(buf, "Go\\\") call VerifyScreenDump(buf, 'Test_pum_scrollbar_01', {'rows': 7}) call term_sendkeys(buf, "\\dd") - call term_wait(buf) + call TermWait(buf) if has('rightleft') call term_sendkeys(buf, ":set rightleft\") - call term_wait(buf) + call TermWait(buf) call term_sendkeys(buf, "Go\\\") call VerifyScreenDump(buf, 'Test_pum_scrollbar_02', {'rows': 7}) endif diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -735,6 +735,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 5143, +/**/ 5142, /**/ 5141,