zt: align GUI translation defaults with shared translator

1 files changed, 27 insertions, 10 deletions

diff --git a/internal/gui/app_test.go b/internal/gui/app_test.go
index c89ebee..446b8a7 100644
--- a/internal/gui/app_test.go
+++ b/internal/gui/app_test.go
@@ -6,11 +6,11 @@ import (
 	"codeberg.org/snonux/totalrecall/internal/translation"
 )
 
-func TestDefaultConfigPrefersGeminiTranslationProvider(t *testing.T) {
+func TestDefaultConfigUsesOpenAITranslationProvider(t *testing.T) {
 	config := DefaultConfig()
 
-	if config.TranslationProvider != translation.ProviderGemini {
-		t.Fatalf("DefaultConfig() translation provider = %q, want %q", config.TranslationProvider, translation.ProviderGemini)
+	if config.TranslationProvider != translation.ProviderOpenAI {
+		t.Fatalf("DefaultConfig() translation provider = %q, want %q", config.TranslationProvider, translation.ProviderOpenAI)
 	}
 }
 
@@ -25,22 +25,32 @@ func TestTranslationConfigForApp(t *testing.T) {
 		wantGoogle string
 	}{
 		{
-			name: "default to gemini when google key is available",
+			name: "default to openai when provider is unset and only openai key is available",
 			config: &Config{
+				OpenAIKey: "openai-key",
+			},
+			wantProv:   translation.ProviderOpenAI,
+			wantOpen:   "openai-key",
+			wantGoogle: "",
+		},
+		{
+			name: "default to openai when provider is unset and both keys are available",
+			config: &Config{
+				OpenAIKey:    "openai-key",
 				GoogleAPIKey: "google-key",
 			},
-			wantProv:   translation.ProviderGemini,
-			wantOpen:   "",
+			wantProv:   translation.ProviderOpenAI,
+			wantOpen:   "openai-key",
 			wantGoogle: "google-key",
 		},
 		{
-			name: "fallback to openai when only openai key is available",
+			name: "default to openai when provider is unset and only google key is available",
 			config: &Config{
-				OpenAIKey: "openai-key",
+				GoogleAPIKey: "google-key",
 			},
 			wantProv:   translation.ProviderOpenAI,
-			wantOpen:   "openai-key",
-			wantGoogle: "",
+			wantOpen:   "",
+			wantGoogle: "google-key",
 		},
 		{
 			name: "honor explicit gemini provider",
@@ -64,6 +74,13 @@ func TestTranslationConfigForApp(t *testing.T) {
 			wantOpen:   "openai-key",
 			wantGoogle: "google-key",
 		},
+		{
+			name:       "nil config still uses openai defaults",
+			config:     nil,
+			wantProv:   translation.ProviderOpenAI,
+			wantOpen:   "",
+			wantGoogle: "",
+		},
 	}
 
 	for _, tt := range tests {