diff options
| author | Paul Buetow <paul@buetow.org> | 2026-03-19 09:05:29 +0200 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2026-03-19 09:05:29 +0200 |
| commit | 0918aad469ac2ff5513a7131661f1106e5ec851c (patch) | |
| tree | 90ecf1245c1d98ec5e3c8ead77978206f2e61155 /internal/llm/provider_test.go | |
| parent | 31394385e72dd3a317585838ed1696076043cc60 (diff) | |
Improve actionable error guidance
Diffstat (limited to 'internal/llm/provider_test.go')
| -rw-r--r-- | internal/llm/provider_test.go | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/internal/llm/provider_test.go b/internal/llm/provider_test.go index 8ccba6e..14de7a6 100644 --- a/internal/llm/provider_test.go +++ b/internal/llm/provider_test.go @@ -1,6 +1,7 @@ package llm import ( + "strings" "testing" ) @@ -8,9 +9,13 @@ func TestNewFromConfig_DefaultsAndErrors(t *testing.T) { // Unknown provider if _, err := NewFromConfig(Config{Provider: "bogus"}, "", "", ""); err == nil { t.Fatalf("expected error for unknown provider") + } else if !strings.Contains(err.Error(), "supported providers:") { + t.Fatalf("expected supported providers hint, got %q", err.Error()) } // OpenAI missing key if _, err := NewFromConfig(Config{Provider: "openai", OpenAIModel: "g"}, "", "", ""); err == nil { t.Fatalf("expected key error") + } else if !strings.Contains(err.Error(), "OPENAI_API_KEY") || !strings.Contains(err.Error(), "HEXAI_OPENAI_API_KEY") { + t.Fatalf("expected actionable API key hint, got %q", err.Error()) } } |
