Resolves #32841 (review)
functional test: correctly detect nonstd TRUC tx vsize in feature_taproot #32859
pull instagibbs wants to merge 1 commits into bitcoin:master from instagibbs:2025-07-fixup-truc-taproot changing 1 files +4 −1-
instagibbs commented at 3:05 PM on July 2, 2025: member
-
DrahtBot commented at 3:06 PM on July 2, 2025: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--006a51241073e994b41acfe9ec718e94-->
Code Coverage & Benchmarks
For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32859.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK darosior, w0xlt, davidgumberg, achow101 If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
<!--5faf32d7da4f0f540f40219e4f7537a3-->
-
in test/functional/feature_taproot.py:1543 in b7e137ec61 outdated
1538 | @@ -1538,7 +1539,8 @@ def test_spenders(self, node, spenders, input_counts): 1539 | is_standard_tx = ( 1540 | fail_input is None # Must be valid to be standard 1541 | and (all(utxo.spender.is_standard for utxo in input_utxos)) # All inputs must be standard 1542 | - and tx.version in TX_STANDARD_VERSIONS) # The tx version must be standard 1543 | + and tx.version in TX_STANDARD_VERSIONS # The tx version must be standard 1544 | + and not (tx.version == 3 and tx.get_vsize() > TRUC_MAX_VSIZE)) # And any additional topological standardness rules followed
maflcko commented at 3:13 PM on July 2, 2025:and not (tx.version == 3 and tx.get_vsize() > TRUC_MAX_VSIZE) # Topological standardness rules must be followed )nit: Could move the
)on the next line, to avoid having to touch the line again next time a rule is added below?
instagibbs commented at 3:24 PM on July 2, 2025:taken, thanks
maflcko approvedmaflcko commented at 3:13 PM on July 2, 2025: memberlgtm
functional test: correctly detect nonstd TRUC tx vsize in feature_taproot f0524cda39instagibbs force-pushed on Jul 2, 2025fanquake requested review from darosior on Jul 2, 2025darosior approveddarosior commented at 4:36 PM on July 2, 2025: memberutACK f0524cda3995cf65adab3d0ca8da0dee4e31c79b
w0xlt commented at 5:04 PM on July 2, 2025: contributordavidgumberg commented at 10:54 PM on July 2, 2025: contributorachow101 commented at 10:55 PM on July 2, 2025: memberACK f0524cda3995cf65adab3d0ca8da0dee4e31c79b
achow101 merged this on Jul 2, 2025achow101 closed this on Jul 2, 2025fanquake referenced this in commit d2f1c539ea on Jul 3, 2025fanquake referenced this in commit 8d8abeaa5f on Jul 3, 2025fanquake referenced this in commit 59a83fb8d1 on Jul 3, 2025glozow referenced this in commit ef380a454c on Jul 18, 2025stringintech referenced this in commit f19e7819e9 on Jul 19, 2025alexanderwiederin referenced this in commit 9997c3d993 on Jul 25, 2025alexanderwiederin referenced this in commit 8d6ebb9c4e on Jul 28, 2025alexanderwiederin referenced this in commit 533a4585b5 on Jul 28, 2025yuvicc referenced this in commit 22f55cf11d on Aug 26, 2025ajtowns referenced this in commit f135cee171 on Sep 4, 2025bug-castercv502 referenced this in commit d4bd8a1259 on Sep 28, 2025stickies-v referenced this in commit a19c56cd7c on Nov 4, 2025morozow referenced this in commit 5e203dbcb3 on May 8, 2026morozow referenced this in commit 8214c3bc7c on May 8, 2026Kino1994 referenced this in commit 1c4cda4406 on Jun 28, 2026BigcoinBGC referenced this in commit 3c5026f4c4 on Jun 30, 2026bitcoin locked this on Jul 30, 2026
github-metadata-mirror
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-08-11 12:51 UTC
This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-08-11 12:51 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me
More mirrored repositories can be found on mirror.b10c.me