Skip to content

Gate full E2E suite in CI / 将完整 E2E 套件交给 CI 门禁 - #675

Open
adibarra wants to merge 1 commit into
masterfrom
chore/gate-e2e-suite
Open

Gate full E2E suite in CI / 将完整 E2E 套件交给 CI 门禁#675
adibarra wants to merge 1 commit into
masterfrom
chore/gate-e2e-suite

Conversation

@adibarra

@adibarra adibarra commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Make bun run test:e2e a local smoke suite with 83 Cypress tests.
  • Add bun run test:e2e:full for the complete local suite.
  • Keep bun run test on all unit tests plus the full E2E suite.
  • Update agent and testing guidance so CI remains the full merge gate.
  • No unit or E2E spec files were deleted.

Validation

  • bun run test:unit passed, 3,477 tests.
  • bun run typecheck passed.
  • bun run test:e2e passed, 83 tests in 36.7 seconds.
  • The full GitHub Actions E2E workflow remains unchanged and continues to run all Cypress coverage.

The local smoke suite excludes navigation.cy.ts because two overview navigation checks were flaky against the local development server. That spec remains in the full suite and CI gate.

中文说明

摘要

  • bun run test:e2e 改为运行 83 个 Cypress smoke 测试的本地快速套件。
  • 新增 bun run test:e2e:full,用于本地运行完整测试套件。
  • 保持 bun run test 运行全部 unit tests 和完整 E2E 套件。
  • 更新 agent 和测试指南,让 CI 继续作为完整的合并门禁。
  • 未删除任何 unit test 或 E2E spec 文件。

验证

  • bun run test:unit 通过,共 3,477 个测试。
  • bun run typecheck 通过。
  • bun run test:e2e 通过,83 个测试耗时 36.7 秒。
  • 完整的 GitHub Actions E2E workflow 保持不变,继续运行全部 Cypress 覆盖范围。

由于本地开发服务器上的两个 overview 导航检查存在不稳定性,本地 smoke 套件暂不包含 navigation.cy.ts。该 spec 仍保留在完整套件和 CI 合并门禁中。


Note

Low Risk
Script and documentation-only changes to the local test workflow; production code and CI E2E coverage are unchanged.

Overview
Local E2E is now a smoke gate. bun run test:e2e delegates to test:e2e:quick, which runs a fixed subset of component specs (chart-buttons, scatter-graph) and integration specs (sanity, core chart pages, zh pages, CSV export overlay, overlay-optimal-only)—about 83 tests aimed at sub-minute feedback. bun run test:e2e:full runs the previous behavior (cypress run --component && cypress run); the root workspace exposes the same script, and bun run test still runs unit tests plus the full E2E suite.

Docs and agent guidance (AGENTS.md, docs/testing.md, docs/adding-entities.md) now tell developers and @claude to iterate on smoke E2E and treat GitHub Actions as the merge gate for the full matrix; day-0 entity verification’s final gate points at test:e2e:full instead of test:e2e.

No Cypress spec files were removed in this diff; the smoke integration list simply omits specs such as navigation.cy.ts (still in full/CI per PR notes).

Reviewed by Cursor Bugbot for commit 164952e. Bugbot is set up for automated code reviews on this repo. Configure here.

中文:将完整 E2E 套件交给 CI 门禁。\n\n默认 bun run test:e2e 运行 83 个 smoke 测试,bun run test:e2e:full 保留完整本地套件。所有 unit tests 和现有 E2E specs 保持不变。
@vercel

vercel Bot commented Aug 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
inferencemax-app Ready Ready Preview Aug 5, 2026 5:48pm

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant