fix(e2e): Step3→Step4 需先生成预览视频才能进入下一步 #1237

Merged
auto-approve-bot merged 1 commits from fix/e2e-step3-preview-generation into develop 2026-08-04 17:41:00 +08:00
Owner

问题

Staging E2E Tests (Run #22464/job #105180) 失败:

  • core-generation.spec.ts:204 — Step 4 heading 选择标题 不可见
  • 原因:Step3→Step4 导航需要 previewReady=true,但测试未生成预览就直接点下一步

修复

在 Step 3 中:

  1. 点击「 生成预览」按钮触发预览生成
  2. 等待「预览生成成功」文案出现(120s 超时)
  3. 再点击「下一步」进入 Step 4

影响范围

仅修改 E2E 测试代码,不影响业务逻辑。

## 问题 Staging E2E Tests (Run #22464/job #105180) 失败: - `core-generation.spec.ts:204` — Step 4 heading `选择标题` 不可见 - 原因:Step3→Step4 导航需要 `previewReady=true`,但测试未生成预览就直接点下一步 ## 修复 在 Step 3 中: 1. 点击「✨ 生成预览」按钮触发预览生成 2. 等待「预览生成成功」文案出现(120s 超时) 3. 再点击「下一步」进入 Step 4 ## 影响范围 仅修改 E2E 测试代码,不影响业务逻辑。
xiaoxia added 1 commit 2026-08-04 17:31:28 +08:00
fix(e2e): Step3→Step4需先生成预览视频
CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped
CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped
CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped
CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped
CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped
CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped
CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped
CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 1m1s
CI/CD Pipeline / Unit Tests (pull_request) Has been skipped
CI/CD Pipeline / PR Build API Image (pull_request) Has been skipped
CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m9s
CI/CD Pipeline / PR Build Worker Image (pull_request) Has been skipped
CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 1m11s
Preview Deploy / Deploy Preview Environment (pull_request) Successful in 1m38s
PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m42s
CI/CD Pipeline / Frontend Lint (pull_request) Successful in 56s
CI/CD Pipeline / Frontend Unit Tests (pull_request) Successful in 1m3s
PR Automation / Auto Approve on CI Green (pull_request) Successful in 2m8s
AI Code Review / AI Code Review (pull_request) Successful in 2m29s
CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 1m40s
CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 4m33s
CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped
CI/CD Pipeline / Integration Tests (pull_request) Has been skipped
CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped
CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped
CI/CD Pipeline / Deploy Production (pull_request) Has been skipped
CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped
CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped
CI/CD Pipeline / CI Gate (pull_request) Successful in 5s
ACR Cleanup / ACR Image Cleanup (pull_request_target) Successful in 12s
Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 17s
8481ebbedc
E2E测试在Step3直接点击下一步导致被previewReady校验拦截,
Step4标题heading不可见。修复:先点击生成预览按钮,
等待预览生成完成(120s超时)后再进入下一步。

Fixes Staging E2E Tests failure in Run #22464

🚀 预览环境已部署

项目 详情
PR号 #1237
预览链接 https://pr-1237.preview.xiaoxiajianji.com
API环境 staging

💡 预览环境使用 staging API 数据,请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后,预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1237 | | 预览链接 | [https://pr-1237.preview.xiaoxiajianji.com](https://pr-1237.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据,请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后,预览环境会自动清理。
auto-approve-bot approved these changes 2026-08-04 17:33:38 +08:00
auto-approve-bot left a comment
Collaborator

CI全绿,自动审批通过。

CI全绿,自动审批通过。
auto-approve-bot approved these changes 2026-08-04 17:33:38 +08:00
auto-approve-bot left a comment
Collaborator

CI全绿,自动审批通过。

CI全绿,自动审批通过。
Collaborator

【阻塞级判定】

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

📊 审查概览

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

🔴 阻塞级问题(必须修复)

💡 改进建议(不阻塞合并)

良好实践

  1. 增加显式等待机制:在点击“下一步”之前,增加了对“预览生成成功”状态的显式等待(toBeVisible),有效避免了因预览渲染未完成而导致的后续步骤失败或竞态条件。
  2. 合理的超时设置:针对后端渲染可能耗时较长的场景,显式设置了 120 秒(120_000)的超时时间,增强了测试在真实环境下的稳定性。
  3. 清晰的注释:代码注释准确地描述了操作意图(如“需要先生成预览视频”),提升了测试用例的可维护性。

🤖 由 AI 代码审查机器人自动生成 | 2026-08-04 09:33:58 | 模型:

### 【阻塞级判定】 - 是否存在阻塞级问题:否 - 阻塞级问题数量:0 个 ### 📊 审查概览 - 整体评价:通过 - 建议级问题数量:0 个 ### 🔴 阻塞级问题(必须修复) 无 ### 💡 改进建议(不阻塞合并) 无 ### ✅ 良好实践 1. **增加显式等待机制**:在点击“下一步”之前,增加了对“预览生成成功”状态的显式等待(`toBeVisible`),有效避免了因预览渲染未完成而导致的后续步骤失败或竞态条件。 2. **合理的超时设置**:针对后端渲染可能耗时较长的场景,显式设置了 120 秒(`120_000`)的超时时间,增强了测试在真实环境下的稳定性。 3. **清晰的注释**:代码注释准确地描述了操作意图(如“需要先生成预览视频”),提升了测试用例的可维护性。 --- <sub>🤖 由 AI 代码审查机器人自动生成 | 2026-08-04 09:33:58 | 模型: </sub> <!-- AI_CODE_REVIEW_AUTO_COMMENT -->
auto-approve-bot merged commit 6ab182737f into develop 2026-08-04 17:41:00 +08:00
auto-approve-bot deleted branch fix/e2e-step3-preview-generation 2026-08-04 17:41:00 +08:00

🗑️ 预览环境已清理

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

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

🗑️ **预览环境已清理** PR #1237 已关闭或合并,对应的预览环境已被清理。 > 如有需要,可以重新打开 PR 来重新生成预览环境。
Sign in to join this conversation.