@@ -10,3 +10,4 @@ export function loginconst result = api.login(email, password);+ result.catch(() => clearPassword()); return result;
失败时清空密码
先读 hunk 头定位,再看这一行 + 的行为。缺测试:失败路径还没覆盖。
下面就是 PR 里那块 unified diff。默认能看见失败时清空密码;空白噪音会用假绿红盖住它;滤掉空白后,真改动重新露出来。
第 0 步先打开:终端跑 git diff,或点 PR 的 Files changed。下面就是那块 unified hunk。
@@ -10,3 +10,4 @@ export function loginconst result = api.login(email, password);+ result.catch(() => clearPassword()); return result;
失败时清空密码
先读 hunk 头定位,再看这一行 + 的行为。缺测试:失败路径还没覆盖。
原因:diff 只显示变了哪几行。下一步:看空白噪音怎样盖住真改动,或滤掉空白。
hunk 上刚走过的阅读顺序。
@@ 锚点,再看 +/- 的行为变化,不要数绿行。-w 或单独看,别掩真改动。提交前、评审中都要先选对比较对象。
git diff / --staged / main..feature → 读 hunk 头 → 盯行为 → 噪音用 -w 滤。都在看 clearPassword,阅读方式不同。
发现缺测试,评论写在这一行上,验收「失败登录会清空密码」。
以为改了很多,真正的兜底被空白盖住,漏测失败路径。
打开一份 diff,上半是缩进从 2 空格改到 4 空格,下半新增了 clearPassword。你先做什么?
术语的技术定义和行为以这些一手或权威资料为准。
和本知识点经常一起出现的概念。