ruff
[ty] Add new `hover` mdtest assertion
#20786
Open

[ty] Add new `hover` mdtest assertion #20786

dcreager wants to merge 40 commits into main from dcreager/hover-mdtest
dcreager
dcreager Add hover assertion type to mdtest framework
37effea8
dcreager Add CheckOutput enum to support hover results in mdtest
be47fbe0
dcreager Implement hover type inference and matching for mdtest
11e5ecb9
dcreager Update PLAN.md - mark steps 4 & 5 complete
6e73b859
dcreager [ty_test] Refactor hover logic into separate module
5c75e91a
dcreager [ty_test] Simplify infer_type_at_position using as_expr_ref()
9d8e35b1
dcreager [ty_test] Fix find_covering_node to correctly find minimal node
6370aea6
dcreager [ty_test] Simplify find_covering_node by comparing range lengths
319f5be7
dcreager [ty_test] Fix hover assertion line number calculation
19600ecd
dcreager [ty_test] Use let-else pattern in generate_hover_outputs
ab261360
dcreager [ty_test] Refactor: rename diagnostic.rs to check_output.rs
ac1b68c5
dcreager [ty_test] Extract HoverOutput type from CheckOutput enum
35a5fd76
dcreager [ty_test] Move HoverOutput to hover module
35b568dd
dcreager [ty_test] Simplify unmatched output handling
01988572
dcreager [ty_test] Calculate hover column at parse time
51d5bc70
dcreager [ty_test] Add use statements to hover.rs
93db8833
dcreager [ty_test] Store HoverAssertion.column as zero-based
8221450c
dcreager [ty_test] Fix hover column calculation to use line position
c3626a6d
dcreager [ty_test] Calculate hover column at parse time
a2eaf7ce
dcreager [ty_test] Use named fields for UnparsedAssertion::Hover
8c12fcb9
dcreager [ty_test] Fix column units: use character offset not byte offset
c574dff6
dcreager [ty_test] Simplify column calculation using line_column
6d1c549c
dcreager [ty_test] Store hover column as OneIndexed
b683da8c
dcreager [ty_test] Change match_line to take slice of CheckOutput
adf58b6c
dcreager [ty_test] Fix clippy warnings in hover module
180d9de4
dcreager [ty_test] Store CheckOutput references in SortedCheckOutputs
41772466
dcreager [ty] Handle expression statements in hover type inference
847e5f0c
dcreager [ty] Update PLAN.md with testing progress
b18d2138
dcreager [ty] Add comprehensive hover.md mdtest (partial)
eaba8bc6
dcreager clean up mdtest
6ddf7298
dcreager [ty] Fix hover to prefer expression nodes over identifiers
c08a2d6d
dcreager use ty_ide
8426cc69
dcreager update tests
7b88440f
dcreager remove finished plan
39353da1
dcreager precommit
1810b7c7
dcreager update tests
fbc211f3
dcreager clean up the diff
b438631d
dcreager dcreager requested a review from carljm carljm 82 days ago
dcreager dcreager requested a review from AlexWaygood AlexWaygood 82 days ago
dcreager dcreager added internal
dcreager dcreager requested a review from sharkdp sharkdp 82 days ago
dcreager dcreager requested a review from MichaReiser MichaReiser 82 days ago
dcreager dcreager added ty
github-actions
github-actions
github-actions
AlexWaygood
AlexWaygood commented on 2025-10-09
dcreager reword some docs
34b463d0
dcreager spelling
0817a367
dcreager clean up some claude-isms
eb95e49f
sharkdp
dcreager
dcreager
sharkdp
AlexWaygood
MichaReiser
MichaReiser commented on 2025-10-10

Login to write a write a comment.

Login via GitHub

Assignees
No one assigned
Labels
Milestone