@@ -27,7 +27,7 @@ func TestConfig_LoadFromReaders(t *testing.T) {
27func TestConfig_setDefaults(t *testing.T) {
27func TestConfig_setDefaults(t *testing.T) {
28 cfg := &Config{}
28 cfg := &Config{}
30− setDefaults("/tmp", cfg)
30+ cfg.setDefaults("/tmp")
32 assert.NotNil(t, cfg.Options)
32 assert.NotNil(t, cfg.Options)
33 assert.NotNil(t, cfg.Options.TUI)
33 assert.NotNil(t, cfg.Options.TUI)
@@ -56,12 +56,12 @@ func TestConfig_configureProviders(t *testing.T) {
58 cfg := &Config{}
58 cfg := &Config{}
59− setDefaults("/tmp", cfg)
59+ cfg.setDefaults("/tmp")
60 env := env.NewFromMap(map[string]string{
60 env := env.NewFromMap(map[string]string{
61 "OPENAI_API_KEY": "test-key",
61 "OPENAI_API_KEY": "test-key",
63 resolver := NewEnvironmentVariableResolver(env)
63 resolver := NewEnvironmentVariableResolver(env)
64− err := configureProviders(cfg, env, resolver, knownProviders)
64+ err := cfg.configureProviders(env, resolver, knownProviders)
65 assert.NoError(t, err)
65 assert.NoError(t, err)
66 assert.Len(t, cfg.Providers, 1)
66 assert.Len(t, cfg.Providers, 1)
@@ -98,12 +98,13 @@ func TestConfig_configureProvidersWithOverride(t *testing.T) {
101− setDefaults("/tmp", cfg)
101+ cfg.setDefaults("/tmp")
102 env := env.NewFromMap(map[string]string{
103 env := env.NewFromMap(map[string]string{
103 "OPENAI_API_KEY": "test-key",
104 "OPENAI_API_KEY": "test-key",
105 resolver := NewEnvironmentVariableResolver(env)
106 resolver := NewEnvironmentVariableResolver(env)
106− err := configureProviders(cfg, env, resolver, knownProviders)
107+ err := cfg.configureProviders(env, resolver, knownProviders)
107 assert.NoError(t, err)
108 assert.NoError(t, err)
108 assert.Len(t, cfg.Providers, 1)
109 assert.Len(t, cfg.Providers, 1)
@@ -139,12 +140,12 @@ func TestConfig_configureProvidersWithNewProvider(t *testing.T) {
142− setDefaults("/tmp", cfg)
143+ cfg.setDefaults("/tmp")
143 env := env.NewFromMap(map[string]string{
144 env := env.NewFromMap(map[string]string{
144 "OPENAI_API_KEY": "test-key",
145 "OPENAI_API_KEY": "test-key",
146 resolver := NewEnvironmentVariableResolver(env)
147 resolver := NewEnvironmentVariableResolver(env)
147− err := configureProviders(cfg, env, resolver, knownProviders)
148+ err := cfg.configureProviders(env, resolver, knownProviders)
148 assert.NoError(t, err)
149 assert.NoError(t, err)
149 // Should be to because of the env variable
150 // Should be to because of the env variable
150 assert.Len(t, cfg.Providers, 2)
151 assert.Len(t, cfg.Providers, 2)
@@ -171,13 +172,13 @@ func TestConfig_configureProvidersBedrockWithCredentials(t *testing.T) {
173 cfg := &Config{}
174 cfg := &Config{}
174− setDefaults("/tmp", cfg)
175+ cfg.setDefaults("/tmp")
175 env := env.NewFromMap(map[string]string{
176 env := env.NewFromMap(map[string]string{
176 "AWS_ACCESS_KEY_ID": "test-key-id",
177 "AWS_ACCESS_KEY_ID": "test-key-id",
177 "AWS_SECRET_ACCESS_KEY": "test-secret-key",
178 "AWS_SECRET_ACCESS_KEY": "test-secret-key",
179 resolver := NewEnvironmentVariableResolver(env)
180 resolver := NewEnvironmentVariableResolver(env)
180− err := configureProviders(cfg, env, resolver, knownProviders)
181+ err := cfg.configureProviders(env, resolver, knownProviders)
181 assert.NoError(t, err)
182 assert.NoError(t, err)
182 assert.Len(t, cfg.Providers, 1)
183 assert.Len(t, cfg.Providers, 1)
@@ -200,10 +201,10 @@ func TestConfig_configureProvidersBedrockWithoutCredentials(t *testing.T) {
202 cfg := &Config{}
203 cfg := &Config{}
203− setDefaults("/tmp", cfg)
204+ cfg.setDefaults("/tmp")
204 env := env.NewFromMap(map[string]string{})
205 env := env.NewFromMap(map[string]string{})
205 resolver := NewEnvironmentVariableResolver(env)
206 resolver := NewEnvironmentVariableResolver(env)
206− err := configureProviders(cfg, env, resolver, knownProviders)
207+ err := cfg.configureProviders(env, resolver, knownProviders)
207 assert.NoError(t, err)
208 assert.NoError(t, err)
208 // Provider should not be configured without credentials
209 // Provider should not be configured without credentials
209 assert.Len(t, cfg.Providers, 0)
210 assert.Len(t, cfg.Providers, 0)
@@ -222,13 +223,13 @@ func TestConfig_configureProvidersBedrockWithoutUnsupportedModel(t *testing.T) {
224 cfg := &Config{}
225 cfg := &Config{}
225− setDefaults("/tmp", cfg)
226+ cfg.setDefaults("/tmp")
226 env := env.NewFromMap(map[string]string{
227 env := env.NewFromMap(map[string]string{
227 "AWS_ACCESS_KEY_ID": "test-key-id",
228 "AWS_ACCESS_KEY_ID": "test-key-id",
228 "AWS_SECRET_ACCESS_KEY": "test-secret-key",
229 "AWS_SECRET_ACCESS_KEY": "test-secret-key",
230 resolver := NewEnvironmentVariableResolver(env)
231 resolver := NewEnvironmentVariableResolver(env)
231− err := configureProviders(cfg, env, resolver, knownProviders)
232+ err := cfg.configureProviders(env, resolver, knownProviders)
232 assert.Error(t, err)
233 assert.Error(t, err)
@@ -245,14 +246,14 @@ func TestConfig_configureProvidersVertexAIWithCredentials(t *testing.T) {
247 cfg := &Config{}
248 cfg := &Config{}
248− setDefaults("/tmp", cfg)
249+ cfg.setDefaults("/tmp")
249 env := env.NewFromMap(map[string]string{
250 env := env.NewFromMap(map[string]string{
250 "GOOGLE_GENAI_USE_VERTEXAI": "true",
251 "GOOGLE_GENAI_USE_VERTEXAI": "true",
251 "GOOGLE_CLOUD_PROJECT": "test-project",
252 "GOOGLE_CLOUD_PROJECT": "test-project",
252 "GOOGLE_CLOUD_LOCATION": "us-central1",
253 "GOOGLE_CLOUD_LOCATION": "us-central1",
254 resolver := NewEnvironmentVariableResolver(env)
255 resolver := NewEnvironmentVariableResolver(env)
255− err := configureProviders(cfg, env, resolver, knownProviders)
256+ err := cfg.configureProviders(env, resolver, knownProviders)
256 assert.NoError(t, err)
257 assert.NoError(t, err)
257 assert.Len(t, cfg.Providers, 1)
258 assert.Len(t, cfg.Providers, 1)
@@ -277,14 +278,14 @@ func TestConfig_configureProvidersVertexAIWithoutCredentials(t *testing.T) {
279 cfg := &Config{}
280 cfg := &Config{}
280− setDefaults("/tmp", cfg)
281+ cfg.setDefaults("/tmp")
281 env := env.NewFromMap(map[string]string{
282 env := env.NewFromMap(map[string]string{
282 "GOOGLE_GENAI_USE_VERTEXAI": "false",
283 "GOOGLE_GENAI_USE_VERTEXAI": "false",
283 "GOOGLE_CLOUD_PROJECT": "test-project",
284 "GOOGLE_CLOUD_PROJECT": "test-project",
284 "GOOGLE_CLOUD_LOCATION": "us-central1",
285 "GOOGLE_CLOUD_LOCATION": "us-central1",
286 resolver := NewEnvironmentVariableResolver(env)
287 resolver := NewEnvironmentVariableResolver(env)
287− err := configureProviders(cfg, env, resolver, knownProviders)
288+ err := cfg.configureProviders(env, resolver, knownProviders)
288 assert.NoError(t, err)
289 assert.NoError(t, err)
289 // Provider should not be configured without proper credentials
290 // Provider should not be configured without proper credentials
290 assert.Len(t, cfg.Providers, 0)
291 assert.Len(t, cfg.Providers, 0)
@@ -303,13 +304,13 @@ func TestConfig_configureProvidersVertexAIMissingProject(t *testing.T) {
305 cfg := &Config{}
306 cfg := &Config{}
306− setDefaults("/tmp", cfg)
307+ cfg.setDefaults("/tmp")
307 env := env.NewFromMap(map[string]string{
308 env := env.NewFromMap(map[string]string{
308 "GOOGLE_GENAI_USE_VERTEXAI": "true",
309 "GOOGLE_GENAI_USE_VERTEXAI": "true",
309 "GOOGLE_CLOUD_LOCATION": "us-central1",
310 "GOOGLE_CLOUD_LOCATION": "us-central1",
311 resolver := NewEnvironmentVariableResolver(env)
312 resolver := NewEnvironmentVariableResolver(env)
312− err := configureProviders(cfg, env, resolver, knownProviders)
313+ err := cfg.configureProviders(env, resolver, knownProviders)
313 assert.NoError(t, err)
314 assert.NoError(t, err)
314 // Provider should not be configured without project
315 // Provider should not be configured without project
315 assert.Len(t, cfg.Providers, 0)
316 assert.Len(t, cfg.Providers, 0)