1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
|
package lsp
import (
"bytes"
"context"
"log"
"testing"
"hexai/internal/llm"
)
// countingLLM counts Chat calls; minimal implementation for tests.
type countingLLM struct{ calls int }
func (f *countingLLM) Chat(_ context.Context, _ []llm.Message, _ ...llm.RequestOption) (string, error) {
f.calls++
return "x := 1", nil
}
func (f *countingLLM) Name() string { return "fake" }
func (f *countingLLM) DefaultModel() string { return "m" }
func TestDefaultTriggerChars_DoesNotIncludeSemicolonOrQuestion(t *testing.T) {
var buf bytes.Buffer
logger := log.New(&buf, "", 0)
s := NewServer(bytes.NewBuffer(nil), &buf, logger, ServerOptions{})
has := func(ch string) bool {
for _, c := range s.triggerChars {
if c == ch { return true }
}
return false
}
if has(";") || has("?") {
t.Fatalf("default trigger chars should not include ';' or '?' got=%v", s.triggerChars)
}
}
func TestTryLLMCompletion_BusySkipsConcurrent(t *testing.T) {
s := &Server{ maxTokens: 32, triggerChars: []string{".", ":", "/", "_"} }
fake := &countingLLM{}
s.llmClient = fake
// Simulate another LLM request in flight
s.llmBusy = true
p := CompletionParams{ Position: Position{ Line: 0, Character: 4 }, TextDocument: TextDocumentIdentifier{URI: "file://x.go"} }
items, ok := s.tryLLMCompletion(p, "", "foo.", "", "", "", false, "")
if !ok {
t.Fatalf("expected ok=true when busy guard skips")
}
if len(items) != 0 {
t.Fatalf("expected zero items when busy, got %d", len(items))
}
if fake.calls != 0 {
t.Fatalf("LLM Chat should not be called when busy; calls=%d", fake.calls)
}
}
func TestTryLLMCompletion_MinPrefixSkipsEarly(t *testing.T) {
s := &Server{ maxTokens: 32, triggerChars: []string{".", ":", "/", "_"} }
fake := &countingLLM{}
s.llmClient = fake
// No trigger character -> skip regardless of prefix
p := CompletionParams{ Position: Position{ Line: 0, Character: 1 }, TextDocument: TextDocumentIdentifier{URI: "file://x.go"} }
items, ok := s.tryLLMCompletion(p, "", "a", "", "", "", false, "")
if !ok {
t.Fatalf("expected ok=true when skipped by min-prefix heuristic")
}
if len(items) != 0 {
t.Fatalf("expected zero items when not triggered")
}
if fake.calls != 0 {
t.Fatalf("LLM Chat should not be called when not triggered; calls=%d", fake.calls)
}
}
func TestTryLLMCompletion_RequiresTriggerChar(t *testing.T) {
s := &Server{ maxTokens: 32, triggerChars: []string{".", ":", "/", "_", " "} }
fake := &countingLLM{}
s.llmClient = fake
// With trigger character '.' directly before cursor -> allowed
items, ok := s.tryLLMCompletion(CompletionParams{ Position: Position{ Line: 0, Character: 1 }, TextDocument: TextDocumentIdentifier{URI: "file://x.go"} }, "", ".", "", "", "", false, "")
if !ok || len(items) == 0 || fake.calls == 0 { t.Fatalf("expected allowed with '.' trigger") }
// Without trigger -> skipped
fake.calls = 0
items, ok = s.tryLLMCompletion(CompletionParams{ Position: Position{ Line: 0, Character: 1 }, TextDocument: TextDocumentIdentifier{URI: "file://y.go"} }, "", "a", "", "", "", false, "")
if !ok || len(items) != 0 || fake.calls != 0 { t.Fatalf("expected skip without trigger; ok=%v len=%d calls=%d", ok, len(items), fake.calls) }
}
func TestTryLLMCompletion_AllowsSpaceTrigger(t *testing.T) {
s := &Server{ maxTokens: 32, triggerChars: []string{".", ":", "/", "_", " "} }
fake := &countingLLM{}
s.llmClient = fake
line := "type Matrix "
p := CompletionParams{ Position: Position{ Line: 0, Character: len(line) }, TextDocument: TextDocumentIdentifier{URI: "file://x.go"} }
items, ok := s.tryLLMCompletion(p, "", line, "", "", "", false, "")
if !ok || len(items) == 0 || fake.calls == 0 {
t.Fatalf("expected allowed with space trigger; ok=%v len=%d calls=%d", ok, len(items), fake.calls)
}
}
|
