test: support injected query tests#584
Closed
PeterCardenas wants to merge 2 commits intonvim-treesitter:masterfrom
Closed
test: support injected query tests#584PeterCardenas wants to merge 2 commits intonvim-treesitter:masterfrom
PeterCardenas wants to merge 2 commits intonvim-treesitter:masterfrom
Conversation
Before we had snapshots to cover injected query tests, but now we can use the cursor and context markers to test these as well. This will mean that we might be installing more parsers and parsing more than we should be, but I think the higher test coverage is worth it.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The
context_spec.luatests do not cover test files that have injected queries. To cover these, we recursively parse the file to get the injected languages parsed, which then can be used later for the context tests. This PR also includes updating the html and markdown test files to take advantage of this capability.Followup to #568
TODO: