Skip to content

test: add PR1672 forwarded local invoice regressions - #106

Open
gpBlockchain wants to merge 1 commit into
nervosnetwork:mainfrom
gpBlockchain:codex/pr1672-forwarded-local-invoice
Open

gpBlockchain wants to merge 1 commit into
nervosnetwork:mainfrom
gpBlockchain:codex/pr1672-forwarded-local-invoice

Conversation

@gpBlockchain

Copy link
Copy Markdown
Collaborator

Summary

  • 新增 Fiber PR fix: scope invoice settlement to local payments fiber#1672 的链下回归测试与对应中文评审文档。
  • 覆盖 C 兑现后 B 同哈希本地发票仍 Open、C 取消后原转发失败且不泄露原像、直达 B 的正常本地收款。
  • 复用 SharedFiberTest 拓扑,在准备、操作、等待与断言旁添加中文注释,方便逐段 review;保留 TEST-MAP 对应关系。
  • 仅提交两个目标文件,不包含原工作区其他改动、节点数据或本地报告。

Coverage and limits

  • PR1672-02 / 03 / 04 已映射;PR1672-01 仍未映射,尚缺可验证的 TLC 维护完成屏障。
  • 失败 TLC 允许以 RemoveAckConfirmed 记录保留,按原 TLC ID、InvoiceCancelled、无原像及余额不变核对失败传播。
  • 不包含并发、即时兑现、MPP 或链上分支。
  • 评审文档中的 reports 路径为既有本地调查材料,不包含在本 PR 中。

Verification

基于上游 main 4be658da51967421be9d58543fe0d5c2888e324b 的独立工作区:

  • python3.12 -m pytest test_cases/fiber/devnet/settle_invoice/test_pr1672_forwarded_local_invoice.py --collect-only -q → 3 tests collected in 0.12s,exit 0;仅收集,未执行节点测试。
  • git diff --cached --check → 无输出,exit 0。
  • 使用原工作区现有 scripts/check_test_map.py --root <PR worktree>(main 尚未包含该脚本):4 reviewed / 3 mapped;未映射 PR1672-01;无重复 ID。检查 exit 1:基线已存在 PMP-01..05 与 SETTLE-01 孤儿映射,本 PR 未修改这些文件。
  • 先前本地环境记录过 3 passed in 63.71s,但不是本 PR 基线上的重新执行结果,不据此宣称当前分支运行通过。

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