Merge #14101: qa: Use named args in validation acceptance tests
fa782a308dbe7bc579c122f63c1c65666fc85e91 qa: Use named args in some tests (MarcoFalke)
b4d33096734d787b0e1d754064039cbb64ce8d61 scripted-diff: Use named arguments in feature_block (MarcoFalke)
749ba35e7c9fbc21dbea27fd1be102b91313d132 scripted-diff: Pass node into p2p_segwit acceptance tests (MarcoFalke)
Pull request description:
It is confusing to use a list of arguments such as `False, False, 16, ...` where it is unclear what each of them means. Run some scripted diffs to put meaning to them.
Backport of Core PR14101
Reviewer note: mining_basic.py was skipped because that section of code has not been backported yet.
The dependency chain for backporting that section of code is complex and depends on deprecation of
the ComparisonTestFramework. It's a trivial change to patch on later, so it can be skipped here.
test_runner.py feature_block test_runner.py p2p_invalid_block
Reviewers: #bitcoin_abc, deadalnix, Fabien
Reviewed By: #bitcoin_abc, Fabien
Differential Revision: https://reviews.bitcoinabc.org/D3861