]> granicus.if.org Git - vim/commitdiff
patch 8.2.5158: TSTP and INT signal tests are not run with valgrind v8.2.5158
authorBram Moolenaar <Bram@vim.org>
Sat, 25 Jun 2022 11:13:28 +0000 (12:13 +0100)
committerBram Moolenaar <Bram@vim.org>
Sat, 25 Jun 2022 11:13:28 +0000 (12:13 +0100)
Problem:    TSTP and INT signal tests are not run with valgrind.
Solution:   Sleep a bit longer. (closes #10614)

src/testdir/test_signals.vim
src/version.c

index 2cf0dd772c9f29be7ccb6079560e9e123428d751..b7a8b2821748d1a7b06aa33af2dc89466983e183 100644 (file)
@@ -85,20 +85,15 @@ func Test_signal_INT()
     throw 'Skipped: INT signal not supported'
   endif
 
-  " Skip the test when running with valgrind as signal INT is not received
-  " somehow by Vim when running with valgrind.
-  let cmd = GetVimCommand()
-  if cmd =~ 'valgrind'
-    throw 'Skipped: cannot test signal INT with valgrind'
-  endif
-
   let buf = RunVimInTerminal('', {'rows': 6})
   let pid_vim = term_getjob(buf)->job_info().process
 
   " Check that an endless loop in Vim is interrupted by signal INT.
+  call term_sendkeys(buf, ":call setline(1, 'running')\n")
   call term_sendkeys(buf, ":while 1 | endwhile\n")
   call WaitForAssert({-> assert_equal(':while 1 | endwhile', term_getline(buf, 6))})
   exe 'silent !kill -s INT ' .. pid_vim
+  sleep 50m
   call term_sendkeys(buf, ":call setline(1, 'INTERRUPTED')\n")
   call WaitForAssert({-> assert_equal('INTERRUPTED', term_getline(buf, 1))})
 
@@ -112,13 +107,6 @@ func Test_signal_TSTP()
     throw 'Skipped: TSTP signal not supported'
   endif
 
-  " Skip the test when running with valgrind as signal TSTP is not received
-  " somehow by Vim when running with valgrind.
-  let cmd = GetVimCommand()
-  if cmd =~ 'valgrind'
-    throw 'Skipped: cannot test signal TSTP with valgrind'
-  endif
-
   " If test fails once, it can leave temporary files and trying to rerun
   " the test would then fail again if they are not deleted first.
   call delete('.Xsig_TERM.swp')
@@ -142,9 +130,10 @@ func Test_signal_TSTP()
   exe 'silent !kill -s TSTP ' .. pid_vim
   call WaitForAssert({-> assert_true(filereadable('.Xsig_TERM.swp'))})
 
-  " We resume after the suspend
+  " We resume after the suspend.  Sleep a bit for the signal to take effect,
+  " also when running under valgrind. 
   exe 'silent !kill -s CONT ' .. pid_vim
-  exe 'silent !sleep 0.006'
+  sleep 100m
 
   call StopVimInTerminal(buf)
 
index 3918d434561b5d017eaa5e799ab29e913e2d65e8..8638e9e37feab1b5f1a810210769dc0692a35b72 100644 (file)
@@ -735,6 +735,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    5158,
 /**/
     5157,
 /**/