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: be more strict in matching diagnostics and quick fixes in regression tests #44644

Open
stamblerre opened this issue Feb 26, 2021 · 0 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

@stamblerre
Copy link
Contributor

Currently, regression tests allow you to match on a single diagnostic in a file without specifying its message. This can lead to us matching the wrong diagnostic or applying the wrong quick fixes.

Whenever we wait for diagnostics, we should wait for all of the diagnostics in the file to ensure we're not producing duplicates or new diagnostics. We should also always require the diagnostic message in all cases.

Finally, we should require a single diagnostic when we apply quick fixes, instead of applying all of the quick fixes available.

/cc @heschik

@stamblerre stamblerre added this to the gopls/v1.0.0 milestone Feb 26, 2021
@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 Feb 26, 2021
@stamblerre stamblerre added this to To Do in gopls on-deck Feb 28, 2021
@stamblerre stamblerre added the Testing An issue that has been verified to require only test changes, not just a test failure. label Jun 28, 2021
@stamblerre stamblerre moved this from To Do to P3 in gopls on-deck Jul 15, 2021
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
No open projects
Development

No branches or pull requests

2 participants