introducing an alternative to `waitForBlock`

[anilhelvaci]

Context

@dckc has raised in here that waitForBlock might be racy and pointed some good implementations for an alternative:

• https://github.com/Agoric/agoric-sdk/blob/f291362e4bc62ec31552cef29ec1a5f5cfbf7bd7/packages/internal/src/utils.js#L268
• https://github.com/Agoric/agoric-sdk/blob/f291362e4bc62ec31552cef29ec1a5f5cfbf7bd7/multichain-testing/tools/sleep.ts#L62

Considerations

What do you think if we include one of the above to @agoric/synthetic-chain ? @turadg

[dckc]

I suggest that the title should be something like: waitForBlock(N) is unreliable for synchronization

As stated, any alternative would address the issue, including one that was functionally identical but refactored in some way.

[turadg]

What do you think if we include one of the above

First implement it locally in a proposal test. If it works well then we can incorporate it upstream.

any alternative would address the issue, including one that was functionally identical but refactored in some way

This would be ideal so the existing uses get automatically improved and no tests have to change.

[anilhelvaci]

Thanks, I'm clear now.