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.

Sign up for GitHub

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

Jump to bottom

Add language server integration tests #266

Open
adalinesimonian opened this issue Oct 21, 2021 · 0 comments
Open

Add language server integration tests #266

adalinesimonian opened this issue Oct 21, 2021 · 0 comments
Labels
effort: moderate Requires experience with the codebase, but feasible. Harder than "effort: casual". status: ready to implement is ready to be worked on by someone type: tests an improvement to testing
Projects
Rolling Work Tracking
Milestone
On Deck

Comments

@adalinesimonian
Copy link
Member

adalinesimonian commented Oct 21, 2021 •
edited

Related to #269

#265 introduces a new modular architecture with 100% unit test coverage. However, the language server still has no integration tests.

Tests should be written that start up an instance of the server and use the language server client package to run tests against it.
@adalinesimonian adalinesimonian added status: blocked is blocked by another issue or pr status: ready to implement is ready to be worked on by someone type: tests an improvement to testing labels Oct 21, 2021
@adalinesimonian adalinesimonian added this to the On Deck milestone Oct 21, 2021
@adalinesimonian adalinesimonian added status: ready to implement is ready to be worked on by someone and removed status: ready to implement is ready to be worked on by someone status: blocked is blocked by another issue or pr labels Oct 21, 2021
@adalinesimonian adalinesimonian mentioned this issue Oct 22, 2021
Open
@adalinesimonian adalinesimonian added the effort: moderate Requires experience with the codebase, but feasible. Harder than "effort: casual". label Oct 29, 2021
@adalinesimonian adalinesimonian mentioned this issue Nov 6, 2021
Merged
@adalinesimonian adalinesimonian mentioned this issue Nov 16, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
effort: moderate Requires experience with the codebase, but feasible. Harder than "effort: casual". status: ready to implement is ready to be worked on by someone type: tests an improvement to testing
Projects
Rolling Work Tracking
To do
Milestone
On Deck
Development

No branches or pull requests
1 participant
@adalinesimonian