实现统一模型 ID 格式 (provider_id/model_name),支持跨协议模型标识和 Smart Passthrough。 核心变更: - 新增 pkg/modelid 包:解析、格式化、校验统一模型 ID - 数据库迁移:models 表使用 UUID 主键 + UNIQUE(provider_id, model_name) 约束 - Repository 层:FindByProviderAndModelName、ListEnabled 方法 - Service 层:联合唯一校验、provider ID 字符集校验 - Conversion 层:ExtractModelName、RewriteRequestModelName/RewriteResponseModelName 方法 - Handler 层:统一模型 ID 路由、Smart Passthrough、Models API 本地聚合 - 新增 error-responses、unified-model-id 规范 测试覆盖: - 单元测试:modelid、conversion、handler、service、repository - 集成测试:统一模型 ID 路由、Smart Passthrough 保真性、跨协议转换 - 迁移测试:UUID 主键、UNIQUE 约束、级联删除 OpenSpec: - 归档 unified-model-id 变更到 archive/2026-04-21-unified-model-id - 同步 11 个 delta specs 到 main specs - 新增 error-responses、unified-model-id 规范文件
303 lines
8.6 KiB
Go
303 lines
8.6 KiB
Go
package repository
|
|
|
|
import (
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
"github.com/stretchr/testify/require"
|
|
"gorm.io/driver/sqlite"
|
|
"gorm.io/gorm"
|
|
|
|
"nex/backend/internal/config"
|
|
"nex/backend/internal/domain"
|
|
)
|
|
|
|
func setupTestDB(t *testing.T) *gorm.DB {
|
|
t.Helper()
|
|
dir := t.TempDir()
|
|
db, err := gorm.Open(sqlite.Open(dir+"/test.db"), &gorm.Config{})
|
|
require.NoError(t, err)
|
|
err = db.AutoMigrate(&config.Provider{}, &config.Model{}, &config.UsageStats{})
|
|
require.NoError(t, err)
|
|
// 关闭数据库连接以便 TempDir 清理
|
|
t.Cleanup(func() {
|
|
sqlDB, _ := db.DB()
|
|
if sqlDB != nil {
|
|
sqlDB.Close()
|
|
}
|
|
})
|
|
return db
|
|
}
|
|
|
|
// ============ ProviderRepository 测试 ============
|
|
|
|
func TestProviderRepository_Create(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewProviderRepository(db)
|
|
|
|
provider := &domain.Provider{
|
|
ID: "test-provider",
|
|
Name: "Test Provider",
|
|
APIKey: "sk-test-key",
|
|
BaseURL: "https://api.test.com",
|
|
Enabled: true,
|
|
}
|
|
err := repo.Create(provider)
|
|
require.NoError(t, err)
|
|
}
|
|
|
|
func TestProviderRepository_GetByID(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewProviderRepository(db)
|
|
|
|
provider := &domain.Provider{
|
|
ID: "test-provider", Name: "Test", APIKey: "sk-test-key", BaseURL: "https://api.test.com",
|
|
}
|
|
err := repo.Create(provider)
|
|
require.NoError(t, err)
|
|
|
|
result, err := repo.GetByID("test-provider")
|
|
require.NoError(t, err)
|
|
assert.Equal(t, "test-provider", result.ID)
|
|
assert.Equal(t, "Test", result.Name)
|
|
}
|
|
|
|
func TestProviderRepository_GetByID_NotFound(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewProviderRepository(db)
|
|
|
|
_, err := repo.GetByID("nonexistent")
|
|
assert.Error(t, err)
|
|
}
|
|
|
|
func TestProviderRepository_List(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewProviderRepository(db)
|
|
|
|
for _, id := range []string{"pA", "pB", "pC"} {
|
|
err := repo.Create(&domain.Provider{ID: id, Name: id, APIKey: "key", BaseURL: "https://test.com"})
|
|
require.NoError(t, err)
|
|
}
|
|
|
|
providers, err := repo.List()
|
|
require.NoError(t, err)
|
|
assert.Len(t, providers, 3)
|
|
}
|
|
|
|
func TestProviderRepository_Update(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewProviderRepository(db)
|
|
|
|
repo.Create(&domain.Provider{ID: "p1", Name: "Old", APIKey: "key", BaseURL: "https://old.com"})
|
|
|
|
err := repo.Update("p1", map[string]interface{}{"name": "New"})
|
|
require.NoError(t, err)
|
|
|
|
result, _ := repo.GetByID("p1")
|
|
assert.Equal(t, "New", result.Name)
|
|
}
|
|
|
|
func TestProviderRepository_Update_NotFound(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewProviderRepository(db)
|
|
|
|
err := repo.Update("nonexistent", map[string]interface{}{"name": "New"})
|
|
assert.Error(t, err)
|
|
}
|
|
|
|
func TestProviderRepository_Delete(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewProviderRepository(db)
|
|
|
|
repo.Create(&domain.Provider{ID: "p1", Name: "Test", APIKey: "key", BaseURL: "https://test.com"})
|
|
err := repo.Delete("p1")
|
|
require.NoError(t, err)
|
|
|
|
_, err = repo.GetByID("p1")
|
|
assert.Error(t, err)
|
|
}
|
|
|
|
func TestProviderRepository_Delete_NotFound(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewProviderRepository(db)
|
|
|
|
err := repo.Delete("nonexistent")
|
|
assert.Error(t, err)
|
|
}
|
|
|
|
// ============ ModelRepository 测试 ============
|
|
|
|
func TestModelRepository_Create(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewModelRepository(db)
|
|
|
|
err := repo.Create(&domain.Model{ID: "m1", ProviderID: "p1", ModelName: "gpt-4", Enabled: true})
|
|
require.NoError(t, err)
|
|
}
|
|
|
|
func TestModelRepository_GetByID(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewModelRepository(db)
|
|
|
|
repo.Create(&domain.Model{ID: "m1", ProviderID: "p1", ModelName: "gpt-4", Enabled: true})
|
|
|
|
result, err := repo.GetByID("m1")
|
|
require.NoError(t, err)
|
|
assert.Equal(t, "m1", result.ID)
|
|
assert.Equal(t, "gpt-4", result.ModelName)
|
|
}
|
|
|
|
func TestModelRepository_FindByProviderAndModelName(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewModelRepository(db)
|
|
|
|
repo.Create(&domain.Model{ID: "m1", ProviderID: "p1", ModelName: "gpt-4", Enabled: true})
|
|
|
|
result, err := repo.FindByProviderAndModelName("p1", "gpt-4")
|
|
require.NoError(t, err)
|
|
assert.Equal(t, "m1", result.ID)
|
|
assert.Equal(t, "p1", result.ProviderID)
|
|
assert.Equal(t, "gpt-4", result.ModelName)
|
|
}
|
|
|
|
func TestModelRepository_FindByProviderAndModelName_NotFound(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewModelRepository(db)
|
|
|
|
repo.Create(&domain.Model{ID: "m1", ProviderID: "p1", ModelName: "gpt-4", Enabled: true})
|
|
|
|
// Wrong provider_id
|
|
_, err := repo.FindByProviderAndModelName("p2", "gpt-4")
|
|
assert.Error(t, err)
|
|
|
|
// Wrong model_name
|
|
_, err = repo.FindByProviderAndModelName("p1", "gpt-3.5")
|
|
assert.Error(t, err)
|
|
|
|
// Both wrong
|
|
_, err = repo.FindByProviderAndModelName("p2", "claude-3")
|
|
assert.Error(t, err)
|
|
}
|
|
|
|
func TestModelRepository_List(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewModelRepository(db)
|
|
|
|
repo.Create(&domain.Model{ID: "m1", ProviderID: "p1", ModelName: "gpt-4"})
|
|
repo.Create(&domain.Model{ID: "m2", ProviderID: "p1", ModelName: "gpt-3.5"})
|
|
repo.Create(&domain.Model{ID: "m3", ProviderID: "p2", ModelName: "claude-3"})
|
|
|
|
all, err := repo.List("")
|
|
require.NoError(t, err)
|
|
assert.Len(t, all, 3)
|
|
|
|
p1Models, err := repo.List("p1")
|
|
require.NoError(t, err)
|
|
assert.Len(t, p1Models, 2)
|
|
}
|
|
|
|
func TestModelRepository_ListEnabled(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
providerRepo := NewProviderRepository(db)
|
|
modelRepo := NewModelRepository(db)
|
|
|
|
// Create two providers (both start enabled due to gorm:"default:true")
|
|
err := providerRepo.Create(&domain.Provider{
|
|
ID: "enabled-provider", Name: "Enabled Provider",
|
|
APIKey: "key1", BaseURL: "https://enabled.com", Enabled: true,
|
|
})
|
|
require.NoError(t, err)
|
|
err = providerRepo.Create(&domain.Provider{
|
|
ID: "disabled-provider", Name: "Disabled Provider",
|
|
APIKey: "key2", BaseURL: "https://disabled.com", Enabled: true,
|
|
})
|
|
require.NoError(t, err)
|
|
|
|
// Disable the second provider via Update (GORM default:true skips zero values on Create)
|
|
err = providerRepo.Update("disabled-provider", map[string]interface{}{"enabled": false})
|
|
require.NoError(t, err)
|
|
|
|
// Create models (all start enabled due to gorm:"default:true")
|
|
err = modelRepo.Create(&domain.Model{ID: "m1", ProviderID: "enabled-provider", ModelName: "gpt-4", Enabled: true})
|
|
require.NoError(t, err)
|
|
err = modelRepo.Create(&domain.Model{ID: "m2", ProviderID: "enabled-provider", ModelName: "gpt-3.5", Enabled: true})
|
|
require.NoError(t, err)
|
|
err = modelRepo.Create(&domain.Model{ID: "m3", ProviderID: "disabled-provider", ModelName: "claude-3", Enabled: true})
|
|
require.NoError(t, err)
|
|
err = modelRepo.Create(&domain.Model{ID: "m4", ProviderID: "disabled-provider", ModelName: "claude-3.5", Enabled: true})
|
|
require.NoError(t, err)
|
|
|
|
// Disable m2 via Update
|
|
err = modelRepo.Update("m2", map[string]interface{}{"enabled": false})
|
|
require.NoError(t, err)
|
|
|
|
// ListEnabled should only return models where both model and provider are enabled:
|
|
// - m1: enabled provider + enabled model -> returned
|
|
// - m2: enabled provider + disabled model -> filtered out
|
|
// - m3: disabled provider + enabled model -> filtered out
|
|
// - m4: disabled provider + enabled model -> filtered out
|
|
enabled, err := modelRepo.ListEnabled()
|
|
require.NoError(t, err)
|
|
require.Len(t, enabled, 1)
|
|
assert.Equal(t, "m1", enabled[0].ID)
|
|
assert.Equal(t, "enabled-provider", enabled[0].ProviderID)
|
|
assert.Equal(t, "gpt-4", enabled[0].ModelName)
|
|
}
|
|
|
|
func TestModelRepository_Update(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewModelRepository(db)
|
|
|
|
repo.Create(&domain.Model{ID: "m1", ProviderID: "p1", ModelName: "gpt-4", Enabled: true})
|
|
|
|
err := repo.Update("m1", map[string]interface{}{"enabled": false})
|
|
require.NoError(t, err)
|
|
|
|
result, _ := repo.GetByID("m1")
|
|
assert.False(t, result.Enabled)
|
|
}
|
|
|
|
func TestModelRepository_Delete(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewModelRepository(db)
|
|
|
|
repo.Create(&domain.Model{ID: "m1", ProviderID: "p1", ModelName: "gpt-4"})
|
|
|
|
err := repo.Delete("m1")
|
|
require.NoError(t, err)
|
|
|
|
_, err = repo.GetByID("m1")
|
|
assert.Error(t, err)
|
|
}
|
|
|
|
// ============ StatsRepository 测试 ============
|
|
|
|
func TestStatsRepository_Record(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewStatsRepository(db)
|
|
|
|
err := repo.Record("provider-1", "gpt-4")
|
|
require.NoError(t, err)
|
|
|
|
// 再次记录应递增
|
|
err = repo.Record("provider-1", "gpt-4")
|
|
require.NoError(t, err)
|
|
|
|
stats, err := repo.Query("provider-1", "", nil, nil)
|
|
require.NoError(t, err)
|
|
require.Len(t, stats, 1)
|
|
assert.Equal(t, 2, stats[0].RequestCount)
|
|
}
|
|
|
|
func TestStatsRepository_Query(t *testing.T) {
|
|
db := setupTestDB(t)
|
|
repo := NewStatsRepository(db)
|
|
|
|
repo.Record("p1", "gpt-4")
|
|
// 注意:当前 schema 只有 date 字段有唯一约束
|
|
// 所以同一 provider + model 只能有一条记录
|
|
stats, err := repo.Query("p1", "", nil, nil)
|
|
require.NoError(t, err)
|
|
assert.Len(t, stats, 1)
|
|
}
|