highperfocused/bitcoin
1
0
Fork 0
mirror of https://github.com/bitcoin/bitcoin.git synced 2025-11-12 15:09:59 +01:00
Code Issues Packages Projects Releases Wiki Activity

tests: Reset node context members on ~BasicTestingSetup

Browse Source 
The destruction/resetting of node context members in the tests should
roughly follow the behaviour of the Shutdown function in `init.cpp`.
This commit is contained in:
TheCharlatan
2023-10-24 08:38:06 +02:00
parent 9759af17ff
commit c1144f0076
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/test/util/setup_common.cpp
View File

@@ -144,6 +144,7 @@ BasicTestingSetup::BasicTestingSetup(const ChainType chainType, const std::vecto
BasicTestingSetup::~BasicTestingSetup() BasicTestingSetup::~BasicTestingSetup()
{ {
m_node.kernel.reset();
SetMockTime(0s); // Reset mocktime for following tests SetMockTime(0s); // Reset mocktime for following tests
LogInstance().DisconnectTestLogger(); LogInstance().DisconnectTestLogger();
fs::remove_all(m_path_root); fs::remove_all(m_path_root);
@@ -202,8 +203,9 @@ ChainTestingSetup::~ChainTestingSetup()
m_node.netgroupman.reset(); m_node.netgroupman.reset();
m_node.args = nullptr; m_node.args = nullptr;
m_node.mempool.reset(); m_node.mempool.reset();
m_node.scheduler.reset(); m_node.fee_estimator.reset();
m_node.chainman.reset(); m_node.chainman.reset();
m_node.scheduler.reset();
} }
void ChainTestingSetup::LoadVerifyActivateChainstate() void ChainTestingSetup::LoadVerifyActivateChainstate()