[avalanche] Reduce scope of block tips being checked in avalanche voting test
ClosedPublic
Actions

Authored by sdulfari on Feb 21 2023, 19:55.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC5fd1489177ac: [avalanche] Reduce scope of block tips being checked in avalanche voting test

Summary

This patch makes more explicit which tip is being tested for avalanche acceptance. Keeping track of the outer-scoped hash_tip variables leads to headaches when making changes to this file.

Test Plan

ninja && ./test/functional/test_runner.py abc_p2p_avalanche_voting

Diff Detail

Repository

rABC Bitcoin ABC

Branch

ava-tip-scope

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 22042
Build 43719: Build Diff	build-debug · build-without-wallet · build-diff · build-clang · build-clang-tidy
Build 43718: arc lint + arc unit

Event Timeline

sdulfari created this revision.Feb 21 2023, 19:55

Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 21 2023, 19:55

sdulfari requested review of this revision.Feb 21 2023, 19:55

Cleanup unnecessary check against getbestblockhash

Harbormaster completed remote builds in B22041: Diff 37967.Feb 21 2023, 20:11

sdulfari added a child revision: D13150: [avalanche] Add regression testing for flip flopping votes on blocks.Feb 21 2023, 20:14

Harbormaster completed remote builds in B22042: Diff 37968.Feb 21 2023, 20:20

Fabien requested changes to this revision.Feb 21 2023, 21:14

Fabien added a subscriber: Fabien.

Fabien added inline comments.

test/functional/abc_p2p_avalanche_voting.py
213	That is super confusing ! Please don't return the function but simply use a lambda here: `self.wait_until(lambda: has_finalized_tip(fork_tip))` and remove the inner check() altogether
227	Nit: using getchaintips to actually check the block is parked would be more accurate
232	Same, please use a lambda; this is super confusing
252	dito
262	dito