Skip to content

Commit 661ad01

Browse files
Copilotobserverw
andcommitted
Add test coverage for TyClient and PyreflyClient, improve example documentation
Co-authored-by: observerw <[email protected]>
1 parent cac1a54 commit 661ad01

File tree

2 files changed

+30
-1
lines changed

2 files changed

+30
-1
lines changed

examples/default_configuration.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,14 +92,16 @@ def example_inspecting_defaults():
9292
if config_map and config_map.has_global_config():
9393
print(f"✓ {client_cls.__name__} has default configuration")
9494

95-
# Try to find inlay hints config
95+
# Try to find configuration - inlay hints for most clients,
96+
# or other features like diagnostics for clients without inlay hints
9697
for section in [
9798
"rust-analyzer.inlayHints",
9899
"gopls.hints",
99100
"python.analysis.inlayHints",
100101
"typescript.inlayHints",
101102
"deno.inlayHints",
102103
"pyrefly.inlayHints",
104+
"ty.diagnostics", # TyClient uses diagnostics instead of inlay hints
103105
]:
104106
value = config_map.get(None, section)
105107
if value is not None:

tests/test_default_configuration.py

Lines changed: 27 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -145,3 +145,30 @@ def test_deno_default_config_has_inlay_hints():
145145
assert config_map is not None
146146
config = config_map.get(None, "deno.inlayHints")
147147
assert config is not None, "deno should have inlay hints configuration"
148+
149+
150+
def test_pyrefly_default_config_has_inlay_hints():
151+
"""Test that pyrefly default config enables inlay hints."""
152+
from lsp_client.clients.pyrefly import PyreflyClient
153+
154+
client = PyreflyClient()
155+
config_map = client.create_default_configuration_map()
156+
157+
assert config_map is not None
158+
config = config_map.get(None, "pyrefly.inlayHints")
159+
assert config is not None, "pyrefly should have inlay hints configuration"
160+
assert isinstance(config, dict), "pyrefly inlay hints should be a dict"
161+
162+
163+
def test_ty_default_config_has_diagnostics():
164+
"""Test that ty default config enables diagnostics and completion."""
165+
from lsp_client.clients.ty import TyClient
166+
167+
client = TyClient()
168+
config_map = client.create_default_configuration_map()
169+
170+
assert config_map is not None
171+
diagnostics = config_map.get(None, "ty.diagnostics")
172+
assert diagnostics is not None, "ty should have diagnostics configuration"
173+
completion = config_map.get(None, "ty.completion")
174+
assert completion is not None, "ty should have completion configuration"

0 commit comments

Comments
 (0)