Skip to content
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

Support for tests #1267

Open
razzeee opened this issue May 12, 2021 · 7 comments
Open

Support for tests #1267

razzeee opened this issue May 12, 2021 · 7 comments
Labels
Milestone

Comments

@razzeee
Copy link
Contributor

razzeee commented May 12, 2021

With testing explorer coming to vscode soonish, it might be nice to discuss if this can be included in the protocol too. Apparently they choose how they wanted to implement this, but decided against something like lsp (or even making it part of lsp?)

https://code.visualstudio.com/updates/v1_56#_testing

@dbaeumer dbaeumer added feature-request Request for new features or functionality new request labels May 14, 2021
@dbaeumer dbaeumer added this to the Backlog milestone May 14, 2021
@dbaeumer
Copy link
Member

We are discussing how we can open up LSP to non language requests which would ease to add something like a test provider.

@mickaelistria
Copy link

This seems related to microsoft/vscode-debugadapter-node#154 .
It would be great to provide a Test Adapter Protocol model that both LSP and DAP could share.

@mfussenegger
Copy link

The Build Server Protocol also covers parts of running tests or tasks. Maybe worth having a look if LSP is to be extended with such functionality.

@dbaeumer dbaeumer added idea and removed feature-request Request for new features or functionality new request labels Nov 4, 2021
@davidanthoff
Copy link

The primary thing that I'd like to see is protocol support so that a LS can notify a client about the location and existence of tests. I don't think test execution needs to be part of the LSP.

@firelizzard18
Copy link

The Go language server (gopls) is planning on adding a custom command for test discovery.

@davidanthoff
Copy link

Yes, that is what we did in the Julia LS as well, add a custom notification that reports test locations back to the client.

@dbaeumer
Copy link
Member

A first version of the base protocol is available here that should ease defining new protocols like test: https://microsoft.github.io/language-server-protocol/specifications/base/0.9/specification/

@MariaSolOs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants