-
-
Notifications
You must be signed in to change notification settings - Fork 273
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
New tests not detected #1997
Comments
It still uses the same LSP code analysis notifications for live updates as before. |
Hmm, when attaching to FSAC, I see that I get an exception from: The test is found but I suspect the notification is not getting to the client. @baronfel, shot in the dark: is |
Describe the bug
When adding a new unit test in an existing file, the new test does not get picked up.
Steps to reproduce
src\Fantomas.Core.Tests\Stroustrup\SynBindingFunctionExpressionTests.fs
[<Test>] let foo () = ()
Expected behaviour
The new test gets picked up.
Screenshots
Machine info
Additional context
I noticed @farlee2121 changed the way unit tests are detected in #1874. There no longer seems to be any notification coming from the LSP server.
Is the file being watched for changes? I can't really find any evidence for this in the current code.
The text was updated successfully, but these errors were encountered: