Merge #588

588: Fix .not postfix completion r=matklad a=flodiebold

The postfix snippets are awesome! I don't think the `.not` one is supposed to always replace the receiver by literally `!not` :smile: 

Co-authored-by: Florian Diebold <[email protected]>

1 files changed, 3 insertions, 3 deletions

diff --git a/crates/ra_ide_api/src/completion/complete_postfix.rs b/crates/ra_ide_api/src/completion/complete_postfix.rs
index cf0252a00..b5c4d2ebd 100644
--- a/crates/ra_ide_api/src/completion/complete_postfix.rs
+++ b/crates/ra_ide_api/src/completion/complete_postfix.rs
@@ -33,7 +33,7 @@ fn postfix_snippet(ctx: &CompletionContext, label: &str, snippet: &str) -> Build
 pub(super) fn complete_postfix(acc: &mut Completions, ctx: &CompletionContext) {
     if let Some(dot_receiver) = ctx.dot_receiver {
         let receiver_text = dot_receiver.syntax().text().to_string();
-        postfix_snippet(ctx, "not", "!not").add_to(acc);
+        postfix_snippet(ctx, "not", &format!("!{}", receiver_text)).add_to(acc);
         postfix_snippet(ctx, "if", &format!("if {} {{$0}}", receiver_text)).add_to(acc);
         postfix_snippet(
             ctx,
@@ -50,8 +50,8 @@ mod tests {
     use crate::completion::completion_item::CompletionKind;
     use crate::completion::completion_item::check_completion;
 
-    fn check_snippet_completion(code: &str, expected_completions: &str) {
-        check_completion(code, expected_completions, CompletionKind::Postfix);
+    fn check_snippet_completion(test_name: &str, code: &str) {
+        check_completion(test_name, code, CompletionKind::Postfix);
     }
 
     #[test]