Patch 9.0.0854

3 views
Skip to first unread message

Bram Moolenaar

unread,
Nov 10, 2022, 8:22:24 AM11/10/22
to vim...@googlegroups.com

Patch 9.0.0854
Problem: No proper test for what 9.0.0846 fixes.
Solution: Run test in a terminal so that the hit-enter prompt can show up.
(closes #11523)
Files: src/testdir/test_assert.vim


*** ../vim-9.0.0853/src/testdir/test_assert.vim 2022-11-09 00:44:25.315439173 +0000
--- src/testdir/test_assert.vim 2022-11-10 13:18:38.249739275 +0000
***************
*** 1,5 ****
--- 1,8 ----
" Test that the methods used for testing work.

+ source check.vim
+ source term_util.vim
+
func Test_assert_false()
call assert_equal(0, assert_false(0))
call assert_equal(0, assert_false(v:false))
***************
*** 338,347 ****
endtry
endfunc

func Test_assert_fails_in_timer()
! " should not cause a hit-enter prompt, which isn't actually checked here
! call timer_start(0, {-> assert_fails('call', 'E471:')})
! sleep 10m
endfunc

func Test_assert_beeps()
--- 341,361 ----
endtry
endfunc

+ " Test that assert_fails() in a timer does not cause a hit-enter prompt.
+ " Requires using a terminal, in regular tests the hit-enter prompt won't be
+ " triggered.
func Test_assert_fails_in_timer()
! CheckRunVimInTerminal
!
! let buf = RunVimInTerminal('', {'rows': 6})
! let cmd = ":call timer_start(0, {-> assert_fails('call', 'E471:')})"
! call term_sendkeys(buf, cmd)
! call WaitForAssert({-> assert_equal(cmd, term_getline(buf, 6))})
! call term_sendkeys(buf, "\<CR>")
! call TermWait(buf, 100)
! call assert_match('E471: Argument required', term_getline(buf, 6))
!
! call StopVimInTerminal(buf)
endfunc

func Test_assert_beeps()
*** ../vim-9.0.0853/src/version.c 2022-11-10 12:38:13.056131021 +0000
--- src/version.c 2022-11-10 13:11:58.462040484 +0000
***************
*** 697,698 ****
--- 697,700 ----
{ /* Add new patch number below this line */
+ /**/
+ 854,
/**/

--
Change is inevitable, except from a vending machine.

/// 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