Merge bitcoin/bitcoin#28036: test: Restore unlimited timeout in IndexWaitSynced

fabed7eb79 test: Restore unlimited timeout in IndexWaitSynced (MarcoFalke)

Pull request description:

  The timeout was unlimited before, so just restore that value for now: https://github.com/bitcoin/bitcoin/pull/27988#issuecomment-1619218007 .

  (Strictly speaking, this is a behavior change for the blockfilterindex and txindex tests, because it only restores the coinstatsindex behavior.)

ACKs for top commit:
  ajtowns:
    utACK fabed7eb79
  mzumsande:
    ACK fabed7eb79
  furszy:
    ACK fabed7eb

Tree-SHA512: 66a878be58bbe53ad8e0c23f05569dd42df688be747551fbd202ada22d20a8285714e58fa2a71664deadb070ddf86cfad88c01042ff95ed26f6b40e4a10cec0a
This commit is contained in:
fanquake 2023-07-07 09:40:22 +01:00
commit 299f17a69e
No known key found for this signature in database
GPG key ID: 2EEB9F5CC09526C1
2 changed files with 2 additions and 5 deletions

View file

@ -5,14 +5,11 @@
#include <test/util/index.h>
#include <index/base.h>
#include <util/check.h>
#include <util/time.h>
void IndexWaitSynced(BaseIndex& index)
void IndexWaitSynced(const BaseIndex& index)
{
const auto timeout{SteadyClock::now() + 120s};
while (!index.BlockUntilSyncedToCurrentChain()) {
Assert(timeout > SteadyClock::now());
UninterruptibleSleep(100ms);
}
}

View file

@ -8,6 +8,6 @@
class BaseIndex;
/** Block until the index is synced to the current chain */
void IndexWaitSynced(BaseIndex& index);
void IndexWaitSynced(const BaseIndex& index);
#endif // BITCOIN_TEST_UTIL_INDEX_H