diff options
| author | Aleksey Kladov <aleksey.kladov@gmail.com> | 2021-05-09 17:41:25 +0300 |
|---|---|---|
| committer | Aleksey Kladov <aleksey.kladov@gmail.com> | 2021-05-09 17:47:02 +0300 |
| commit | edeb4927829380ed25e3899e85b2809bbb39a846 (patch) | |
| tree | 6714a4956f928e2259ad4a946a8be615443f2b4b | |
| parent | 5637ce4250247eef4c2ae1892cead69b6d6a172b (diff) | |
| download | rust-edeb4927829380ed25e3899e85b2809bbb39a846.tar.gz rust-edeb4927829380ed25e3899e85b2809bbb39a846.zip | |
minor: fix test style
| -rw-r--r-- | crates/ide/src/join_lines.rs | 12 | ||||
| -rw-r--r-- | crates/ide_assists/src/handlers/replace_impl_trait_with_generic.rs | 86 |
2 files changed, 31 insertions, 67 deletions
diff --git a/crates/ide/src/join_lines.rs b/crates/ide/src/join_lines.rs index 3e30e71ce91..61dcbb399dd 100644 --- a/crates/ide/src/join_lines.rs +++ b/crates/ide/src/join_lines.rs @@ -67,18 +67,6 @@ fn remove_newlines(edit: &mut TextEditBuilder, token: &SyntaxToken, range: TextR fn remove_newline(edit: &mut TextEditBuilder, token: &SyntaxToken, offset: TextSize) { if token.kind() != WHITESPACE || token.text().bytes().filter(|&b| b == b'\n').count() != 1 { - let mut no_space = false; - if let Some(string) = ast::String::cast(token.clone()) { - if let Some(range) = string.open_quote_text_range() { - cov_mark::hit!(join_string_literal_open_quote); - no_space |= range.end() == offset; - } - if let Some(range) = string.close_quote_text_range() { - cov_mark::hit!(join_string_literal_close_quote); - no_space |= range.start() == offset + TextSize::of('\n'); - } - } - let n_spaces_after_line_break = { let suff = &token.text()[TextRange::new( offset - token.text_range().start() + TextSize::of('\n'), diff --git a/crates/ide_assists/src/handlers/replace_impl_trait_with_generic.rs b/crates/ide_assists/src/handlers/replace_impl_trait_with_generic.rs index ff25b61ea27..8509a5e11f3 100644 --- a/crates/ide_assists/src/handlers/replace_impl_trait_with_generic.rs +++ b/crates/ide_assists/src/handlers/replace_impl_trait_with_generic.rs @@ -55,12 +55,8 @@ mod tests { fn replace_impl_trait_with_generic_params() { check_assist( replace_impl_trait_with_generic, - r#" - fn foo<G>(bar: $0impl Bar) {} - "#, - r#" - fn foo<G, B: Bar>(bar: B) {} - "#, + r#"fn foo<G>(bar: $0impl Bar) {}"#, + r#"fn foo<G, B: Bar>(bar: B) {}"#, ); } @@ -68,12 +64,8 @@ mod tests { fn replace_impl_trait_without_generic_params() { check_assist( replace_impl_trait_with_generic, - r#" - fn foo(bar: $0impl Bar) {} - "#, - r#" - fn foo<B: Bar>(bar: B) {} - "#, + r#"fn foo(bar: $0impl Bar) {}"#, + r#"fn foo<B: Bar>(bar: B) {}"#, ); } @@ -81,12 +73,8 @@ mod tests { fn replace_two_impl_trait_with_generic_params() { check_assist( replace_impl_trait_with_generic, - r#" - fn foo<G>(foo: impl Foo, bar: $0impl Bar) {} - "#, - r#" - fn foo<G, B: Bar>(foo: impl Foo, bar: B) {} - "#, + r#"fn foo<G>(foo: impl Foo, bar: $0impl Bar) {}"#, + r#"fn foo<G, B: Bar>(foo: impl Foo, bar: B) {}"#, ); } @@ -94,12 +82,8 @@ mod tests { fn replace_impl_trait_with_empty_generic_params() { check_assist( replace_impl_trait_with_generic, - r#" - fn foo<>(bar: $0impl Bar) {} - "#, - r#" - fn foo<B: Bar>(bar: B) {} - "#, + r#"fn foo<>(bar: $0impl Bar) {}"#, + r#"fn foo<B: Bar>(bar: B) {}"#, ); } @@ -108,13 +92,13 @@ mod tests { check_assist( replace_impl_trait_with_generic, r#" - fn foo< - >(bar: $0impl Bar) {} - "#, +fn foo< +>(bar: $0impl Bar) {} +"#, r#" - fn foo<B: Bar - >(bar: B) {} - "#, +fn foo<B: Bar +>(bar: B) {} +"#, ); } @@ -123,12 +107,8 @@ mod tests { fn replace_impl_trait_with_exist_generic_letter() { check_assist( replace_impl_trait_with_generic, - r#" - fn foo<B>(bar: $0impl Bar) {} - "#, - r#" - fn foo<B, C: Bar>(bar: C) {} - "#, + r#"fn foo<B>(bar: $0impl Bar) {}"#, + r#"fn foo<B, C: Bar>(bar: C) {}"#, ); } @@ -137,19 +117,19 @@ mod tests { check_assist( replace_impl_trait_with_generic, r#" - fn foo< - G: Foo, - F, - H, - >(bar: $0impl Bar) {} - "#, - r#" - fn foo< - G: Foo, - F, - H, B: Bar - >(bar: B) {} - "#, +fn foo< + G: Foo, + F, + H, +>(bar: $0impl Bar) {} +"#, + r#" +fn foo< + G: Foo, + F, + H, B: Bar +>(bar: B) {} +"#, ); } @@ -157,12 +137,8 @@ mod tests { fn replace_impl_trait_multiple() { check_assist( replace_impl_trait_with_generic, - r#" - fn foo(bar: $0impl Foo + Bar) {} - "#, - r#" - fn foo<F: Foo + Bar>(bar: F) {} - "#, + r#"fn foo(bar: $0impl Foo + Bar) {}"#, + r#"fn foo<F: Foo + Bar>(bar: F) {}"#, ); } } |
