Skip to content

test(bridge): add unit tests for should_retrigger() and retry exhaustion path #161

Description

@bug-ops

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    P3Low: cosmetic, edge case unlikely in practiceenhancementNew feature or requestmcpls-coremcpls-core crate changes

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions