Discovered while working on backports related to send_txs_and_test.
Before this patch, the test happens to pass because the check at line 270 is looking for a tx not being in the mempool of node0, when it should be checking node1 (see calls to check_for_no_ban_on_rejected_tx on node_std at line 270-273).
Details
Details
- Reviewers
deadalnix markblundeberg - Group Reviewers
Restricted Project - Commits
- rSTAGINGf694f58845f6: Fix incorrect node being checked in segwit recovery test
rABCf694f58845f6: Fix incorrect node being checked in segwit recovery test
test_runner.py abc-segwit-recovery
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- segwit-recovery-test
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 7121 Build 12288: Bitcoin ABC Buildbot (legacy) Build 12287: arc lint + arc unit