Prechádzať zdrojové kódy

fix(进件): 订正我上一轮给 R57 留下的那句结论 —— 它可被变异证伪(Task 10 自纠)

`LsyMerchantServiceTest` 里 F5 的 javadoc:上一提交(9e7f22f)把**前提**换成了实测读数,
却**原样保留了旧句的结论**「⇒ 删掉这段处理全量照样绿」。今天补跑两向变异,那句话被证伪:

  变异体:`LsyMerchantService.updateReviewStatusByMchNo` 的 `if (updated == 0)`
          改成 `if (updated == -1)`(= 让这段处理不再处理任何东西)
  读数:**全量红 3 条** —— updateAffectsZeroRows_returnsNotFound_withoutThrowing、
        everyFailBranch_logsReviewedReason、failBranchLogs_sanitizeReviewedReason
        (后两条经 failBranchLogs 遍历全分支时走到它)
  还原:变异体已逐字节还原(sha256 c4edb3a4…,CR=519 / LF=519,与改前一致)

⇒ 假话就落在我自己刚改过的那两行上。顺带订正台账 R57 的前提:「零覆盖」只对
**加这段处理之前**成立 —— 三个钉住它的用例与这段处理是**同一个提交**(5e12a66
「Task 6 评审 F1–F5(修复轮 1)… + updated==0 补覆盖」)里一起进来的;
在 `5e12a66^` 上这三个用例名各 0 命中,在 HEAD 上各 1 命中。

新句改成「加这段处理之前…那时确实没有用例钉它;现在有三条」+ 当天实测(含量法与
2026-09-23 日期锚),不再出现会随套件增长而失真的总数。

注释改动:6 增 4 删,非注释行 0 条;该文件行数 889 → 891(javadoc 段落多两行,
行尾仍是 CRLF:CR=891 / LF=891)。

回归:Tests run: 645, Failures: 0, Errors: 0, Skipped: 0(BUILD SUCCESS;需库隧道在位)。
alphaH 3 týždňov pred
rodič
commit
a9d1b8331e

+ 6 - 4
java/src/test/java/com/payment/platform/module/payment/leshouying/service/LsyMerchantServiceTest.java

@@ -703,10 +703,12 @@ class LsyMerchantServiceTest {
     /**
      * ★ {@code update} 返回 <b>0</b> ⇒ {@code NOT_FOUND},且<b>不抛</b>。
      *
-     * <p>可达窗口只有「定位到写入之间那一行没了」(并发删除 / 主键换了)。正因为窗口窄,
-     * 它此前<b>零覆盖</b>:加这段处理之前,{@code merchantMapper.update} 返回 0 的桩全文件只有一处,
-     * 且属于 {@code save} 那条路径的用例({@code save_updateBranch_zeroAffectedRows_throws404})⇒ 删掉这段处理全量照样绿。
-     * 而它是这条路径上唯一把「DB 说一行都没改到」翻译成「怎么应答上游」的地方。
+     * <p>可达窗口只有「定位到写入之间那一行没了」(并发删除 / 主键换了)。加这段处理**之前**,全文件
+     * {@code merchantMapper.update} 返回 0 的桩只有一处、且属于 {@code save} 路径的用例
+     * ({@code save_updateBranch_zeroAffectedRows_throws404})—— 那时确实没有用例钉它。**现在有三条**:
+     * 本用例 + 两个经 {@code failBranchLogs} 遍历全分支的用例(2026-09-23 实测:把 {@code updated == 0}
+     * 改成 {@code updated == -1} 跑全量,红 3 条、正是这三条)。它是这条路径上唯一把「DB 说一行都没改到」
+     * 翻译成「怎么应答上游」的地方。
      *
      * <p>为什么不抛:调用方要的是一个结果去决定怎么应答上游,而这种情况的正确答案与
      * 「找不到」<b>同向</b>(让上游重推)。抛出去会把「重推一次就好」变成一个 500。