search

Issafalcon / neotest-dotnet

Neotest adapter for dotnet
MIT License
63 stars 20 forks source link

[BUG] lua/neotest-dotnet/strategies/netcoredbg.lua:62: attempt to index field 'dap' (a nil value) #91

Closed VladimirBeliakov closed 4 months ago

VladimirBeliakov commented 4 months ago

NeoVim Version NVIM v0.10.0-dev-2507+g3df1211ebc Build type: RelWithDebInfo LuaJIT 2.1.1702233742 Run "nvim -V1 -v" for more info

Describe the bug Run tests by issuing the following command

require("neotest").run.run({ strategy = "dap" })

Sorry for not providing any additional information but the bug is too obvious :)

For some reason this line

https://github.com/Issafalcon/neotest-dotnet/blob/c19df2a139d88c5b4130b830d2cbe63a2c6c6c0c/lua/neotest-dotnet/init.lua#L150

was not changed in the commit where the line

https://github.com/Issafalcon/neotest-dotnet/blob/c19df2a139d88c5b4130b830d2cbe63a2c6c6c0c/lua/neotest-dotnet/init.lua#L7

was changed.

So it causes an error on this line

https://github.com/Issafalcon/neotest-dotnet/blob/c19df2a139d88c5b4130b830d2cbe63a2c6c6c0c/lua/neotest-dotnet/strategies/netcoredbg.lua#L62

Issafalcon commented 4 months ago

Thanks for raising this @VladimirBeliakov ! Not sure how this slipped through the net, but I've pushed the fix up.