highperfocused/bitcoin
1
0
Fork 0
mirror of https://github.com/bitcoin/bitcoin.git synced 2025-06-23 07:13:33 +02:00
Code Issues Packages Projects Releases Wiki Activity

test: Remove AddrMan unit test that fails consistency checks

Browse Source 
Now that Size() performs internal consistency checks,
it will rightfully fail (and assert) when dealing with
a corrupted AddrMan. Therefore remove this check.
This commit is contained in:
Martin Zumsande 2023-02-01 10:14:02 -05:00
parent 8fc3bcf93d
commit 59cc66abb9
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/test/addrman_tests.cpp
View File

@ -947,8 +947,6 @@ BOOST_AUTO_TEST_CASE(load_addrman_corrupted)
} catch (const std::exception&) { } catch (const std::exception&) {
exceptionThrown = true; exceptionThrown = true;
} }
// Even though de-serialization failed addrman is not left in a clean state.
BOOST_CHECK(addrman1.Size() == 1);
BOOST_CHECK(exceptionThrown); BOOST_CHECK(exceptionThrown);
// Test that ReadFromStream fails if peers.dat is corrupt // Test that ReadFromStream fails if peers.dat is corrupt