aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorbors[bot] <26634292+bors[bot]@users.noreply.github.com>2020-07-04 14:06:04 +0100
committerGitHub <[email protected]>2020-07-04 14:06:04 +0100
commit1ee524bb42c6dd2e24e8ee8561427e0a9991e62f (patch)
treee6d02b96f801cf835eb65d5e8b0dce38eb48fbeb
parent9f754e0518e7f2860891079a117a3e4bc6f4a8fd (diff)
parent9dacd2338cbc7b414fc7a22be108500abfa826f4 (diff)
Merge #5220
5220: Fix lookup in tests r=matklad a=matklad bors r+ 🤖 Co-authored-by: Aleksey Kladov <[email protected]>
-rw-r--r--crates/ra_ide/src/completion/complete_fn_param.rs124
-rw-r--r--crates/ra_ide/src/completion/complete_macro_in_item_position.rs135
-rw-r--r--crates/ra_ide/src/completion/complete_pattern.rs1
-rw-r--r--crates/ra_ide/src/completion/complete_qualified_path.rs1
-rw-r--r--crates/ra_ide/src/completion/complete_unqualified_path.rs9
-rw-r--r--crates/ra_ide/src/completion/presentation.rs54
6 files changed, 141 insertions, 183 deletions
diff --git a/crates/ra_ide/src/completion/complete_fn_param.rs b/crates/ra_ide/src/completion/complete_fn_param.rs
index f5573ddf7..9fb5c050e 100644
--- a/crates/ra_ide/src/completion/complete_fn_param.rs
+++ b/crates/ra_ide/src/completion/complete_fn_param.rs
@@ -54,85 +54,81 @@ pub(super) fn complete_fn_param(acc: &mut Completions, ctx: &CompletionContext)
54 54
55#[cfg(test)] 55#[cfg(test)]
56mod tests { 56mod tests {
57 use crate::completion::{test_utils::do_completion, CompletionItem, CompletionKind}; 57 use expect::{expect, Expect};
58 use insta::assert_debug_snapshot;
59 58
60 fn do_magic_completion(code: &str) -> Vec<CompletionItem> { 59 use crate::completion::{test_utils::do_completion, CompletionKind};
61 do_completion(code, CompletionKind::Magic) 60
61 fn check(ra_fixture: &str, expect: Expect) {
62 let actual = do_completion(ra_fixture, CompletionKind::Magic);
63 expect.assert_debug_eq(&actual);
62 } 64 }
63 65
64 #[test] 66 #[test]
65 fn test_param_completion_last_param() { 67 fn test_param_completion_last_param() {
66 assert_debug_snapshot!( 68 check(
67 do_magic_completion( 69 r#"
68 r" 70fn foo(file_id: FileId) {}
69 fn foo(file_id: FileId) {} 71fn bar(file_id: FileId) {}
70 fn bar(file_id: FileId) {} 72fn baz(file<|>) {}
71 fn baz(file<|>) {} 73"#,
72 ", 74 expect![[r#"
73 ), 75 [
74 @r###" 76 CompletionItem {
75 [ 77 label: "file_id: FileId",
76 CompletionItem { 78 source_range: 61..65,
77 label: "file_id: FileId", 79 delete: 61..65,
78 source_range: 61..65, 80 insert: "file_id: FileId",
79 delete: 61..65, 81 lookup: "file_id",
80 insert: "file_id: FileId", 82 },
81 lookup: "file_id", 83 ]
82 }, 84 "#]],
83 ]
84 "###
85 ); 85 );
86 } 86 }
87 87
88 #[test] 88 #[test]
89 fn test_param_completion_nth_param() { 89 fn test_param_completion_nth_param() {
90 assert_debug_snapshot!( 90 check(
91 do_magic_completion( 91 r#"
92 r" 92fn foo(file_id: FileId) {}
93 fn foo(file_id: FileId) {} 93fn bar(file_id: FileId) {}
94 fn bar(file_id: FileId) {} 94fn baz(file<|>, x: i32) {}
95 fn baz(file<|>, x: i32) {} 95"#,
96 ", 96 expect![[r#"
97 ), 97 [
98 @r###" 98 CompletionItem {
99 [ 99 label: "file_id: FileId",
100 CompletionItem { 100 source_range: 61..65,
101 label: "file_id: FileId", 101 delete: 61..65,
102 source_range: 61..65, 102 insert: "file_id: FileId",
103 delete: 61..65, 103 lookup: "file_id",
104 insert: "file_id: FileId", 104 },
105 lookup: "file_id", 105 ]
106 }, 106 "#]],
107 ]
108 "###
109 ); 107 );
110 } 108 }
111 109
112 #[test] 110 #[test]
113 fn test_param_completion_trait_param() { 111 fn test_param_completion_trait_param() {
114 assert_debug_snapshot!( 112 check(
115 do_magic_completion( 113 r#"
116 r" 114pub(crate) trait SourceRoot {
117 pub(crate) trait SourceRoot { 115 pub fn contains(&self, file_id: FileId) -> bool;
118 pub fn contains(&self, file_id: FileId) -> bool; 116 pub fn module_map(&self) -> &ModuleMap;
119 pub fn module_map(&self) -> &ModuleMap; 117 pub fn lines(&self, file_id: FileId) -> &LineIndex;
120 pub fn lines(&self, file_id: FileId) -> &LineIndex; 118 pub fn syntax(&self, file<|>)
121 pub fn syntax(&self, file<|>) 119}
122 } 120"#,
123 ", 121 expect![[r#"
124 ), 122 [
125 @r###" 123 CompletionItem {
126 [ 124 label: "file_id: FileId",
127 CompletionItem { 125 source_range: 208..212,
128 label: "file_id: FileId", 126 delete: 208..212,
129 source_range: 208..212, 127 insert: "file_id: FileId",
130 delete: 208..212, 128 lookup: "file_id",
131 insert: "file_id: FileId", 129 },
132 lookup: "file_id", 130 ]
133 }, 131 "#]],
134 ]
135 "###
136 ); 132 );
137 } 133 }
138} 134}
diff --git a/crates/ra_ide/src/completion/complete_macro_in_item_position.rs b/crates/ra_ide/src/completion/complete_macro_in_item_position.rs
index 4c33f41d4..d6613ed7b 100644
--- a/crates/ra_ide/src/completion/complete_macro_in_item_position.rs
+++ b/crates/ra_ide/src/completion/complete_macro_in_item_position.rs
@@ -15,130 +15,27 @@ pub(super) fn complete_macro_in_item_position(acc: &mut Completions, ctx: &Compl
15 15
16#[cfg(test)] 16#[cfg(test)]
17mod tests { 17mod tests {
18 use insta::assert_debug_snapshot; 18 use expect::{expect, Expect};
19 19
20 use crate::completion::{test_utils::do_completion, CompletionItem, CompletionKind}; 20 use crate::completion::{test_utils::completion_list, CompletionKind};
21 21
22 fn do_reference_completion(code: &str) -> Vec<CompletionItem> { 22 fn check(ra_fixture: &str, expect: Expect) {
23 do_completion(code, CompletionKind::Reference) 23 let actual = completion_list(ra_fixture, CompletionKind::Reference);
24 expect.assert_eq(&actual)
24 } 25 }
25 26
26 #[test] 27 #[test]
27 fn completes_macros_as_item() { 28 fn completes_macros_as_item() {
28 assert_debug_snapshot!( 29 check(
29 do_reference_completion( 30 r#"
30 " 31macro_rules! foo { () => {} }
31 //- /main.rs 32fn foo() {}
32 macro_rules! foo { 33
33 () => {} 34<|>
34 } 35"#,
35 36 expect![[r#"
36 fn foo() {} 37 ma foo!(…) macro_rules! foo
37 38 "#]],
38 <|> 39 )
39 "
40 ),
41 @r###"
42 [
43 CompletionItem {
44 label: "foo!(…)",
45 source_range: 48..48,
46 delete: 48..48,
47 insert: "foo!($0)",
48 kind: Macro,
49 detail: "macro_rules! foo",
50 },
51 ]
52 "###
53 );
54 }
55
56 #[test]
57 fn completes_vec_macros_with_square_brackets() {
58 assert_debug_snapshot!(
59 do_reference_completion(
60 "
61 //- /main.rs
62 /// Creates a [`Vec`] containing the arguments.
63 ///
64 /// - Create a [`Vec`] containing a given list of elements:
65 ///
66 /// ```
67 /// let v = vec![1, 2, 3];
68 /// assert_eq!(v[0], 1);
69 /// assert_eq!(v[1], 2);
70 /// assert_eq!(v[2], 3);
71 /// ```
72 macro_rules! vec {
73 () => {}
74 }
75
76 fn foo() {}
77
78 <|>
79 "
80 ),
81 @r###"
82 [
83 CompletionItem {
84 label: "vec![…]",
85 source_range: 282..282,
86 delete: 282..282,
87 insert: "vec![$0]",
88 kind: Macro,
89 detail: "macro_rules! vec",
90 documentation: Documentation(
91 "Creates a [`Vec`] containing the arguments.\n\n- Create a [`Vec`] containing a given list of elements:\n\n```\nlet v = vec![1, 2, 3];\nassert_eq!(v[0], 1);\nassert_eq!(v[1], 2);\nassert_eq!(v[2], 3);\n```",
92 ),
93 },
94 ]
95 "###
96 );
97 }
98
99 #[test]
100 fn completes_macros_braces_guessing() {
101 assert_debug_snapshot!(
102 do_reference_completion(
103 "
104 //- /main.rs
105 /// Foo
106 ///
107 /// Not call `fooo!()` `fooo!()`, or `_foo![]` `_foo![]`.
108 /// Call as `let _=foo! { hello world };`
109 macro_rules! foo {
110 () => {}
111 }
112
113 fn main() {
114 <|>
115 }
116 "
117 ),
118 @r###"
119 [
120 CompletionItem {
121 label: "foo! {…}",
122 source_range: 164..164,
123 delete: 164..164,
124 insert: "foo! {$0}",
125 kind: Macro,
126 detail: "macro_rules! foo",
127 documentation: Documentation(
128 "Foo\n\nNot call `fooo!()` `fooo!()`, or `_foo![]` `_foo![]`.\nCall as `let _=foo! { hello world };`",
129 ),
130 },
131 CompletionItem {
132 label: "main()",
133 source_range: 164..164,
134 delete: 164..164,
135 insert: "main()$0",
136 kind: Function,
137 lookup: "main",
138 detail: "fn main()",
139 },
140 ]
141 "###
142 );
143 } 40 }
144} 41}
diff --git a/crates/ra_ide/src/completion/complete_pattern.rs b/crates/ra_ide/src/completion/complete_pattern.rs
index 367e2bbce..9b79e7a09 100644
--- a/crates/ra_ide/src/completion/complete_pattern.rs
+++ b/crates/ra_ide/src/completion/complete_pattern.rs
@@ -130,6 +130,7 @@ mod tests {
130 delete: 90..90, 130 delete: 90..90,
131 insert: "m!($0)", 131 insert: "m!($0)",
132 kind: Macro, 132 kind: Macro,
133 lookup: "m!",
133 detail: "macro_rules! m", 134 detail: "macro_rules! m",
134 }, 135 },
135 ] 136 ]
diff --git a/crates/ra_ide/src/completion/complete_qualified_path.rs b/crates/ra_ide/src/completion/complete_qualified_path.rs
index a16866cd2..5175c8afe 100644
--- a/crates/ra_ide/src/completion/complete_qualified_path.rs
+++ b/crates/ra_ide/src/completion/complete_qualified_path.rs
@@ -1189,6 +1189,7 @@ mod tests {
1189 delete: 82..82, 1189 delete: 82..82,
1190 insert: "foo!($0)", 1190 insert: "foo!($0)",
1191 kind: Macro, 1191 kind: Macro,
1192 lookup: "foo!",
1192 detail: "#[macro_export]\nmacro_rules! foo", 1193 detail: "#[macro_export]\nmacro_rules! foo",
1193 }, 1194 },
1194 CompletionItem { 1195 CompletionItem {
diff --git a/crates/ra_ide/src/completion/complete_unqualified_path.rs b/crates/ra_ide/src/completion/complete_unqualified_path.rs
index a0a04bb58..0d1e9f8ea 100644
--- a/crates/ra_ide/src/completion/complete_unqualified_path.rs
+++ b/crates/ra_ide/src/completion/complete_unqualified_path.rs
@@ -785,6 +785,7 @@ mod tests {
785 delete: 256..256, 785 delete: 256..256,
786 insert: "bar!($0)", 786 insert: "bar!($0)",
787 kind: Macro, 787 kind: Macro,
788 lookup: "bar!",
788 detail: "macro_rules! bar", 789 detail: "macro_rules! bar",
789 }, 790 },
790 CompletionItem { 791 CompletionItem {
@@ -793,6 +794,7 @@ mod tests {
793 delete: 256..256, 794 delete: 256..256,
794 insert: "baz!($0)", 795 insert: "baz!($0)",
795 kind: Macro, 796 kind: Macro,
797 lookup: "baz!",
796 detail: "#[macro_export]\nmacro_rules! baz", 798 detail: "#[macro_export]\nmacro_rules! baz",
797 }, 799 },
798 CompletionItem { 800 CompletionItem {
@@ -801,6 +803,7 @@ mod tests {
801 delete: 256..256, 803 delete: 256..256,
802 insert: "foo!($0)", 804 insert: "foo!($0)",
803 kind: Macro, 805 kind: Macro,
806 lookup: "foo!",
804 detail: "macro_rules! foo", 807 detail: "macro_rules! foo",
805 }, 808 },
806 CompletionItem { 809 CompletionItem {
@@ -854,6 +857,7 @@ mod tests {
854 delete: 50..50, 857 delete: 50..50,
855 insert: "foo!($0)", 858 insert: "foo!($0)",
856 kind: Macro, 859 kind: Macro,
860 lookup: "foo!",
857 detail: "macro_rules! foo", 861 detail: "macro_rules! foo",
858 }, 862 },
859 CompletionItem { 863 CompletionItem {
@@ -893,6 +897,7 @@ mod tests {
893 delete: 58..58, 897 delete: 58..58,
894 insert: "foo!($0)", 898 insert: "foo!($0)",
895 kind: Macro, 899 kind: Macro,
900 lookup: "foo!",
896 detail: "macro_rules! foo", 901 detail: "macro_rules! foo",
897 }, 902 },
898 CompletionItem { 903 CompletionItem {
@@ -932,6 +937,7 @@ mod tests {
932 delete: 51..51, 937 delete: 51..51,
933 insert: "foo!($0)", 938 insert: "foo!($0)",
934 kind: Macro, 939 kind: Macro,
940 lookup: "foo!",
935 detail: "macro_rules! foo", 941 detail: "macro_rules! foo",
936 }, 942 },
937 CompletionItem { 943 CompletionItem {
@@ -1005,6 +1011,7 @@ mod tests {
1005 delete: 80..80, 1011 delete: 80..80,
1006 insert: "m!($0)", 1012 insert: "m!($0)",
1007 kind: Macro, 1013 kind: Macro,
1014 lookup: "m!",
1008 detail: "macro_rules! m", 1015 detail: "macro_rules! m",
1009 }, 1016 },
1010 CompletionItem { 1017 CompletionItem {
@@ -1058,6 +1065,7 @@ mod tests {
1058 delete: 80..81, 1065 delete: 80..81,
1059 insert: "m!($0)", 1066 insert: "m!($0)",
1060 kind: Macro, 1067 kind: Macro,
1068 lookup: "m!",
1061 detail: "macro_rules! m", 1069 detail: "macro_rules! m",
1062 }, 1070 },
1063 CompletionItem { 1071 CompletionItem {
@@ -1111,6 +1119,7 @@ mod tests {
1111 delete: 80..81, 1119 delete: 80..81,
1112 insert: "m!($0)", 1120 insert: "m!($0)",
1113 kind: Macro, 1121 kind: Macro,
1122 lookup: "m!",
1114 detail: "macro_rules! m", 1123 detail: "macro_rules! m",
1115 }, 1124 },
1116 CompletionItem { 1125 CompletionItem {
diff --git a/crates/ra_ide/src/completion/presentation.rs b/crates/ra_ide/src/completion/presentation.rs
index dc391c46b..946bbef7c 100644
--- a/crates/ra_ide/src/completion/presentation.rs
+++ b/crates/ra_ide/src/completion/presentation.rs
@@ -174,6 +174,7 @@ impl Completions {
174 builder 174 builder
175 .insert_snippet(cap, format!("{}!{}$0{}", name, bra, ket)) 175 .insert_snippet(cap, format!("{}!{}$0{}", name, bra, ket))
176 .label(format!("{}!{}…{}", name, bra, ket)) 176 .label(format!("{}!{}…{}", name, bra, ket))
177 .lookup_by(format!("{}!", name))
177 } 178 }
178 None if needs_bang => builder.insert_text(format!("{}!", name)), 179 None if needs_bang => builder.insert_text(format!("{}!", name)),
179 _ => { 180 _ => {
@@ -1079,4 +1080,57 @@ fn go(world: &WorldSnapshot) { go(w<|>) }
1079 "#]], 1080 "#]],
1080 ); 1081 );
1081 } 1082 }
1083
1084 #[test]
1085 fn guesses_macro_braces() {
1086 check_edit(
1087 "vec!",
1088 r#"
1089/// Creates a [`Vec`] containing the arguments.
1090///
1091/// ```
1092/// let v = vec![1, 2, 3];
1093/// assert_eq!(v[0], 1);
1094/// assert_eq!(v[1], 2);
1095/// assert_eq!(v[2], 3);
1096/// ```
1097macro_rules! vec { () => {} }
1098
1099fn fn main() { v<|> }
1100"#,
1101 r#"
1102/// Creates a [`Vec`] containing the arguments.
1103///
1104/// ```
1105/// let v = vec![1, 2, 3];
1106/// assert_eq!(v[0], 1);
1107/// assert_eq!(v[1], 2);
1108/// assert_eq!(v[2], 3);
1109/// ```
1110macro_rules! vec { () => {} }
1111
1112fn fn main() { vec![$0] }
1113"#,
1114 );
1115
1116 check_edit(
1117 "foo!",
1118 r#"
1119/// Foo
1120///
1121/// Don't call `fooo!()` `fooo!()`, or `_foo![]` `_foo![]`,
1122/// call as `let _=foo! { hello world };`
1123macro_rules! foo { () => {} }
1124fn main() { <|> }
1125"#,
1126 r#"
1127/// Foo
1128///
1129/// Don't call `fooo!()` `fooo!()`, or `_foo![]` `_foo![]`,
1130/// call as `let _=foo! { hello world };`
1131macro_rules! foo { () => {} }
1132fn main() { foo! {$0} }
1133"#,
1134 )
1135 }
1082} 1136}