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

Branch

PR22725

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 20547
Build 40764: Build Diff	lint-circular-dependencies · build-without-wallet · build-debug · build-diff · build-clang · build-clang-tidy
Build 40763: arc lint + arc unit

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 35746

View Options

src/test/addrman_tests.cpp