Skip to content

test: replace plenary with minimal fork#874

Merged
clason merged 1 commit intomainfrom
test/plentest
Mar 21, 2026
Merged

test: replace plenary with minimal fork#874
clason merged 1 commit intomainfrom
test/plentest

Conversation

@clason
Copy link
Copy Markdown
Collaborator

@clason clason commented Mar 20, 2026

Problem: plenary.nvim as test runner is overkill and no longer
maintained.

Solution: Replace with a minimal fork based on Neovim API and vendored
luassert.

Follow-up to nvim-treesitter/nvim-treesitter#8592

This finally fixes user config polluting the system-under-testing by running the instances with --clean (which requires manual handling of rtp in the minimal init to ensure queries and parsers are available; follow-up should add a nvim-treesitter.setup call for explicit custom parser and query dir).

Problem: plenary.nvim as test runner is overkill and no longer
maintained.

Solution: Replace with a minimal fork based on Neovim API and vendored
luassert.
@clason
Copy link
Copy Markdown
Collaborator Author

clason commented Mar 20, 2026

@kiyoon instead of messing with XDG environment variables, wouldn't it be easier to set a custom $NVIM_APPNAME (as that is precisely what it's for)?

@kiyoon
Copy link
Copy Markdown
Collaborator

kiyoon commented Mar 20, 2026

that still makes your test directory saved multiple places, and it's harder to clean up.

I followed the approach here which is battle-tested and worked for me for a long time.

https://github.com/folke/lazy.nvim/blob/main/bootstrap.lua

@clason clason merged commit 93d60a4 into main Mar 21, 2026
3 checks passed
@clason clason deleted the test/plentest branch March 21, 2026 10:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants