Skip to content

test(lsp): add initialization tests for installed servers#2515

Open
bajrangCoder wants to merge 2 commits into
mainfrom
test-lsp-server-initialization
Open

test(lsp): add initialization tests for installed servers#2515
bajrangCoder wants to merge 2 commits into
mainfrom
test-lsp-server-initialization

Conversation

@bajrangCoder

Copy link
Copy Markdown
Member

Closes: #2511

@greptile-apps

greptile-apps Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds initialization tests for installed LSP servers. The main changes are:

  • Checks server installation before starting each test.
  • Starts a runtime and validates the LSP initialize response.
  • Adds per-test timeouts and connection cleanup.
  • Registers the LSP suite with the test runner.

Confidence Score: 5/5

This looks safe to merge.

  • The missing-server branch exits through the runner's skip mechanism.
  • No blocking issues remain in the updated code.

Important Files Changed

Filename Overview
src/test/lsp.tests.js Adds installation checks, runtime startup, initialization validation, timeouts, and cleanup for LSP server tests.
src/test/test-definitions.js Registers the new LSP initialization test suite.
src/test/tester.js Adds per-test timeout support and refreshes suite registration before reruns.

Reviews (2): Last reviewed commit: "fix" | Re-trigger Greptile

Comment thread src/test/lsp.tests.js
@bajrangCoder

This comment was marked as outdated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

Add LSP Tests to the Test Suite

1 participant