Ver Fonte

docs(进件): Task 1 评审的两条 Minor - 删死 import、订正一处可被证伪的 javadoc 断言

评审(task-1-review.md)判定 Task 1 Approved、0 Critical、0 Important,余三条 Minor:
- 第 1 条:LeshouyingClientApiEnvelopeTest 的 import org.mockito.Mockito 是死的
  —— 该文件只用静态导入的 verify/when。删掉;删后 9/9 仍绿,反证它确实没被用。
- 第 2 条:postApi javadoc 写「值序列化契约与 post 逐字相同」,但 post 的 javadoc 并未
  被逐字复述(它还多写「14 位」「无小数点无千分位」「拼 map 之前就格式化好」)。
  实质等价、措辞不同 ⇒ 「逐字相同」是一句十秒内可被证伪的话,改成「相同(见其 javadoc,
  那里的条目更细)」。
- 第 3 条:两条腿都没被任何测试钉住「出站 trace 日志过 maskByKey」(全局约束 2)。
  属**既存**盲区,本 diff 未新增任何日志、不是回归,brief 也未要求 ⇒ 不在此修,
  裁定与理由记入台账 progress.md。

注:评审报告给第 2 条的行号是 :52-56,实测该断言在 :104;行号以实测为准。
alphaH há 3 semanas atrás
pai
commit
74a80043b3

+ 1 - 1
java/src/main/java/com/payment/platform/module/payment/leshouying/client/LeshouyingClient.java

@@ -101,7 +101,7 @@ public class LeshouyingClient {
      * 所以在进件这条腿上,能被 `catch` 接住的失败和「上游拒绝」是同一件事,不存在 v3 那种
      * 「抛异常 = 通信层失败 / 正常返回 = 业务结果」的二分。
      *
-     * <p>值序列化契约与 {@link #post} 逐字相同(见其 javadoc):进 map 的每个值都必须是
+     * <p>值序列化契约与 {@link #post} 相同(见其 javadoc,那里的条目更细):进 map 的每个值都必须是
      * **最终字符串形状** —— 日期 `yyyyMMddHHmmss`、金额「分」的字符串,不得放
      * `LocalDateTime` / `Date`。
      *

+ 0 - 1
java/src/test/java/com/payment/platform/module/payment/leshouying/client/LeshouyingClientApiEnvelopeTest.java

@@ -6,7 +6,6 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.ArgumentCaptor;
 import org.mockito.Mock;
-import org.mockito.Mockito;
 import org.mockito.junit.jupiter.MockitoExtension;
 import org.springframework.http.HttpEntity;
 import org.springframework.http.HttpStatus;