-
Notifications
You must be signed in to change notification settings - Fork 5.7k
Closed
Labels
choresomething that we should get around to eventuallysomething that we should get around to eventuallylsprelated to the language serverrelated to the language servertestsrelated to testsrelated to tests
Description
As seen in eg. #22054 adding new global symbols requires us to update tests for the LSP.
It feels strange that touching an unrelated part of the codebase requires updates to a test that says lsp_jsr_autom_import_completion_import_map()
. These two are completely unrelated. We should tweak the test to not require these changes if new global symbols are added (or removed).
Metadata
Metadata
Assignees
Labels
choresomething that we should get around to eventuallysomething that we should get around to eventuallylsprelated to the language serverrelated to the language servertestsrelated to testsrelated to tests