Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(test): add completion tests with applying in editor #276

Merged
merged 3 commits into from
Feb 19, 2025

Conversation

xpyctumo
Copy link
Contributor

Resolves: #275

As an example, I took all the existing tests for completions and duplicated them, but, now, the tests choose the most relevant option for completion (depending on the beginning of the string).

How completions are applied:

  • If one exists, the template is applied.
  • If there are no suitable variants, the first available variant in the code is applied.

Want to do:

  • It might be worth working on auto-duplicating tests in the future

@xpyctumo xpyctumo added the lsp: completion `textDocument/` Completion Request label Feb 19, 2025
@i582 i582 changed the title test: Add Complete test with relevant selection in editor feat(test): add completion tests with applying in editor Feb 19, 2025
@i582 i582 merged commit cdfbe2f into master Feb 19, 2025
4 checks passed
@i582 i582 deleted the completion_select_tests branch February 19, 2025 15:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
lsp: completion `textDocument/` Completion Request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add fully completions tests
2 participants