Page MenuHomePhabricator

Clean up addrman_tests.cpp
ClosedPublic

Authored by Fabien on Oct 21 2018, 16:47.

Details

Summary

Backport from core PR 10371

Test Plan
./src/test/test_bitcoin -t addrman_tests

Diff Detail

Repository
rABC Bitcoin ABC
Branch
fix_addrman
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3645
Build 5365: Bitcoin ABC Buildbot (legacy)
Build 5364: arc lint + arc unit

Event Timeline

Rebase to cleanup the stack

deadalnix requested changes to this revision.Oct 21 2018, 19:11
deadalnix added a subscriber: deadalnix.
deadalnix added inline comments.
src/test/addrman_tests.cpp
110

There is no test 4 in the original PR, there is something wrong here. Where does this 4 come from ?

123

You forgot to backport here

130

dito

This revision now requires changes to proceed.Oct 21 2018, 19:11
deadalnix requested changes to this revision.Oct 21 2018, 19:52

Almost there

src/test/addrman_tests.cpp
114 ↗(On Diff #5490)

You forgot that one.

118 ↗(On Diff #5490)

dito

120 ↗(On Diff #5490)

dito

127 ↗(On Diff #5490)

dito

This revision now requires changes to proceed.Oct 21 2018, 19:52
This revision is now accepted and ready to land.Oct 21 2018, 21:49
This revision was automatically updated to reflect the committed changes.