diff options
| author | zeertzjq <zeertzjq@outlook.com> | 2026-07-04 08:32:38 +0800 |
|---|---|---|
| committer | zeertzjq <zeertzjq@outlook.com> | 2026-07-04 08:47:05 +0800 |
| commit | 60e46704bd793e52b7f62278719d4f92746c3d78 (patch) | |
| tree | 528ed2224cd1cdb181bbf877eb0e0cffaac8811d /test/functional/vimscript/executable_spec.lua | |
| parent | d7b7922dcb41cb435af408440322e4382532aad2 (diff) | |
| download | rneovim-60e46704bd793e52b7f62278719d4f92746c3d78.tar.gz rneovim-60e46704bd793e52b7f62278719d4f92746c3d78.tar.bz2 rneovim-60e46704bd793e52b7f62278719d4f92746c3d78.zip | |
vim-patch:9.2.0782: tests: missing cleanup in test_mksession.vim
Problem: Some test_mksession tests do not cleanup all the state
Solution: Add commands to clean up state introduced by the test
(Illia Bobyr)
Before this change the following 4 tests were failing when executed
individually:
Test_mksession_arglocal_localdir
Test_mksession_buffer_count
Test_mksession_one_buffer_two_windows
Test_mksession_winminheight
As in
```
TEST_FILTER=winminheight make test_mksession
```
Yet, when ran as part of the whole test suite they succeeded.
This was due to some state leaking from one test into another.
I think this is bad, as it can confuse someone making changes in the
relevant area.
`Test_mksession_winminheight` is actually still broken a bit, and
requires `winheight` and `winwidth` set at the beginning of the test,
rather than later, when it actually matters. This exposes a subtle bug
in the session restore script. I have a patch in a separate commit.
closes: vim/vim#20691
https://github.com/vim/vim/commit/834b8d218f1db92e587ecd449e5ce88b41fbe256
Co-authored-by: Illia Bobyr <illia.bobyr@gmail.com>
Diffstat (limited to 'test/functional/vimscript/executable_spec.lua')
0 files changed, 0 insertions, 0 deletions