Differential D13061

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

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

Details

Reviewers

sdulfari

Group Reviewers

Restricted Project

Commits

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

Summary

Now that we have a variant VoteItem, there is no need anymore for the specialized methods in the providers.
There is no change in behavior.

Test Plan

ninja check-avalanche

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

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

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

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

Fabien added a child revision: D13062: [avalanche] Simplify the processor test item providers by removing addToReconcile specialization.Jan 25 2023, 14:53

Harbormaster completed remote builds in B21870: Diff 37725.Jan 25 2023, 15:04

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

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

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

This revision was automatically updated to reflect the committed changes.

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

Revision Contents
Changeset List

Path

Size

src/

avalanche/

test/

processor_tests.cpp

87 lines

Diff 37732

View Options

src/avalanche/test/processor_tests.cpp

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