Python
/
ruff
mirror of https://github.com/astral-sh/ruff
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[ty_test] Add use statements to hover.rs 

Import InlineFileAssertions, ParsedAssertion, and UnparsedAssertion
at the module level instead of using fully qualified crate::assertion
names throughout the code.

This makes the code cleaner and more idiomatic.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
This commit is contained in:
Douglas Creager 2025-10-08 14:16:16 -04:00
parent 51d5bc709d
commit 93db8833ef
1 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
crates/ty_test/src/hover.rs
View File

@ -11,6 +11,7 @@ use ruff_python_ast::AnyNodeRef;
use ruff_text_size::{Ranged, TextSize}; use ruff_text_size::{Ranged, TextSize};
use ty_python_semantic::{HasType, SemanticModel}; use ty_python_semantic::{HasType, SemanticModel};
use crate::assertion::{InlineFileAssertions, ParsedAssertion, UnparsedAssertion};
use crate::check_output::CheckOutput; use crate::check_output::CheckOutput;
use crate::db::Db; use crate::db::Db;
@ -82,7 +83,7 @@ fn infer_type_at_position(db: &Db, file: File, offset: TextSize) -> Option<Strin
pub(crate) fn generate_hover_outputs( pub(crate) fn generate_hover_outputs(
db: &Db, db: &Db,
file: File, file: File,
assertions: &crate::assertion::InlineFileAssertions, assertions: &InlineFileAssertions,
) -> Vec<CheckOutput> { ) -> Vec<CheckOutput> {
let source = source_text(db, file); let source = source_text(db, file);
let lines = line_index(db, file); let lines = line_index(db, file);
@ -95,12 +96,12 @@ pub(crate) fn generate_hover_outputs(
// Look for hover assertions in this line's assertions // Look for hover assertions in this line's assertions
for assertion in line_assertions.iter() { for assertion in line_assertions.iter() {
let crate::assertion::UnparsedAssertion::Hover(_, _) = assertion else { let UnparsedAssertion::Hover(_, _) = assertion else {
continue; continue;
}; };
// Parse the assertion to get the column // Parse the assertion to get the column
let Ok(crate::assertion::ParsedAssertion::Hover(hover)) = assertion.parse() else { let Ok(ParsedAssertion::Hover(hover)) = assertion.parse() else {
// Invalid hover assertion - will be caught as error by matcher // Invalid hover assertion - will be caught as error by matcher
continue; continue;
}; };