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

[ty] Switch completion tests to use "insertion" text in snapshots 

I think this gives us better tests overall, since it
tells us what is actually going to be inserted. Plus,
we'll want this in the next commit.
This commit is contained in:
Andrew Gallant 2025-12-11 14:06:29 -05:00 committed by Andrew Gallant
parent 051f6896ac
commit 89fed85a8d
1 changed files with 15 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

40
crates/ty_ide/src/completion.rs
View File

@ -2594,9 +2594,7 @@ def frob(): ...
assert_snapshot!( assert_snapshot!(
builder.skip_keywords().skip_builtins().build().snapshot(), builder.skip_keywords().skip_builtins().build().snapshot(),
@r" @"foo=",
foo
",
); );
} }
@ -2610,9 +2608,7 @@ def frob(): ...
assert_snapshot!( assert_snapshot!(
builder.skip_keywords().skip_builtins().build().snapshot(), builder.skip_keywords().skip_builtins().build().snapshot(),
@r" @"foo=",
foo
",
); );
} }
@ -3269,7 +3265,7 @@ bar(o<CURSOR>
builder.skip_keywords().skip_builtins().skip_auto_import().build().snapshot(), builder.skip_keywords().skip_builtins().skip_auto_import().build().snapshot(),
@r" @r"
foo foo
okay okay=
" "
); );
} }
@ -3290,7 +3286,7 @@ bar(o<CURSOR>
builder.skip_keywords().skip_builtins().skip_auto_import().build().snapshot(), builder.skip_keywords().skip_builtins().skip_auto_import().build().snapshot(),
@r" @r"
foo foo
okay okay=
" "
); );
} }
@ -3308,9 +3304,9 @@ foo(b<CURSOR>
assert_snapshot!( assert_snapshot!(
builder.skip_keywords().skip_builtins().skip_auto_import().build().snapshot(), builder.skip_keywords().skip_builtins().skip_auto_import().build().snapshot(),
@r" @r"
bar bar=
barbaz barbaz=
baz baz=
" "
); );
} }
@ -3327,9 +3323,7 @@ foo(bar=1, b<CURSOR>
assert_snapshot!( assert_snapshot!(
builder.skip_keywords().skip_builtins().skip_auto_import().build().snapshot(), builder.skip_keywords().skip_builtins().skip_auto_import().build().snapshot(),
@r" @"baz="
baz
"
); );
} }
@ -3347,9 +3341,7 @@ abc(o<CURSOR>
assert_snapshot!( assert_snapshot!(
builder.skip_keywords().skip_builtins().skip_auto_import().build().snapshot(), builder.skip_keywords().skip_builtins().skip_auto_import().build().snapshot(),
@r" @"okay="
okay
"
); );
} }
@ -3365,9 +3357,7 @@ abc(okay=1, ba<CURSOR> baz=5
assert_snapshot!( assert_snapshot!(
builder.skip_keywords().skip_builtins().skip_auto_import().build().snapshot(), builder.skip_keywords().skip_builtins().skip_auto_import().build().snapshot(),
@r" @"bar="
bar
"
); );
} }
@ -3411,9 +3401,9 @@ bar(o<CURSOR>
builder.skip_keywords().skip_builtins().skip_auto_import().build().snapshot(), builder.skip_keywords().skip_builtins().skip_auto_import().build().snapshot(),
@r" @r"
foo foo
okay okay=
okay_abc okay_abc=
okay_okay okay_okay=
" "
); );
} }
@ -3433,7 +3423,7 @@ bar(<CURSOR>
assert_snapshot!(builder.skip_keywords().skip_builtins().build().snapshot(), @r" assert_snapshot!(builder.skip_keywords().skip_builtins().build().snapshot(), @r"
bar bar
foo foo
okay okay=
"); ");
} }
@ -6885,7 +6875,7 @@ def f(zqzqzq: str):
self.filtered self.filtered
.iter() .iter()
.map(|c| { .map(|c| {
let mut snapshot = c.name.as_str().to_string(); let mut snapshot = c.insert.as_deref().unwrap_or(c.name.as_str()).to_string();
if self.type_signatures { if self.type_signatures {
let ty = let ty =
c.ty.map(|ty| ty.display(self.db).to_string()) c.ty.map(|ty| ty.display(self.db).to_string())