Mercurial > vim
comparison src/testdir/runtest.vim @ 32697:e5cd5e8627da v9.0.1679
patch 9.0.1679: Cleanup Tests from leftover files
Commit: https://github.com/vim/vim/commit/84bc00e9b52b1174888f2f696f8b628a83c49988
Author: Christian Brabandt <cb@256bit.org>
Date: Thu Jul 13 11:45:54 2023 +0200
patch 9.0.1679: Cleanup Tests from leftover files
Problem: Tests may leave leftover files around
Solution: Clean up tests and remove files
There were a few failures in 'linux (huge, gcc, testgui, true, true)'
e.g. here: https://github.com/vim/vim/actions/runs/5497376153/jobs/10018060156
,----
| Error detected while processing command line..script /home/runner/work/vim/vim/src/testdir/runtest.vim[585]..function RunTheTest[54]..Test_lvimgrep_crash[16]..TestTimeout[12]..VimLeavePre Autocommands for "*"..function EarlyExit[7]..FinishTesting:
| line 70:
| E445: Other window contains changes
| E937: Attempt to delete a buffer that is in use: Xtest_stable_xxd.txt
| E937: Attempt to delete a buffer that is in use: Xtest_stable_xxd.txt
| E937: Attempt to delete a buffer that is in use: Xtest_stable_xxd.txtmalloc(): unsorted double linked list corrupted
`----
Which is puzzling, because the Xtest_stable_xxd file should have been
long gone after test_crypt.vim is run (and definitely no longer be
staying around in test_quickfix.vim).
So try to clean up properly after a test script is run, just in case any
X<file> is still around. During testing, a found a few leftover files,
which I also fixed in the relevant test-file.
Unfortunately, the test workflow 'linux (huge, gcc, testgui, true,
true)' now seems to fail with 'E1230: Encryption: sodium_mlock()' in
test_crypt.vim. Hopefully this is only temporary.
author | Christian Brabandt <cb@256bit.org> |
---|---|
date | Tue, 08 Aug 2023 20:45:05 +0200 |
parents | 0651d9be4d7c |
children | 5eab5c00bb7c |
comparison
equal
deleted
inserted
replaced
32696:35aeee78375e | 32697:e5cd5e8627da |
---|---|
409 call delete(name) | 409 call delete(name) |
410 endfor | 410 endfor |
411 endif | 411 endif |
412 endfunc | 412 endfunc |
413 | 413 |
414 function Delete_Xtest_Files() | |
415 for file in glob('X*', v:false, v:true) | |
416 if file ==? 'XfakeHOME' | |
417 " Clean up files created by setup.vim | |
418 call delete('XfakeHOME', 'rf') | |
419 continue | |
420 endif | |
421 " call add(v:errors, file .. " exists when it shouldn't, trying to delete it!") | |
422 call delete(file) | |
423 if !empty(glob(file, v:false, v:true)) | |
424 " call add(v:errors, file .. " still exists after trying to delete it!") | |
425 if has('unix') | |
426 call system('rm -rf ' .. file) | |
427 endif | |
428 endif | |
429 endfor | |
430 endfunc | |
431 | |
414 func AfterTheTest(func_name) | 432 func AfterTheTest(func_name) |
415 if len(v:errors) > 0 | 433 if len(v:errors) > 0 |
416 if match(s:may_fail_list, '^' .. a:func_name) >= 0 | 434 if match(s:may_fail_list, '^' .. a:func_name) >= 0 |
417 let s:fail_expected += 1 | 435 let s:fail_expected += 1 |
418 call add(s:errors_expected, 'Found errors in ' . g:testfunc . ':') | 436 call add(s:errors_expected, 'Found errors in ' . g:testfunc . ':') |
437 endfunc | 455 endfunc |
438 | 456 |
439 " This function can be called by a test if it wants to abort testing. | 457 " This function can be called by a test if it wants to abort testing. |
440 func FinishTesting() | 458 func FinishTesting() |
441 call AfterTheTest('') | 459 call AfterTheTest('') |
460 call Delete_Xtest_Files() | |
442 | 461 |
443 " Don't write viminfo on exit. | 462 " Don't write viminfo on exit. |
444 set viminfo= | 463 set viminfo= |
445 | |
446 " Clean up files created by setup.vim | |
447 call delete('XfakeHOME', 'rf') | |
448 | 464 |
449 if s:fail == 0 && s:fail_expected == 0 | 465 if s:fail == 0 && s:fail_expected == 0 |
450 " Success, create the .res file so that make knows it's done. | 466 " Success, create the .res file so that make knows it's done. |
451 exe 'split ' . fnamemodify(g:testname, ':r') . '.res' | 467 exe 'split ' . fnamemodify(g:testname, ':r') . '.res' |
452 write | 468 write |