# HG changeset patch # User Bram Moolenaar # Date 1608475505 -3600 # Node ID f2d05fb28e5418bca4c1df353de71804dbc9d202 # Parent f619a77afefe1164eaac31eb6eae2ebc970c5099 patch 8.2.2169: Vim9: test leaves file behind Commit: https://github.com/vim/vim/commit/090728ad4d54287c911894ef42bfe92844ce0fa5 Author: Bram Moolenaar Date: Sun Dec 20 15:43:31 2020 +0100 patch 8.2.2169: Vim9: test leaves file behind Problem: Vim9: test leaves file behind. Solution: Rename script files. (Dominique Pell?, closes https://github.com/vim/vim/issues/7511) Use try/finally. diff --git a/src/testdir/test_vim9_script.vim b/src/testdir/test_vim9_script.vim --- a/src/testdir/test_vim9_script.vim +++ b/src/testdir/test_vim9_script.vim @@ -2412,7 +2412,7 @@ def Test_vim9_comment() 'delcommand Echo', ]) CheckScriptSuccess([ - 'vim9script' + 'vim9script', 'command Echo cd # comment', 'Echo', 'delcommand Echo', @@ -2949,6 +2949,7 @@ def Test_restoring_cpo() endif delete('Xsourced') delete('Xclose') + delete('Xdone') enddef diff --git a/src/testdir/vim9.vim b/src/testdir/vim9.vim --- a/src/testdir/vim9.vim +++ b/src/testdir/vim9.vim @@ -5,13 +5,18 @@ let s:sequence = 1 " Check that "lines" inside a ":def" function has no error. func CheckDefSuccess(lines) - let fname = 'Xdef' .. s:sequence + let cwd = getcwd() + let fname = 'XdefSuccess' .. s:sequence let s:sequence += 1 call writefile(['def Func()'] + a:lines + ['enddef', 'defcompile'], fname) - exe 'so ' .. fname - call Func() - delfunc! Func - call delete(fname) + try + exe 'so ' .. fname + call Func() + delfunc! Func + finally + call chdir(cwd) + call delete(fname) + endtry endfunc " Check that "lines" inside ":def" results in an "error" message. @@ -19,12 +24,17 @@ endfunc " Add a line before and after to make it less likely that the line number is " accidentally correct. func CheckDefFailure(lines, error, lnum = -3) - let fname = 'Xdef' .. s:sequence + let cwd = getcwd() + let fname = 'XdefFailure' .. s:sequence + let s:sequence += 1 call writefile(['def Func()', '# comment'] + a:lines + ['#comment', 'enddef', 'defcompile'], fname) - call assert_fails('so ' .. fname, a:error, a:lines, a:lnum + 1) - delfunc! Func - call delete(fname) - let s:sequence += 1 + try + call assert_fails('so ' .. fname, a:error, a:lines, a:lnum + 1) + finally + call chdir(cwd) + call delete(fname) + delfunc! Func + endtry endfunc " Check that "lines" inside ":def" results in an "error" message when executed. @@ -32,29 +42,44 @@ endfunc " Add a line before and after to make it less likely that the line number is " accidentally correct. func CheckDefExecFailure(lines, error, lnum = -3) - let fname = 'Xdef' .. s:sequence + let cwd = getcwd() + let fname = 'XdefExecFailure' .. s:sequence let s:sequence += 1 call writefile(['def Func()', '# comment'] + a:lines + ['#comment', 'enddef'], fname) - exe 'so ' .. fname - call assert_fails('call Func()', a:error, a:lines, a:lnum + 1) - delfunc! Func - call delete(fname) + try + exe 'so ' .. fname + call assert_fails('call Func()', a:error, a:lines, a:lnum + 1) + finally + call chdir(cwd) + call delete(fname) + delfunc! Func + endtry endfunc def CheckScriptFailure(lines: list, error: string, lnum = -3) - var fname = 'Xdef' .. s:sequence + var cwd = getcwd() + var fname = 'XScriptFailure' .. s:sequence s:sequence += 1 writefile(lines, fname) - assert_fails('so ' .. fname, error, lines, lnum) - delete(fname) + try + assert_fails('so ' .. fname, error, lines, lnum) + finally + chdir(cwd) + delete(fname) + endtry enddef def CheckScriptSuccess(lines: list) - var fname = 'Xdef' .. s:sequence + var cwd = getcwd() + var fname = 'XScriptSuccess' .. s:sequence s:sequence += 1 writefile(lines, fname) - exe 'so ' .. fname - delete(fname) + try + exe 'so ' .. fname + finally + chdir(cwd) + delete(fname) + endtry enddef def CheckDefAndScriptSuccess(lines: list) diff --git a/src/version.c b/src/version.c --- a/src/version.c +++ b/src/version.c @@ -751,6 +751,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ /**/ + 2169, +/**/ 2168, /**/ 2167,