Differential D13062

[avalanche] Simplify the processor test item providers by removing addToReconcile specialization
ClosedPublic
Actions

Authored by Fabien on Jan 25 2023, 14:53.

Details

Reviewers

sdulfari

Group Reviewers

Restricted Project

Commits

rABCde5f069c9564: [avalanche] Simplify the processor test item providers by removing…

Summary

Similar to D13061, the addToReconcile specialization is no longer needed and can be removed. There is no change in behavior.

Depends on D13061.

Test Plan

ninja check-avalanche

Diff Detail

Repository

rABC Bitcoin ABC

Branch

avalanche_simplify_processor_test_addtoreconcile

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 21871
Build 43377: Build Diff	lint-circular-dependencies · build-debug · build-without-wallet · build-diff · build-clang-tidy · build-clang
Build 43376: arc lint + arc unit

Event Timeline

Fabien created this revision.Jan 25 2023, 14:53

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 25 2023, 14:53

Fabien requested review of this revision.Jan 25 2023, 14:53

Harbormaster completed remote builds in B21871: Diff 37726.Jan 25 2023, 15:06

sdulfari accepted this revision.Jan 25 2023, 16:44

This revision is now accepted and ready to land.Jan 25 2023, 16:44

Closed by commit rABCde5f069c9564: [avalanche] Simplify the processor test item providers by removing… (authored by Fabien). · Explain WhyJan 25 2023, 19:48

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABCde5f069c9564: [avalanche] Simplify the processor test item providers by removing….

Revision Contents
Changeset List

Path

Size

src/

avalanche/

test/

processor_tests.cpp

52 lines

Diff 37726

View Options

src/avalanche/test/processor_tests.cpp

[avalanche] Simplify the processor test item providers by removing addToReconcile specializationClosedPublicActions