daoqi/erqiwang_youle
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

二七王：CLAUDE.md 补记断言反向验证与 CRLF 脚本改写注意事项

Browse Source 
两条都是本轮开发中真实踩到的：
- 新增守卫断言若不做「故意破坏→看是否变红」，可能加的是空转断言；
- 仓库为 CRLF 行尾，脚本化改写用 `\n` 匹配会一行不改却静默成功，
  曾据此差点得出「测试抓不住原缺陷」的错误结论。

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
joywayer
2026-08-27 21:07:15 +08:00
co-authored by Claude Opus 5
parent af779f473a
commit cfa0409d94
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
CLAUDE.md
+2
View File
@@ -74,4 +74,6 @@ git config core.hooksPath .githooks
## 测试与 Git ## 测试与 Git
- **测试纪律**：细则见 client 05 §10 / server 04 §10。要点——测试代码可用现代语法（只跑 Node，不上线）；**正式代码禁止为测试而加逻辑/放宽校验**，禁止为过测而改正式代码或降低测试标准（skip、软化断言、吞异常等）；失败先用证据裁定「业务缺陷 vs 脚本缺陷」；正面/反面/边界用例都要覆盖。 - **测试纪律**：细则见 client 05 §10 / server 04 §10。要点——测试代码可用现代语法（只跑 Node，不上线）；**正式代码禁止为测试而加逻辑/放宽校验**，禁止为过测而改正式代码或降低测试标准（skip、软化断言、吞异常等）；失败先用证据裁定「业务缺陷 vs 脚本缺陷」；正面/反面/边界用例都要覆盖。
- **新增断言要反向验证**：写完一条守卫断言，**故意破坏它守的那处逻辑，确认它真会变红**——否则你加的可能是一条空转断言（「全绿但什么都没验」比红更有害）。
- ⚠️ **本仓库文件是 CRLF 行尾**：任何基于行尾的脚本化改写（`sed`/正则批量删改）必须按 `\r?\n` 处理。写 `\n` 会**一行都匹配不上却静默成功**——曾据此得出「测试抓不住缺陷」的错误结论。脚本改完先 `git diff --stat` 确认文件真的变了，再据其结果下判断。
- **Git 提交**（细则见 server 04 §11）：完成一个可独立成立的逻辑改动即可**自动提交、无需逐次确认**（`push` 按需）；提交信息用中文、聚焦一件事、结尾保留 `Co-Authored-By` 署名；不用 `git add -A`/`git add .`、不跳过 hooks、不做 `reset --hard`/`push --force`。 - **Git 提交**（细则见 server 04 §11）：完成一个可独立成立的逻辑改动即可**自动提交、无需逐次确认**（`push` 按需）；提交信息用中文、聚焦一件事、结尾保留 `Co-Authored-By` 署名；不用 `git add -A`/`git add .`、不跳过 hooks、不做 `reset --hard`/`push --force`。