Skip to content

fix(test): 来源数量断言与仓库实际(26)对齐 - #1

Open
Mosquito1123 wants to merge 1 commit into
dososo:mainfrom
Mosquito1123:fix/source-count-assertion
Open

Mosquito1123 wants to merge 1 commit into
dososo:mainfrom
Mosquito1123:fix/source-count-assertion

Conversation

@Mosquito1123

@Mosquito1123 Mosquito1123 commented Sep 26, 2026 •

Copy link
Copy Markdown

变更说明 / Summary

test_rules_validate 断言 source_count >= 30,但仓库实际为 26:rules/sources.yaml 共 26 条来源,validate_repo.py 输出 26,README.md / README.en.md 也均标注「来源 26」。该断言与其余文件同在初始发布 commit 中,应为笔误,导致 pytest 与 CI(.github/workflows/validate.yml)自 v1.0.0 起一直失败。

本 PR 将断言修正为 >= 26,pytest 变为 9 passed,与 README 的预期一致。

若本意是要求来源数 >= 30,正确做法是补充 4 个来源而非放宽断言,请告知,我可以改为提交来源。

类型 / Type

  • 规则新增或修改 / Rule add or change
  • Skill 内容 / Skill content
  • 校验工具或测试 / Tooling or tests
  • 文档 / Documentation
  • 示例 / Examples

检查项 / Checklist

  • python scripts/validate_repo.py 通过 / passes
  • pytest 通过 / passes
  • python scripts/check_links.py 通过 / passes
  • 新增规则有明确失败模式与来源等级(H1–H4)/ New rules have a clear failure mode and source tier(本 PR 不涉及 / N/A)
  • 未把项目建议冒充官方规范 / Project recommendations are not presented as official
  1. 点 Update comment

test_rules_validate 断言 source_count >= 30,但仓库实际为 26:
rules/sources.yaml 共 26 条来源,validate_repo.py 输出 26,
README.md / README.en.md 也均标注「来源 26」。

该断言与其余文件同在初始发布 commit 中,应为笔误,导致
pytest 与 CI(.github/workflows/validate.yml)自 v1.0.0 起一直失败。

修正为 >= 26 后 pytest 9 passed,与 README 的预期一致。
若本意是要求来源数 >= 30,则应改为补充来源,而非放宽断言。
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