lsp/completions: Drop period from input new text (#799)

This is not consistent with other completions Signed-off-by: Charlie Egan <charlie@styra.com>
StyraInc · Jun 5, 2024 · 89d3a7a · 89d3a7a
1 parent 5f6e238
commit 89d3a7a
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/internal/lsp/completions/providers/input.go b/internal/lsp/completions/providers/input.go
@@ -58,7 +58,7 @@ You can also experiment with input in the [Rego Playground](https://play.openpol
 						Line: params.Position.Line, Character: params.Position.Character,
 					},
 				},
-				NewText: "input.",
+				NewText: "input",
 			},
 		})
 	}

diff --git a/internal/lsp/completions/providers/input_test.go b/internal/lsp/completions/providers/input_test.go
@@ -44,4 +44,8 @@ allow if i`
 	if labels[0] != "input" {
 		t.Fatalf("Expected 'input' completion, got: %v", labels[0])
 	}
+
+	if exp, got := "input", completions[0].TextEdit.NewText; exp != got {
+		t.Fatalf("Expected '%s' as new text, got: %s", exp, got)
+	}
 }