Merge #250

250: Improve the suggestion for test functions r=DJMcNab a=DJMcNab

I haven't fully updated the previous commented out test - I don't know why it was commented out so some clarification would be welcome.

Co-authored-by: Daniel McNab <[email protected]>
Co-authored-by: DJMcNab <[email protected]>

Author: bors[bot]

crates/ra_analysis/src/completion/mod.rs

@@ -427,15 +427,15 @@ mod tests {
         // check_snippet_completion(r"
         //     <|>
         //     ",
-        //     r##"[CompletionItem { label: "tfn", lookup: None, snippet: Some("#[test]\nfn $1() {\n    $0\n}") }]"##,
+        //     r##"[CompletionItem { label: "Test function", lookup: None, snippet: Some("#[test]\nfn test_${1:feature}() {\n$0\n}"##,
         // );
         check_snippet_completion(r"
             #[cfg(test)]
             mod tests {
                 <|>
             }
             ",
-            r##"[CompletionItem { label: "tfn", lookup: None, snippet: Some("#[test]\nfn $1() {\n    $0\n}") },
+            r##"[CompletionItem { label: "Test function", lookup: Some("tfn"), snippet: Some("#[test]\nfn ${1:feature}() {\n$0\n}") },
                  CompletionItem { label: "pub(crate)", lookup: None, snippet: Some("pub(crate) $0") }]"##,
         );
     }

crates/ra_analysis/src/completion/reference_completion.rs

@@ -177,9 +177,15 @@ fn complete_path(
 
 fn complete_mod_item_snippets(acc: &mut Vec<CompletionItem>) {
     acc.push(CompletionItem {
-        label: "tfn".to_string(),
-        lookup: None,
-        snippet: Some("#[test]\nfn $1() {\n    $0\n}".to_string()),
+        label: "Test function".to_string(),
+        lookup: Some("tfn".to_string()),
+        snippet: Some(
+            "#[test]\n\
+             fn ${1:feature}() {\n\
+             $0\n\
+             }"
+            .to_string(),
+        ),
     });
     acc.push(CompletionItem {
         label: "pub(crate)".to_string(),