Differential D18208

[avalanche] Make the abc_p2p_avalanche_transaction_finalization.py test cases independant from each other
ClosedPublic
Actions

Authored by Fabien on Jun 5 2025, 12:49.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABCbb983fec074d: [avalanche] Make the abc_p2p_avalanche_transaction_finalization.py test cases…

Summary

This makes the tests easier to reason about. The block full and min tx fee cases have huge impact on the node behavior due to the options they use, so it's better to restart between these cases. This makes the test run slower unfortunately, but it also makes it possible to split it into several files if needed.

Test Plan

./test/functional/test_runner.py abc_p2p_avalanche_transaction_finalization

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Jun 5 2025, 12:49

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 5 2025, 12:49

Fabien requested review of this revision.Jun 5 2025, 12:49

Harbormaster completed remote builds in B33547: Diff 54360.Jun 5 2025, 13:12

PiRK accepted this revision.Jun 5 2025, 15:41

This revision is now accepted and ready to land.Jun 5 2025, 15:41

Closed by commit rABCbb983fec074d: [avalanche] Make the abc_p2p_avalanche_transaction_finalization.py test cases… (authored by Fabien). · Explain WhyJun 5 2025, 18:18

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCbb983fec074d: [avalanche] Make the abc_p2p_avalanche_transaction_finalization.py test cases….

Revision Contents
Changeset List

Path

Size

test/

functional/

abc_p2p_avalanche_transaction_finalization.py

45 lines

Diff 54366

View Options

test/functional/abc_p2p_avalanche_transaction_finalization.py

[avalanche] Make the abc_p2p_avalanche_transaction_finalization.py test cases independant from each otherClosedPublicActions