diff options
author | Justin M. Keyes <justinkz@gmail.com> | 2025-01-04 16:48:00 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-01-04 16:48:00 -0800 |
commit | 64b0e6582ae8c273efbe39109bcb261079c2bcd4 (patch) | |
tree | 2e9de1f2224814dc10af8c5ef338e3aaa974f181 /runtime/lua/vim/treesitter/query.lua | |
parent | a8ace2c58a318552869462a36859aabf1cdfaa68 (diff) | |
download | rneovim-64b0e6582ae8c273efbe39109bcb261079c2bcd4.tar.gz rneovim-64b0e6582ae8c273efbe39109bcb261079c2bcd4.tar.bz2 rneovim-64b0e6582ae8c273efbe39109bcb261079c2bcd4.zip |
refactor(tests): merge n.spawn/n.spawn_argv into n.new_session #31859
Problem:
- `n.spawn()` is misleading because it also connects RPC, it's not just
"spawning" a process.
- It's confusing that `n.spawn()` and `n.spawn_argv()` are separate.
Solution:
- Replace `n.spawn()`/`n.spawn_argv()` with a single function `n.new_session()`.
This name aligns with the existing functions `n.set_session`/`n.get_session`.
- Note: removes direct handling of `prepend_argv`, but I doubt that was
important or intentional. If callers want to control use of `prepend_argv`
then we should add a new flag to `test.session.Opts`.
- Move `keep` to first parameter of `n.new_session()`.
- Add a `merge` flag to `test.session.Opts`
- Mark `_new_argv()` as private. Test should use clear/new_session/spawn_wait
instead.
Diffstat (limited to 'runtime/lua/vim/treesitter/query.lua')
0 files changed, 0 insertions, 0 deletions