Patch 9.0.0989

4 views
Skip to first unread message

Bram Moolenaar

unread,
Dec 2, 2022, 10:06:46 AM12/2/22
to vim...@googlegroups.com

Patch 9.0.0989
Problem: Popupwin test is more flaky on MacOS.
Solution: Use a longer wait time.
Files: src/testdir/test_popupwin.vim


*** ../vim-9.0.0988/src/testdir/test_popupwin.vim 2022-11-27 12:45:37.311707363 +0000
--- src/testdir/test_popupwin.vim 2022-12-02 15:04:15.080540352 +0000
***************
*** 1197,1204 ****
call assert_equal(0, popup_locate(1, 21))
call assert_equal(0, popup_locate(2, 1))

! sleep 700m
redraw
let line = join(map(range(1, 5), '1->screenstring(v:val)'), '')
call assert_equal('hello', line)

--- 1197,1207 ----
call assert_equal(0, popup_locate(1, 21))
call assert_equal(0, popup_locate(2, 1))

! " Mac is usually a bit slow
! let delay = has('mac') ? '900m' : '700m'
! exe 'sleep ' .. delay
redraw
+
let line = join(map(range(1, 5), '1->screenstring(v:val)'), '')
call assert_equal('hello', line)

***************
*** 1212,1218 ****
let line = join(map(range(1, 30), 'screenstring(&lines, v:val)'), '')
call assert_match('.*on the command line.*', line)

! sleep 700m
redraw
let line = join(map(range(1, 30), 'screenstring(&lines, v:val)'), '')
call assert_notmatch('.*on the command line.*', line)
--- 1215,1221 ----
let line = join(map(range(1, 30), 'screenstring(&lines, v:val)'), '')
call assert_match('.*on the command line.*', line)

! exe 'sleep ' .. delay
redraw
let line = join(map(range(1, 30), 'screenstring(&lines, v:val)'), '')
call assert_notmatch('.*on the command line.*', line)
*** ../vim-9.0.0988/src/version.c 2022-12-02 13:37:32.886620865 +0000
--- src/version.c 2022-12-02 15:05:27.288692206 +0000
***************
*** 697,698 ****
--- 697,700 ----
{ /* Add new patch number below this line */
+ /**/
+ 989,
/**/

--
$ echo pizza > /dev/oven

/// Bram Moolenaar -- Br...@Moolenaar.net -- http://www.Moolenaar.net \\\
/// \\\
\\\ sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
Reply all
Reply to author
Forward
0 new messages