diff options
| author | Martin Zumsande <mzumsande@gmail.com> | 2021-12-14 12:28:47 +0000 |
|---|---|---|
| committer | Martin Zumsande <mzumsande@gmail.com> | 2022-01-03 22:25:45 +0100 |
| commit | ea4c9fd4ab9aaa2e8f2c2e38a75c9f05d0bfc866 (patch) | |
| tree | 5763e3974acf70c755cd983ef8f70d925c6eae11 | |
| parent | 4f1bb467b556ec93c9b8f758783fda4d050da491 (diff) | |
test: Cover eviction by timeout in addrman_evictionworks
| -rw-r--r-- | src/test/addrman_tests.cpp | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/src/test/addrman_tests.cpp b/src/test/addrman_tests.cpp index a5f40d2d22..2eb168f1c8 100644 --- a/src/test/addrman_tests.cpp +++ b/src/test/addrman_tests.cpp @@ -823,6 +823,8 @@ BOOST_AUTO_TEST_CASE(addrman_evictionworks) // Should swap 36 for 19. addrman->ResolveCollisions(); BOOST_CHECK(addrman->SelectTriedCollision().first.ToString() == "[::]:0"); + AddressPosition addr_pos{addrman->FindAddressEntry(CAddress(addr, NODE_NONE)).value()}; + BOOST_CHECK(addr_pos.tried); // If 36 was swapped for 19, then adding 36 to tried should fail because we // are attempting to add a duplicate. @@ -837,8 +839,15 @@ BOOST_AUTO_TEST_CASE(addrman_evictionworks) BOOST_CHECK(!addrman->Good(addr19)); BOOST_CHECK_EQUAL(addrman->SelectTriedCollision().first.ToString(), "250.1.1.36:0"); + // Eviction is also successful if too much time has passed since last try + SetMockTime(GetTime() + 4 * 60 *60); addrman->ResolveCollisions(); BOOST_CHECK(addrman->SelectTriedCollision().first.ToString() == "[::]:0"); + //Now 19 is in tried again, and 36 back to new + AddressPosition addr_pos19{addrman->FindAddressEntry(CAddress(addr19, NODE_NONE)).value()}; + BOOST_CHECK(addr_pos19.tried); + AddressPosition addr_pos36{addrman->FindAddressEntry(CAddress(addr, NODE_NONE)).value()}; + BOOST_CHECK(!addr_pos36.tried); } static CDataStream AddrmanToStream(const AddrMan& addrman) |