Skip to content

test-fixture-consolidation #13

Description

@wlvh

背景

PASS completion correctness hotfix 为 Shadow Case A 增加了正式 PR body、逐 PASS 后置条件和 owned-doc readiness 回归。专项测试因此在本次明确豁免下达到 2050 行。

本 issue 必须在 Shadow Case A 通过后、继续 Shadow Case B/C 前完成。

目标

只通过合并重复 fixture、setup 和断言驱动数据,把 tests/test_workflow_sync_skill.py 重新压回 1800 行以内。

硬边界

  • 不删除任何现有独立语义断言或回归不变量。
  • 不降低 sentinel、scope、PASS body completion、owned-doc readiness、pin、seal 或 SUBMIT 的验证粒度。
  • 不修改生产行为、result/run schema 或 workflow 协议。
  • 不加入新的 threat-model 场景。
  • 不把代码压成长行规避行数预算。

建议合并点

  • 统一临时 upstream/target repo 的构造与 PREPARE/PASS_1 连续重试 setup。
  • 以数据表驱动合法/损坏 sentinel body 场景。
  • 合并真实 sync 全序列中重复的 body row 填充和 state/result 读取。
  • 保留真实 skeleton、完整 reconcile 表、全部 PASS execution rows 和未来 PASS 占位行。

验收

  • tests/test_workflow_sync_skill.py 不超过 1800 行。
  • 重构前后的独立语义断言逐项对账,无删除、合并丢义或弱化。
  • 全量 pytest、无真实 gh PATH 的专项与全量、scoped flake8、git diff --check、两个 Skill quick validation 全部通过。
  • PR 报告精确新增/删除行数,并说明每个 fixture 合并映射。

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions