-
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
Fail and time is not displaying #22
Comments
Hi @quolpr 😄 Yes, this was actually something I removed in #20. Basically, for each error that occurs, you are supposed to register it in a lua table, which looks like this: errors = {
{ message = "foo", line = 42 },
{ message = "bar", line = 69 },
} neotest-golang/lua/neotest-golang/init.lua Lines 272 to 276 in 2e42066
For each error, diagnostics are also registered, which you can see at the bottom of my screenshots in the #20 PR. Notice how the first one shows (incorrectly) 10 problems while the second screenshot shows the (accurate) 1 problem. I was previously excessively recording too many error messages, which had the side-effect of showing the FAIL stuff along with the time. But this also messed with other things, such as the number of diagnostics errors, as you can also see by the number of icons in front of the diagnostics with the time duration. This means if you want to view the details around your test run, such as the time info, you'll have to either open the output window or the output panel. |
@fredrikaverpil ahh, got it! Thanks for clarifications |
Here how it looks:
While on your screenshot I can see "Fail" + how much took near test
The output:
Config:
Also, I am curious, is it possible to add how much time test took even for green tests?
Great job btw! Thank you for the plugin 🙂
The text was updated successfully, but these errors were encountered: