xiaoxia/xiaoxia-saas
1
0
Fork 0
Code Issues 9 Pull Requests 2 Actions Packages Projects Releases 7 Wiki Activity

fix: 配额单元测试同步添加 pro 套餐 #1265

Merged
xiaoxia merged 1 commits from fix/quota-test-wave78-pro-sync into develop 2026-08-07 18:00:12 +08:00
Conversation 2 Commits 1 Files Changed 1
+2 -2
xiaoxia commented 2026-08-07 17:12:08 +08:00
Owner
Copy Link
Copy Source

PR #1262 添加了 pro 套餐到 QUOTA_TIERS，但遗漏了 test_wave78_duplication_domain.py 中的两处断言，导致 develop CI Unit Tests 失败。

修复：

  • 第 465 行：QUOTA_TIERS.keys() 断言添加 "pro"
  • 第 691 行：list_tiers 断言添加 "pro"

本地验证：80 passed

PR #1262 添加了 pro 套餐到 QUOTA_TIERS，但遗漏了 test_wave78_duplication_domain.py 中的两处断言，导致 develop CI Unit Tests 失败。 修复： - 第 465 行：QUOTA_TIERS.keys() 断言添加 "pro" - 第 691 行：list_tiers 断言添加 "pro" 本地验证：80 passed
xiaoxia added 1 commit 2026-08-07 17:47:53 +08:00
fix: 配额单元测试同步添加 pro 套餐
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 44s
Details
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 42s
Details
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
Details
CI/CD Pipeline / Frontend Lint (pull_request) Has been skipped
Details
CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been skipped
Details
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 1m3s
Details
CI/CD Pipeline / PR Build Web Image (pull_request) Has been skipped
Details
AI Code Review / AI Code Review (pull_request) Successful in 1m8s
Details
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
Details
Preview Deploy / Deploy Preview Environment (pull_request) Failing after 40s
Details
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
Details
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
Details
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m23s
Details
PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m40s
Details
CI/CD Pipeline / Unit Tests (pull_request) Successful in 3m45s
Details
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 4m56s
Details
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
Details
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
Details
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
Details
CI/CD Pipeline / Integration Tests (pull_request) Successful in 2m16s
Details
CI/CD Pipeline / PR Build API Image (pull_request) Successful in 6m21s
Details
CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 6m17s
Details
CI/CD Pipeline / CI Gate (pull_request) Successful in 6s
Details
ACR Cleanup / ACR Image Cleanup (pull_request_target) Successful in 47s
Details
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 1m4s
Details
700e31452c 
PR #1262 添加了 pro 套餐到 QUOTA_TIERS，但遗漏了
test_wave78_duplication_domain.py 中的两处断言，导致
develop CI Unit Tests 失败。

修复：
- 第 465 行：QUOTA_TIERS.keys() 断言添加 "pro"
- 第 691 行：list_tiers 断言添加 "pro"
xiaoxia force-pushed fix/quota-test-wave78-pro-sync from a509c1b827 to 700e31452c 2026-08-07 17:47:53 +08:00 Compare
auto-approve-bot commented 2026-08-07 17:49:19 +08:00
Collaborator
Copy Link
Copy Source

【阻塞级判定】

  • 是否存在阻塞级问题：否
  • 阻塞级问题数量：0 个

📊 审查概览

  • 整体评价：通过
  • 建议级问题数量：0 个

🔴 阻塞级问题（必须修复）

无

💡 改进建议（不阻塞合并）

无

✅ 良好实践

  • 测试断言更新准确，与新增 pro 层级的业务逻辑保持一致。
  • 使用集合 (set) 进行无序比较，避免了因字典键顺序变化导致的测试误报，逻辑严谨。

🤖 由 AI 代码审查机器人自动生成 | 2026-08-07 09:49:18 | 模型:

### 【阻塞级判定】 - 是否存在阻塞级问题：否 - 阻塞级问题数量：0 个 ### 📊 审查概览 - 整体评价：通过 - 建议级问题数量：0 个 ### 🔴 阻塞级问题（必须修复） 无 ### 💡 改进建议（不阻塞合并） 无 ### ✅ 良好实践 - 测试断言更新准确，与新增 `pro` 层级的业务逻辑保持一致。 - 使用集合 (`set`) 进行无序比较，避免了因字典键顺序变化导致的测试误报，逻辑严谨。 --- <sub>🤖 由 AI 代码审查机器人自动生成 | 2026-08-07 09:49:18 | 模型: </sub> <!-- AI_CODE_REVIEW_AUTO_COMMENT -->
xiaoxia merged commit aaca53a76a into develop 2026-08-07 18:00:12 +08:00
gitea-actions bot commented 2026-08-07 18:01:17 +08:00
Copy Link
Copy Source

🗑️ 预览环境已清理

PR #1265 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #1265 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
P0
阻塞性，必须立即处理
 P1
高优先级
 P2
中优先级
 P3
低优先级/后考虑
 architecture
架构设计
 backend
后端相关
 blocked
阻塞中
 bug
Bug修复
 ci-bug
CI本身的bug
 ci-optimization
CI优化类任务
 ci-pipeline
CI流水线/workflow相关
 cleanup
代码清理
 devops
运维/部署/CI
 documentation
文档相关
 feature
新功能
 frontend
前端相关
 question
需要讨论确认
 ready
PR CI通过，可合并
 refactor
代码重构
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
auto-approve-bot xiaoxia
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1265
Delete Branch "fix/quota-test-wave78-pro-sync"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?