Differential D10270

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

Authored by Fabien on Oct 6 2021, 13:41.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Maniphest Tasks

Restricted Maniphest Task

Commits

rABC8784dd4cfff1: [avalanche] Convert the poll_and_response test to use context

Summary

Similar to D10262, depends on D10262 and D10268.

Ref T1854.

Test Plan

ninja check-avalanche-processor_tests

Diff Detail

Repository

rABC Bitcoin ABC

Branch

avalanche_processor_test_context_poll_and_response

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 16910
Build 33658: Build Diff	lint-circular-dependencies · build-without-wallet · build-diff · build-debug · build-clang-tidy · build-clang
Build 33657: arc lint + arc unit

Event Timeline

Fabien created this revision.Oct 6 2021, 13:41

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 6 2021, 13:41

Fabien requested review of this revision.Oct 6 2021, 13:41

Fabien added a child revision: D10271: [avalanche] Register votes on conflicting proofs.Oct 6 2021, 13:47

Harbormaster completed remote builds in B16910: Diff 30345.Oct 6 2021, 14:04

PiRK accepted this revision.Oct 8 2021, 12:44

This revision is now accepted and ready to land.Oct 8 2021, 12:44

Closed by commit rABC8784dd4cfff1: [avalanche] Convert the poll_and_response test to use context (authored by Fabien). · Explain WhyOct 12 2021, 08:44

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC8784dd4cfff1: [avalanche] Convert the poll_and_response test to use context.

Revision Contents
Changeset List

Path

Size

src/

avalanche/

test/

processor_tests.cpp

56 lines

Diff 30345

View Options

src/avalanche/test/processor_tests.cpp

[avalanche] Convert the poll_and_response test to use contextClosedPublicActions