test: Fix generate calls and comments in feature_segwit #24535

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2203-test_fix_seg-🏌 changing 1 files +9 −11

MarcoFalke commented at 3:43 PM on March 11, 2022: member

There are currently a few incorrect comments: Block 432 is mined "twice" (The second one is actually 433).

There isn't any need to mine this many blocks anyway, so remove a few calls.
fanquake added the label Tests on Mar 11, 2022
DrahtBot commented at 2:31 AM on March 17, 2022: member



The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Conflicts

No conflicts as of last run.
aureleoules commented at 3:38 AM on March 17, 2022: member

tACK fa4126898de75030f734215f20eff0f9e6bab7d3 (test/functional/feature_segwit.py) Tested on NixOS 22.05 64 bits.

Verified that the number of blocks mined is reduced and do not alter the tests.
DrahtBot added the label Needs rebase on Mar 18, 2022
test: Fix generate calls and comments in feature_segwit fa8593f898
MarcoFalke force-pushed on Mar 18, 2022
DrahtBot removed the label Needs rebase on Mar 18, 2022
theStack approved
theStack commented at 6:38 PM on March 19, 2022: member

Tested ACK fa8593f89892ddb09b64a7740087b725b3d7b5eb
MarcoFalke merged this on Mar 22, 2022
MarcoFalke closed this on Mar 22, 2022
MarcoFalke deleted the branch on Mar 22, 2022
sidhujag referenced this in commit acb82bff46 on Mar 23, 2022
DrahtBot locked this on Mar 22, 2023

Contributors

Labels