Description
PR #160 added should_retrigger() (client.rs) and the retry loop for ServerCancelled (-32802), but no unit tests cover this logic.
Missing coverage
should_retrigger(None) → true (retrigger when data absent)
should_retrigger({"retriggerRequest": false}) → false
should_retrigger({"retriggerRequest": true}) → true
- Retry exhaustion: after 3 retries, the original
ServerCancelled error is returned
Expected behavior
Unit tests in lsp/client.rs or a dedicated test module covering all four cases above.
Environment
Description
PR #160 added
should_retrigger()(client.rs) and the retry loop forServerCancelled(-32802), but no unit tests cover this logic.Missing coverage
should_retrigger(None)→true(retrigger when data absent)should_retrigger({"retriggerRequest": false})→falseshould_retrigger({"retriggerRequest": true})→trueServerCancellederror is returnedExpected behavior
Unit tests in
lsp/client.rsor a dedicated test module covering all four cases above.Environment