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

x/tools/gopls: add source test for inlay hints #53315

Open
suzmue opened this issue Jun 9, 2022 · 2 comments
Open

x/tools/gopls: add source test for inlay hints #53315

suzmue opened this issue Jun 9, 2022 · 2 comments
Labels
gopls Issues related to the Go language server, gopls. Testing An issue that has been verified to require only test changes, not just a test failure. Tools This label describes issues relating to any tools in the x/tools repository.
Milestone

Comments

@suzmue
Copy link
Contributor

suzmue commented Jun 9, 2022

Track progress of source tests for inlay hints.

@gopherbot gopherbot added Tools This label describes issues relating to any tools in the x/tools repository. gopls Issues related to the Go language server, gopls. labels Jun 9, 2022
@gopherbot gopherbot added this to the Unreleased milestone Jun 9, 2022
@sidntrivedi012
Copy link

Hey @suzmue 👋 , would like to help out with this issue. Would be great if you can please provide some more context regarding what needs to be done here. Thanks! 🙌

@suzmue suzmue modified the milestones: Unreleased, gopls/later Jun 13, 2022
@suzmue
Copy link
Contributor Author

suzmue commented Jun 13, 2022

With the addition of features that are exposed through inlay hints, we may want to add source tests for these features (code pointer here).

These tests would need to call the source package's implementation here.

The lsp tests currently take the approach of getting the inlay hints for the whole file and then applying them to the document as if they were text edits. It will probably require some additional set up to be able to track what inlay hints we are expecting from these source tests.

@findleyr findleyr added the Testing An issue that has been verified to require only test changes, not just a test failure. label Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gopls Issues related to the Go language server, gopls. Testing An issue that has been verified to require only test changes, not just a test failure. Tools This label describes issues relating to any tools in the x/tools repository.
Projects
None yet
Development

No branches or pull requests

4 participants