fix(completion): remove bound insert of type in trait
This commit is contained in:
parent
a06525517b
commit
67cbd8f7c1
@ -37,7 +37,7 @@
|
|||||||
traits::get_missing_assoc_items, SymbolKind,
|
traits::get_missing_assoc_items, SymbolKind,
|
||||||
};
|
};
|
||||||
use syntax::{
|
use syntax::{
|
||||||
ast::{self, edit_in_place::AttrsOwnerEdit},
|
ast::{self, edit_in_place::AttrsOwnerEdit, HasTypeBounds},
|
||||||
AstNode, SyntaxElement, SyntaxKind, TextRange, T,
|
AstNode, SyntaxElement, SyntaxKind, TextRange, T,
|
||||||
};
|
};
|
||||||
use text_edit::TextEdit;
|
use text_edit::TextEdit;
|
||||||
@ -265,10 +265,21 @@ fn add_type_alias_impl(
|
|||||||
};
|
};
|
||||||
|
|
||||||
let start = transformed_ty.syntax().text_range().start();
|
let start = transformed_ty.syntax().text_range().start();
|
||||||
let Some(end) = transformed_ty
|
|
||||||
.eq_token()
|
let end = if let Some(end) =
|
||||||
.map(|tok| tok.text_range().start())
|
transformed_ty.colon_token().map(|tok| tok.text_range().start())
|
||||||
.or(transformed_ty.semicolon_token().map(|tok| tok.text_range().start())) else { return };
|
{
|
||||||
|
end
|
||||||
|
} else if let Some(end) = transformed_ty.eq_token().map(|tok| tok.text_range().start())
|
||||||
|
{
|
||||||
|
end
|
||||||
|
} else if let Some(end) =
|
||||||
|
transformed_ty.semicolon_token().map(|tok| tok.text_range().start())
|
||||||
|
{
|
||||||
|
end
|
||||||
|
} else {
|
||||||
|
return;
|
||||||
|
};
|
||||||
|
|
||||||
let len = end - start;
|
let len = end - start;
|
||||||
let mut decl = transformed_ty.syntax().text().slice(..len).to_string();
|
let mut decl = transformed_ty.syntax().text().slice(..len).to_string();
|
||||||
|
@ -1,7 +1,7 @@
|
|||||||
//! Completion tests for item list position.
|
//! Completion tests for item list position.
|
||||||
use expect_test::{expect, Expect};
|
use expect_test::{expect, Expect};
|
||||||
|
|
||||||
use crate::tests::{completion_list, BASE_ITEMS_FIXTURE};
|
use crate::tests::{check_edit, completion_list, BASE_ITEMS_FIXTURE};
|
||||||
|
|
||||||
fn check(ra_fixture: &str, expect: Expect) {
|
fn check(ra_fixture: &str, expect: Expect) {
|
||||||
let actual = completion_list(&format!("{}{}", BASE_ITEMS_FIXTURE, ra_fixture));
|
let actual = completion_list(&format!("{}{}", BASE_ITEMS_FIXTURE, ra_fixture));
|
||||||
@ -277,3 +277,91 @@ fn after_unit_struct() {
|
|||||||
"#]],
|
"#]],
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn type_in_impl_trait() {
|
||||||
|
check_edit(
|
||||||
|
"type O",
|
||||||
|
r"
|
||||||
|
struct A;
|
||||||
|
trait B {
|
||||||
|
type O: ?Sized;
|
||||||
|
}
|
||||||
|
impl B for A {
|
||||||
|
$0
|
||||||
|
}
|
||||||
|
",
|
||||||
|
r#"
|
||||||
|
struct A;
|
||||||
|
trait B {
|
||||||
|
type O: ?Sized;
|
||||||
|
}
|
||||||
|
impl B for A {
|
||||||
|
type O = $0;
|
||||||
|
}
|
||||||
|
"#,
|
||||||
|
);
|
||||||
|
check_edit(
|
||||||
|
"type O",
|
||||||
|
r"
|
||||||
|
struct A;
|
||||||
|
trait B {
|
||||||
|
type O;
|
||||||
|
}
|
||||||
|
impl B for A {
|
||||||
|
$0
|
||||||
|
}
|
||||||
|
",
|
||||||
|
r#"
|
||||||
|
struct A;
|
||||||
|
trait B {
|
||||||
|
type O;
|
||||||
|
}
|
||||||
|
impl B for A {
|
||||||
|
type O = $0;
|
||||||
|
}
|
||||||
|
"#,
|
||||||
|
);
|
||||||
|
check_edit(
|
||||||
|
"type O",
|
||||||
|
r"
|
||||||
|
struct A;
|
||||||
|
trait B {
|
||||||
|
type O: ?Sized = u32;
|
||||||
|
}
|
||||||
|
impl B for A {
|
||||||
|
$0
|
||||||
|
}
|
||||||
|
",
|
||||||
|
r#"
|
||||||
|
struct A;
|
||||||
|
trait B {
|
||||||
|
type O: ?Sized = u32;
|
||||||
|
}
|
||||||
|
impl B for A {
|
||||||
|
type O = $0;
|
||||||
|
}
|
||||||
|
"#,
|
||||||
|
);
|
||||||
|
check_edit(
|
||||||
|
"type O",
|
||||||
|
r"
|
||||||
|
struct A;
|
||||||
|
trait B {
|
||||||
|
type O = u32;
|
||||||
|
}
|
||||||
|
impl B for A {
|
||||||
|
$0
|
||||||
|
}
|
||||||
|
",
|
||||||
|
r"
|
||||||
|
struct A;
|
||||||
|
trait B {
|
||||||
|
type O = u32;
|
||||||
|
}
|
||||||
|
impl B for A {
|
||||||
|
type O = $0;
|
||||||
|
}
|
||||||
|
",
|
||||||
|
)
|
||||||
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user