Page MenuHomePhabricator
Differential D10262

[avalanche] Convert the multi_block_register test to use the context
ClosedPublic
Actions

Authored by Fabien on Oct 6 2021, 11:01.

Details

Reviewers
PiRK
Group Reviewers
Restricted Project
Maniphest Tasks
Restricted Maniphest Task
Commits
rABCd2a6721e8e09: [avalanche] Convert the multi_block_register test to use the context
Summary

This is a follow-up to D10261.

Ref T1854.

Depends on D10261.

Test Plan
ninja check-avalanche-processor_tests

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Fabien created this revision.Oct 6 2021, 11:01
Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 6 2021, 11:01
Fabien requested review of this revision.Oct 6 2021, 11:01
Harbormaster completed remote builds in B16898: Diff 30332.Oct 6 2021, 11:11
Fabien planned changes to this revision.Oct 6 2021, 11:54
Fabien updated this revision to Diff 30337.Oct 6 2021, 12:03

Let the context build the votes in order, fix a comment nit

Fabien edited the summary of this revision. (Show Details)Oct 6 2021, 12:03
Harbormaster completed remote builds in B16903: Diff 30337.Oct 6 2021, 12:16
Fabien added a child revision: D10270: [avalanche] Convert the poll_and_response test to use context.Oct 6 2021, 13:41
PiRK accepted this revision.Oct 8 2021, 07:40
This revision is now accepted and ready to land.Oct 8 2021, 07:40
Closed by commit rABCd2a6721e8e09: [avalanche] Convert the multi_block_register test to use the context (authored by Fabien). · Explain WhyOct 12 2021, 08:41
This revision was automatically updated to reflect the committed changes.
Fabien added a commit: rABCd2a6721e8e09: [avalanche] Convert the multi_block_register test to use the context.

Revision Contents
Changeset List

PathSize
src/
avalanche/
test/
100 lines

Diff 30438

View Options

src/avalanche/test/processor_tests.cpp

Loading...