changeset 16415:a0e98821a2ed v8.1.1212

patch 8.1.1212: signal PWR is not tested commit https://github.com/vim/vim/commit/520e245237fa92f9022968a52eb418da6eade9b4 Author: Bram Moolenaar <Bram@vim.org> Date: Sat Apr 27 17:32:40 2019 +0200 patch 8.1.1212: signal PWR is not tested Problem: Signal PWR is not tested. Solution: Test that PWR updates the swap file. (Dominique Pelle, closes #4312)
author Bram Moolenaar <Bram@vim.org>
date Sat, 27 Apr 2019 17:45:05 +0200
parents c2472b6835e6
children 5f6b2c27ca41
files src/testdir/test_signals.vim src/version.c
diffstat 2 files changed, 41 insertions(+), 11 deletions(-) [+]
line wrap: on
line diff
--- a/src/testdir/test_signals.vim
+++ b/src/testdir/test_signals.vim
@@ -4,18 +4,17 @@ if !has('unix')
   finish
 endif
 
-if has('gui_running')
-  " Signals only work for terminals, and won't work for GUI.
-  finish
-endif
+source shared.vim
 
-source shared.vim
+" Check whether a signal is available on this system.
+func HasSignal(signal)
+  let signals = system('kill -l')
+  return signals =~# '\<' .. a:signal .. '\>'
+endfunc
 
 " Test signal WINCH (window resize signal)
 func Test_signal_WINCH()
-  let signals = system('kill -l')
-  if signals !~ '\<WINCH\>'
-    " signal WINCH is not available, skip the test.
+  if has('gui_running') || !HasSignal('WINCH')
     return
   endif
 
@@ -31,15 +30,15 @@ func Test_signal_WINCH()
   let new_lines = &lines - 2
   let new_columns = &columns - 2
 
-  exe 'set lines=' . new_lines
-  exe 'set columns=' . new_columns
+  exe 'set lines=' .. new_lines
+  exe 'set columns=' .. new_columns
   call assert_equal(new_lines, &lines)
   call assert_equal(new_columns, &columns)
 
   " Send signal and wait for signal to be processed.
   " 'lines' and 'columns' should have been restored
   " after handing signal WINCH.
-  exe 'silent !kill -s WINCH ' . getpid()
+  exe 'silent !kill -s WINCH ' .. getpid()
   call WaitForAssert({-> assert_equal(old_lines, &lines)})
   call assert_equal(old_columns, &columns)
 
@@ -47,3 +46,32 @@ func Test_signal_WINCH()
     let &t_WS = old_WS
   endif
 endfunc
+
+" Test signal PWR, which should update the swap file.
+func Test_signal_PWR()
+  if !HasSignal('PWR')
+    return
+  endif
+
+  " Set a very large 'updatetime' and 'updatecount', so that we can be sure
+  " that swap file is updated as a result of sending PWR signal, and not
+  " because of exceeding 'updatetime' or 'updatecount' when changing buffer.
+  set updatetime=100000 updatecount=100000
+  new Xtest_signal_PWR
+  let swap_name = swapname('%')
+  call setline(1, '123')
+  preserve
+  let swap_content = readfile(swap_name, 'b')
+
+  " Update the buffer and check that the swap file is not yet updated,
+  " since we set 'updatetime' and 'updatecount' to large values.
+  call setline(1, 'abc')
+  call assert_equal(swap_content, readfile(swap_name, 'b'))
+
+  " Sending PWR signal should update the swap file.
+  exe 'silent !kill -s PWR ' .. getpid()
+  call WaitForAssert({-> assert_notequal(swap_content, readfile(swap_name, 'b'))})
+
+  bwipe!
+  set updatetime& updatecount&
+endfunc
--- a/src/version.c
+++ b/src/version.c
@@ -768,6 +768,8 @@ static char *(features[]) =
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    1212,
+/**/
     1211,
 /**/
     1210,