Skip to content

test: default compact getblocktxn for a missing extra tx - #476

Merged
reardencode merged 2 commits into
masterfrom
test/p2p-getblocktxn
Sep 13, 2026
Merged

test: default compact getblocktxn for a missing extra tx#476
reardencode merged 2 commits into
masterfrom
test/p2p-getblocktxn

Conversation

@rearden-grok

@rearden-grok rearden-grok Bot commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Summary

Testing-audit Category B: in-tree P2P client for compact getblocktxn (the missing-tx path after coinbase-only HB compact).

  • p2p_compact_getblocktxn_missing_extra_tx (default, 20s wall): genesis follow, relay mempool on the follower, 2-tx compact (coinbase prefilled, extra not in mempool) on the live inbound writer. Asserts seed inbound getblocktxn bytes. Fake prevout — no 100-block pad.
  • Kept: getblocktxn-depth / catch-up compact guts. Handshake feeler/self/obsolete needles and getaddr_cache_* stay.

Test plan

  • cargo test -p rbitcoin-test --test integration_multinode p2p_compact_getblocktxn_missing_extra_tx
  • cargo test -p rbitcoin-test --test integration_multinode p2p_hb_compact_tip_follow
  • cargo test -p rbitcoin-test --test integration_multinode p2p_orphan_child_getdatas_parent
  • Required CI green (fmt, deny, clippy, ast-grep, test, windows, macos, multinode, coverage)

Made with Cursor

rbitcoin-grok and others added 2 commits September 12, 2026 20:03
Default follow-session journey: 2-tx compact with coinbase prefilled
and the extra tx absent from mempool must GetBlockTxn. Keep depth-10
full-block serve guts.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@reardencode
reardencode merged commit 9468948 into master Sep 13, 2026
16 checks passed
@rearden-grok
rearden-grok Bot deleted the test/p2p-getblocktxn branch September 13, 2026 03:12
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