feat(config): 添加配置初始化和基础URL的单元测试

- 新增 TestInitWithMissingConfig 测试无配置文件时的默认行为
- 新增 TestInitWithEnvOverrides 测试环境变量覆盖配置的功能
- 新增 TestGetBaseURL 测试基础URL生成
- 新增 TestGetBaseURLWithCustomHostPort 测试自定义主机端口
- 新增 TestGetConfigPathFallback 测试配置路径回退逻辑
- 移除旧的 TestMain，改用独立的测试函数

Author: Sisyphus Agent

oho/internal/config/config_test.go

@@ -2,18 +2,143 @@ package config
 
 import (
 	"os"
+	"path/filepath"
 	"testing"
 )
 
-func TestMain(m *testing.M) {
-	// 设置测试环境变量
-	os.Setenv("OPENCODE_SERVER_HOST", "127.0.0.1")
-	os.Setenv("OPENCODE_SERVER_PORT", "4096")
-	os.Setenv("OPENCODE_SERVER_USERNAME", "opencode")
-	os.Setenv("OPENCODE_SERVER_PASSWORD", "test")
+func TestInitWithMissingConfig(t *testing.T) {
+	// 清除所有可能影响测试的环境变量
+	os.Unsetenv("OPENCODE_SERVER_HOST")
+	os.Unsetenv("OPENCODE_SERVER_PORT")
+	os.Unsetenv("OPENCODE_SERVER_USERNAME")
+	os.Unsetenv("OPENCODE_SERVER_PASSWORD")
+
+	// 确保没有配置文件在默认路径
+	home, _ := os.UserHomeDir()
+	configPath := filepath.Join(home, ".config", "oho", "config.json")
+	os.RemoveAll(filepath.Dir(configPath))
 
 	// 初始化配置
-	_ = Init()
+	err := Init()
+	if err != nil {
+		t.Fatalf("Init() returned error: %v", err)
+	}
+
+	// 验证默认值被使用
+	cfg := Get()
+	if cfg.Host != "127.0.0.1" {
+		t.Errorf("Expected Host '127.0.0.1', got '%s'", cfg.Host)
+	}
+	if cfg.Port != 4096 {
+		t.Errorf("Expected Port 4096, got %d", cfg.Port)
+	}
+	if cfg.Username != "opencode" {
+		t.Errorf("Expected Username 'opencode', got '%s'", cfg.Username)
+	}
+	if cfg.Password != "" {
+		t.Errorf("Expected empty Password, got '%s'", cfg.Password)
+	}
+}
+
+func TestInitWithEnvOverrides(t *testing.T) {
+	// 设置环境变量
+	os.Setenv("OPENCODE_SERVER_HOST", "192.168.1.1")
+	os.Setenv("OPENCODE_SERVER_PORT", "8080")
+	os.Setenv("OPENCODE_SERVER_USERNAME", "testuser")
+	os.Setenv("OPENCODE_SERVER_PASSWORD", "testpass")
+	defer func() {
+		os.Unsetenv("OPENCODE_SERVER_HOST")
+		os.Unsetenv("OPENCODE_SERVER_PORT")
+		os.Unsetenv("OPENCODE_SERVER_USERNAME")
+		os.Unsetenv("OPENCODE_SERVER_PASSWORD")
+	}()
+
+	err := Init()
+	if err != nil {
+		t.Fatalf("Init() returned error: %v", err)
+	}
+
+	cfg := Get()
+	if cfg.Host != "192.168.1.1" {
+		t.Errorf("Expected Host '192.168.1.1', got '%s'", cfg.Host)
+	}
+	if cfg.Port != 8080 {
+		t.Errorf("Expected Port 8080, got %d", cfg.Port)
+	}
+	if cfg.Username != "testuser" {
+		t.Errorf("Expected Username 'testuser', got '%s'", cfg.Username)
+	}
+	if cfg.Password != "testpass" {
+		t.Errorf("Expected Password 'testpass', got '%s'", cfg.Password)
+	}
+}
+
+func TestGetBaseURL(t *testing.T) {
+	os.Unsetenv("OPENCODE_SERVER_HOST")
+	os.Unsetenv("OPENCODE_SERVER_PORT")
+	os.Unsetenv("OPENCODE_SERVER_USERNAME")
+	os.Unsetenv("OPENCODE_SERVER_PASSWORD")
+
+	Init()
+
+	baseURL := GetBaseURL()
+	expected := "http://127.0.0.1:4096"
+	if baseURL != expected {
+		t.Errorf("Expected BaseURL '%s', got '%s'", expected, baseURL)
+	}
+}
+
+func TestGetBaseURLWithCustomHostPort(t *testing.T) {
+	os.Setenv("OPENCODE_SERVER_HOST", "10.0.0.1")
+	os.Setenv("OPENCODE_SERVER_PORT", "3000")
+	defer func() {
+		os.Unsetenv("OPENCODE_SERVER_HOST")
+		os.Unsetenv("OPENCODE_SERVER_PORT")
+	}()
+
+	Init()
+
+	baseURL := GetBaseURL()
+	expected := "http://10.0.0.1:3000"
+	if baseURL != expected {
+		t.Errorf("Expected BaseURL '%s', got '%s'", expected, baseURL)
+	}
+}
+
+func TestGetConfigPathFallback(t *testing.T) {
+	// 测试 getConfigPath 返回有效路径
+	path := getConfigPath()
+	if path == "" {
+		t.Error("getConfigPath() returned empty string")
+	}
+
+	// 验证路径包含 oho 和 config.json
+	if !filepath.IsAbs(path) && path != filepath.Join(".", ".config", "oho", "config.json") {
+		// 如果不是绝对路径，应该是相对路径 fallback
+		expectedRelative := filepath.Join(".", ".config", "oho", "config.json")
+		if path != expectedRelative {
+			t.Errorf("Unexpected relative path: got %s, expected %s", path, expectedRelative)
+		}
+	}
+
+	// 如果是绝对路径，应该包含正确的目录结构
+	if filepath.IsAbs(path) {
+		if !contains(path, ".config") || !contains(path, "oho") || !contains(path, "config.json") {
+			t.Errorf("Path does not contain expected components: %s", path)
+		}
+	}
+}
+
+// contains 是一个简单的字符串包含检查
+func contains(s, substr string) bool {
+	return len(s) >= len(substr) && (s == substr || len(s) > 0 && containsHelper(s, substr))
+}
 
-	m.Run()
+func containsHelper(s, substr string) bool {
+	for i := 0; i <= len(s)-len(substr); i++ {
+		if s[i:i+len(substr)] == substr {
+			return true
+		}
+	}
+	return false
 }