Skip to content

test(lsp): migrate client tests to effect fixtures#29045

Open
kitlangton wants to merge 2 commits into
test/config-effect-dslfrom
test/lsp-client-effect-fixtures
Open

test(lsp): migrate client tests to effect fixtures#29045
kitlangton wants to merge 2 commits into
test/config-effect-dslfrom
test/lsp-client-effect-fixtures

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

Summary

  • migrate LSP client tests to the Effect test harness with scoped instance fixtures
  • use AppFileSystem for test file writes
  • keep subprocess and timing interactions on live Effect tests

Verification

  • bunx prettier --write packages/opencode/test/lsp/client.test.ts
  • bunx oxlint packages/opencode/test/lsp/client.test.ts
  • bun typecheck (packages/opencode)
  • bun test --timeout 5000 test/lsp/client.test.ts (packages/opencode)

Notes

  • Test names were compared before/after and are unchanged.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant