Skip to content

fix: make LSP signature help integration test reliable#318

Open
eugenioenko wants to merge 1 commit into
mainfrom
fix/lsp-signature-help-test
Open

fix: make LSP signature help integration test reliable#318
eugenioenko wants to merge 1 commit into
mainfrom
fix/lsp-signature-help-test

Conversation

@eugenioenko

Copy link
Copy Markdown
Owner

Summary

  • Added retry loop (up to 3 attempts) to the signature help test — types (, waits 5s for LSP response, backspaces and retries if the server returns null
  • Increased waitStable delays after ctrl+g (Go to Line) from 100ms to 1000ms in the completion tests
  • Simplified signature help test file to skip unnecessary Go to Line navigation
  • Root cause: typescript-language-server occasionally returns null for signature help when it hasn't finished indexing, even after reporting "initialized"

Test plan

  • Signature help test passes 20/20 locally (was 7/10 before fix)
  • All 3 LSP tests pass together 3/3 runs
  • Retry kicks in when needed (observed 11s run where retry succeeded)

🤖 Generated with Claude Code

https://claude.ai/code/session_018GG9dYw2AG18ro5c2dkPtm

The signature help test was flaky (~30% failure rate) because the
typescript-language-server occasionally returns null when it hasn't
finished indexing. Added a retry loop that types '(', checks for the
response, and retries up to 3 times. Also increased waitStable delays
for Go to Line dialog initialization in the completion tests.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018GG9dYw2AG18ro5c2dkPtm
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant