To: vim_dev@googlegroups.com
Subject: Patch 9.0.0854
Fcc: outbox
From: Bram Moolenaar <Bram@moolenaar.net>
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
------------

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