Differential D12274

[MOVEONLY] [tests] Move addrman ser/deser tests to addrman_tests.cpp
ClosedPublic
Actions

Authored by Fabien on Oct 17 2022, 12:11.

Details

Reviewers

PiRK

Group Reviewers

Restricted Project

Commits

rABC0dc0af8aa302: [MOVEONLY] [tests] Move addrman ser/deser tests to addrman_tests.cpp

Summary

Addrman serialization/deserialization tests are currently in net_tests.cpp.
Move them to addrman_tests.cpp with the rest of the addrman tests.

Backport of core#22725.

Depends on D12272.

Test Plan

ninja check

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

Fabien created this revision.Oct 17 2022, 12:11

Herald added a reviewer: Restricted Project. · View Herald TranscriptOct 17 2022, 12:11

Fabien requested review of this revision.Oct 17 2022, 12:11

teamcity edited the summary of this revision. (Show Details)Oct 17 2022, 12:11

Harbormaster completed remote builds in B20547: Diff 35746.Oct 17 2022, 12:23

Fabien added a child revision: D12275: [addrman] Merge the two Add() functions.Oct 17 2022, 13:56

PiRK accepted this revision.Oct 17 2022, 15:24

This revision is now accepted and ready to land.Oct 17 2022, 15:24

Closed by commit rABC0dc0af8aa302: [MOVEONLY] [tests] Move addrman ser/deser tests to addrman_tests.cpp (authored by Fabien). · Explain WhyOct 17 2022, 15:28

This revision was automatically updated to reflect the committed changes.

Fabien added a commit: rABC0dc0af8aa302: [MOVEONLY] [tests] Move addrman ser/deser tests to addrman_tests.cpp.

Revision Contents
Changeset List

Path

Size

src/

test/

addrman_tests.cpp

131 lines

net_tests.cpp

127 lines

Diff 35764

View Options

src/test/addrman_tests.cpp