huang
1da680a790
All checks were successful
构建并部署到测试环境（无 SSH） / build-and-deploy (push) Successful in 5m18s
主要变更： - 重构分配模型：从加价模式（pricing_mode/pricing_value）改为返佣模式（base_commission + tier_commission） - 删除独立的 my_package 接口，统一到 /api/admin/packages（通过数据权限自动过滤） - 新增批量分配和批量调价功能，支持事务和性能优化 - 新增配置版本管理，订单创建时锁定返佣配置 - 新增成本价历史记录，支持审计和纠纷处理 - 新增统计缓存系统（Redis + 异步任务），优化梯度返佣计算性能 - 删除冗余的梯度佣金独立 CRUD 接口（合并到分配配置中） - 归档 3 个已完成的 OpenSpec changes 并同步 8 个新 capabilities 到 main specs 技术细节： - 数据库迁移：000026_refactor_shop_package_allocation - 新增 Store：AllocationConfigStore, PriceHistoryStore, CommissionStatsStore - 新增 Service：BatchAllocationService, BatchPricingService, CommissionStatsService - 新增异步任务：统计更新、定时同步、周期归档 - 测试覆盖：批量操作集成测试、梯度佣金 CRUD 清理验证 影响： - API 变更：删除 4 个梯度 CRUD 接口（POST/GET/PUT/DELETE /:id/tiers） - API 新增：批量分配、批量调价接口 - 数据模型：重构 shop_series_allocation 表结构 - 性能优化：批量操作使用 CreateInBatches，统计使用 Redis 缓存 相关文档： - openspec/changes/archive/2026-01-28-refactor-shop-package-allocation/ - openspec/specs/agent-available-packages/ - openspec/specs/allocation-config-versioning/ - 等 8 个新 capability specs
60 lines
2.2 KiB
Markdown
60 lines
2.2 KiB
Markdown
## Why
|
||
|
||
集成测试基础设施严重不统一，导致"单模块测试通过但全量测试失败"的问题。当前存在三种不同的测试方式：testcontainers（Docker 容器）、共享数据库 + DELETE 清理、事务隔离。这些方式混用导致测试不可靠、难以维护，且测试结果不可信。
|
||
|
||
## What Changes
|
||
|
||
- **BREAKING** 移除所有 testcontainers 依赖，统一使用事务隔离模式
|
||
- 重构所有集成测试，使用 `testutils.NewTestTransaction` 替代直接数据库连接
|
||
- 删除所有 `DELETE FROM ... WHERE xxx LIKE 'test%'` 的手动清理代码
|
||
- 统一测试环境配置，从 `testutils/db.go` 集中管理，消除硬编码 DSN
|
||
- 增强 `testutils` 包，支持集成测试的完整生命周期管理
|
||
- 创建统一的测试环境设置模式，提供标准化的 `setupXxxTestEnv` 函数模板
|
||
|
||
## Capabilities
|
||
|
||
### New Capabilities
|
||
|
||
- `test-infrastructure`: 统一的测试基础设施规范，包括事务隔离、Redis 清理、环境配置的标准化模式
|
||
|
||
### Modified Capabilities
|
||
|
||
<!-- 无现有 spec 需要修改 -->
|
||
|
||
## Impact
|
||
|
||
### 受影响的代码
|
||
|
||
| 目录/文件 | 影响 |
|
||
|-----------|------|
|
||
| `tests/integration/*.go` | 15-20 个测试文件需要重构 |
|
||
| `tests/testutils/` | 增强现有工具函数 |
|
||
| `go.mod` | 移除 testcontainers 依赖 |
|
||
|
||
### 具体测试文件
|
||
|
||
需要重构的测试文件（使用不统一方式）：
|
||
- `role_test.go` - 使用 testcontainers
|
||
- `shop_management_test.go` - 使用 DELETE 清理
|
||
- `shop_account_management_test.go` - 使用 DELETE 清理
|
||
- `account_test.go` - 需要检查
|
||
- `permission_test.go` - 需要检查
|
||
- `carrier_test.go` - 需要检查
|
||
- `package_test.go` - 需要检查
|
||
- 其他集成测试文件
|
||
|
||
### 预期收益
|
||
|
||
| 指标 | 改进前 | 改进后 |
|
||
|------|--------|--------|
|
||
| 测试可靠性 | 不稳定，偶发失败 | 稳定，100% 可重复 |
|
||
| 测试隔离 | 部分隔离 | 完全隔离 |
|
||
| Docker 依赖 | 必须安装 Docker | 不需要 |
|
||
| 测试速度 | 慢（容器启动） | 快（事务回滚） |
|
||
| 维护成本 | 高（三种模式） | 低（一种模式） |
|
||
|
||
### 风险
|
||
|
||
- 重构范围较大，可能引入新问题
|
||
- 需要确保所有测试在重构后仍能正确验证业务逻辑
|