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
2.2 KiB
2.2 KiB
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
Impact
受影响的代码
| 目录/文件 | 影响 |
|---|---|
tests/integration/*.go |
15-20 个测试文件需要重构 |
tests/testutils/ |
增强现有工具函数 |
go.mod |
移除 testcontainers 依赖 |
具体测试文件
需要重构的测试文件（使用不统一方式）：
role_test.go- 使用 testcontainersshop_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 | 不需要 |
| 测试速度 | 慢（容器启动） | 快（事务回滚） |
| 维护成本 | 高（三种模式） | 低（一种模式） |
风险
- 重构范围较大，可能引入新问题
- 需要确保所有测试在重构后仍能正确验证业务逻辑