-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bug: tests don't start, crash from nio #61
Comments
Hi @asfaltboy and thanks for the report. I can't reproduce this on my end. What does your neotest config look like, and how do you register the load neotest-golang along with any options? It looks like the |
Having the same issue with default config: return {
{
"nvim-neotest/neotest",
dependencies = {
"nvim-neotest/nvim-nio",
"nvim-lua/plenary.nvim",
"antoinemadec/FixCursorHold.nvim",
"nvim-treesitter/nvim-treesitter",
"fredrikaverpil/neotest-golang",
},
config = function()
require("neotest").setup {
adapters = {
require "neotest-golang",
},
}
end,
},
} Issue doesn't reproduce with 'extra everything' config from README |
Thanks for the details. I believe I might have found the issue and I've pushed a PR. Could you please help me out and see if this fixes this issue on your end? {
"fredrikaverpil/neotest-golang",
branch = "fix/options",
}, |
@fredrikaverpil yes, now it's fine. Thanks) |
Many thanks for the confirmation @uroborosq, I'm glad to hear this got fixed on your end as well 🥳 @asfaltboy I'm going to merge this fix in straight away as this was a significant mistake on my end. Please re-open this issue if it did not fix the problem on your end. |
Did you check docs and existing issues?
Neovim version (nvim -v)
v0.10.0
Operating system/version
MacOS 14.5
Describe the bug
Whenever I try to run tests with latest version of nio, neotest, and neotest-golang, I see this error:
Whether I select a specific test, or all tests in a file. Same tests run ok when using neotest-go instead
Steps To Reproduce
Expected Behavior
Should run tests
The text was updated successfully, but these errors were encountered: