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