search

Issafalcon / neotest-dotnet

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

Test appears as failed when using `Fact(DisplayName = "test name")` #79

Closed bartoszluka closed 7 months ago

bartoszluka commented 8 months ago

true negative

When using DisplayName property on Fact attribute, the test appears as failed even though it actually succeeded. Not setting this property works fine.

this is a true negative or a false positive (not sure which one)

version info:

neotest

url    https://github.com/nvim-neotest/neotest
branch master
commit d424d26

neotest-dotent

url     https://github.com/Issafalcon/neotest-dotnet
version 1.5.1
tag     stable
branch  main
commit  f7bdb60

neovim:

NVIM v0.10.0-dev-582d7f4
Build type: RelWithDebInfo
LuaJIT 2.1.1699801871
Issafalcon commented 7 months ago

Thanks for raising the issue @bartoszluka . I'll take a look at it soon.

Issafalcon commented 7 months ago

Hi @bartoszluka - Just managed to get round to take a look at this properly. Think the PR should fix it.

bartoszluka commented 7 months ago

I can confirm that it works now. Thank you!