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

Patch 8.2.3853
Problem:    Vim9: not enough tests.
Solution:   Run more existing tests for Vim9 script.
Files:      src/testdir/test_listdict.vim


*** ../vim-8.2.3852/src/testdir/test_listdict.vim	2021-12-19 15:17:16.217857772 +0000
--- src/testdir/test_listdict.vim	2021-12-19 17:25:46.236303442 +0000
***************
*** 1294,1322 ****
    call CheckLegacyAndVim9Failure(['echo function("min")[0]'], 'E695:')
    call CheckLegacyAndVim9Failure(['echo v:true[0]'], 'E909:')
    call CheckLegacyAndVim9Failure(['echo v:null[0]'], 'E909:')
! 
!   let d = {'k' : 10}
!   call assert_fails('echo d.', 'E15:')
!   call CheckDefAndScriptFailure2(['var d = {k: 10}', 'echo d.'], 'E1127', 'E15:')
! 
    call CheckLegacyAndVim9Failure(['VAR d = {"k": 10}', 'echo d[1 : 2]'], 'E719:')
  
    call assert_fails("let v = [4, 6][{-> 1}]", 'E729:')
    call CheckDefAndScriptFailure2(['var v = [4, 6][() => 1]'], 'E1012', 'E703:')
  
!   call assert_fails("let v = range(5)[2:[]]", 'E730:')
    call assert_fails("let v = range(5)[2:{-> 2}(]", ['E15:', 'E116:'])
!   call assert_fails("let v = range(5)[2:3", 'E111:')
!   call assert_fails("let l = insert([1,2,3], 4, 10)", 'E684:')
!   call assert_fails("let l = insert([1,2,3], 4, -10)", 'E684:')
!   call assert_fails("let l = insert([1,2,3], 4, [])", 'E745:')
!   let l = [1, 2, 3]
!   call assert_fails("let l[i] = 3", 'E121:')
!   call assert_fails("let l[1.1] = 4", 'E805:')
!   call assert_fails("let l[:i] = [4, 5]", 'E121:')
!   call assert_fails("let l[:3.2] = [4, 5]", 'E805:')
!   let t = test_unknown()
!   call assert_fails("echo t[0]", 'E685:')
  endfunc
  
  " Test for a null list
--- 1294,1320 ----
    call CheckLegacyAndVim9Failure(['echo function("min")[0]'], 'E695:')
    call CheckLegacyAndVim9Failure(['echo v:true[0]'], 'E909:')
    call CheckLegacyAndVim9Failure(['echo v:null[0]'], 'E909:')
!   call CheckLegacyAndVim9Failure(['VAR d = {"k": 10}', 'echo d.'], ['E15:', 'E1127:', 'E15:'])
    call CheckLegacyAndVim9Failure(['VAR d = {"k": 10}', 'echo d[1 : 2]'], 'E719:')
  
    call assert_fails("let v = [4, 6][{-> 1}]", 'E729:')
    call CheckDefAndScriptFailure2(['var v = [4, 6][() => 1]'], 'E1012', 'E703:')
  
!   call CheckLegacyAndVim9Failure(['VAR v = range(5)[2 : []]'], ['E730:', 'E1012:', 'E730:'])
! 
    call assert_fails("let v = range(5)[2:{-> 2}(]", ['E15:', 'E116:'])
!   call CheckDefAndScriptFailure(['var v = range(5)[2 : () => 2(]'], 'E15:')
! 
!   call CheckLegacyAndVim9Failure(['VAR v = range(5)[2 : 3'], ['E111:', 'E1097:', 'E111:'])
!   call CheckLegacyAndVim9Failure(['VAR l = insert([1, 2, 3], 4, 10)'], 'E684:')
!   call CheckLegacyAndVim9Failure(['VAR l = insert([1, 2, 3], 4, -10)'], 'E684:')
!   call CheckLegacyAndVim9Failure(['VAR l = insert([1, 2, 3], 4, [])'], ['E745:', 'E1013:', 'E1210:'])
! 
!   call CheckLegacyAndVim9Failure(['VAR l = [1, 2, 3]', 'LET l[i] = 3'], ['E121:', 'E1001:', 'E121:'])
!   call CheckLegacyAndVim9Failure(['VAR l = [1, 2, 3]', 'LET l[1.1] = 4'], ['E805:', 'E1012:', 'E805:'])
!   call CheckLegacyAndVim9Failure(['VAR l = [1, 2, 3]', 'LET l[: i] = [4, 5]'], ['E121:', 'E1001:', 'E121:'])
!   call CheckLegacyAndVim9Failure(['VAR l = [1, 2, 3]', 'LET l[: 3.2] = [4, 5]'], ['E805:', 'E1012:', 'E805:'])
!   call CheckLegacyAndVim9Failure(['VAR t = test_unknown()', 'echo t[0]'], 'E685:')
  endfunc
  
  " Test for a null list
*** ../vim-8.2.3852/src/version.c	2021-12-19 15:17:16.221857770 +0000
--- src/version.c	2021-12-19 17:26:39.748144935 +0000
***************
*** 751,752 ****
--- 751,754 ----
  {   /* Add new patch number below this line */
+ /**/
+     3853,
  /**/

-- 
hundred-and-one symptoms of being an internet addict:
79. All of your most erotic dreams have a scrollbar at the right side.

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