Ryan Ofsky 99b3af78bd
Merge bitcoin/bitcoin#28044: test: indexes, fix on error infinite loop
89ba8905f5c68ae29412f9c4010314c5a113c234 test: indexes, fix on error infinite loop (furszy)

Pull request description:

  Coming from https://github.com/bitcoin/bitcoin/pull/28036#issuecomment-1623813703, I thought that we were going to fix it there but seems that got merged without it for some reason.

  As index sync failures trigger a shutdown request without notifying `BaseIndex::BlockUntilSyncedToCurrentChain` in any way, we also need to check whether a shutdown was requested or not inside 'IndexWaitSynced'.

  Otherwise, any error inside the index sync process will hang the test forever.

ACKs for top commit:
  MarcoFalke:
    lgtm ACK 89ba8905f5c68ae29412f9c4010314c5a113c234
  jamesob:
    ACK 89ba890
  ryanofsky:
    Code review ACK 89ba8905f5c68ae29412f9c4010314c5a113c234. Just comment update since last review

Tree-SHA512: 1f6daf34e51d3fbc802799bfa4ac0ef0d8f774db5f9e2f5d35df18a77679778475c94efc3da1fb723ebaf3583e4075e4a5cbe4a5104ad0c50e2b32076e247b29
2023-07-11 12:30:56 -04:00
..
2023-06-29 16:14:07 -06:00
2023-05-26 15:47:55 -07:00
2023-03-23 12:55:18 +01:00
2022-12-24 23:49:50 +00:00
2023-06-16 10:38:19 +01:00
2023-01-05 09:05:14 +00:00
2023-06-16 10:38:19 +01:00
2023-03-23 19:38:38 +01:00
2023-03-23 19:38:38 +01:00
2023-01-26 10:44:05 +01:00
2023-03-23 12:55:18 +01:00
2023-03-23 12:55:18 +01:00
2022-12-24 23:49:50 +00:00
2023-03-23 12:55:18 +01:00
2023-06-16 10:38:19 +01:00
2023-06-23 18:03:06 -03:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2022-12-24 23:49:50 +00:00
2023-01-12 13:42:44 +00:00
2023-06-16 10:38:19 +01:00
2023-06-30 12:09:45 +02:00
2023-01-25 09:33:26 +01:00
2023-06-16 10:38:19 +01:00
2023-03-23 12:55:18 +01:00