From d99bdf981dbee7e038e4a8e262504c1a15047c38 Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Thu, 4 Sep 2025 16:29:57 +0300 Subject: tests: add more table-driven cases and negative provider tests; update report --- internal/lsp/build_prompts_table_test.go | 16 ++++++++++++++ internal/lsp/compute_textedit_table_test.go | 33 +++++++++++++++++++++++++++++ 2 files changed, 49 insertions(+) create mode 100644 internal/lsp/build_prompts_table_test.go create mode 100644 internal/lsp/compute_textedit_table_test.go (limited to 'internal/lsp') diff --git a/internal/lsp/build_prompts_table_test.go b/internal/lsp/build_prompts_table_test.go new file mode 100644 index 0000000..b0092e2 --- /dev/null +++ b/internal/lsp/build_prompts_table_test.go @@ -0,0 +1,16 @@ +package lsp + +import "testing" + +func TestBuildPrompts_Table(t *testing.T) { + p := CompletionParams{TextDocument: TextDocumentIdentifier{URI: "file:///x.go"}, Position: Position{Line:5, Character:7}} + cases := []struct{ name string; inParams bool }{ + {"generic", false}, + {"in_params", true}, + } + for _, c := range cases { + sys, user := buildPrompts(c.inParams, p, "above", "current", "below", "func ctx") + if sys == "" || user == "" { t.Fatalf("%s: prompts empty", c.name) } + } +} + diff --git a/internal/lsp/compute_textedit_table_test.go b/internal/lsp/compute_textedit_table_test.go new file mode 100644 index 0000000..d82e91d --- /dev/null +++ b/internal/lsp/compute_textedit_table_test.go @@ -0,0 +1,33 @@ +package lsp + +import "testing" + +func TestComputeTextEditAndFilter_Table(t *testing.T) { + cases := []struct{ + name string + inParams bool + current string + pos Position + cleaned string + }{ + {"ident_replace", false, "ab cd", Position{Line:1, Character:4}, "X"}, + {"params_inside", true, "func add(a int, b string)", Position{Line:0, Character:15}, "c bool"}, + {"params_at_close", true, "func add(a int)", Position{Line:0, Character:len("func add(a int)")}, "b string"}, + } + for _, c := range cases { + te, filter := computeTextEditAndFilter(c.cleaned, c.inParams, c.current, CompletionParams{Position: c.pos}) + if te == nil { + t.Fatalf("%s: expected edit", c.name) + } + if c.inParams && te.Range.Start.Character == 0 { + t.Fatalf("%s: expected param range (non-zero start)", c.name) + } + if filter == "" && c.current != "" { + // For ident_replace, filter may be non-empty; for params, it can be empty when replacing entire segment + } + if te.NewText != c.cleaned { + t.Fatalf("%s: newText got %q want %q", c.name, te.NewText, c.cleaned) + } + } +} + -- cgit v1.2.3