chore: use ERC20 subscriptions instead of eventually to check balances

[2-towns]

This PR uses ERC20 subscriptions instead of eventually to check balances.

It catches HttpConnectionError when using eventually, to prevent failures on the first HTTP request error.

It also enables logs for repair from local and remote store to help identify the cause when the test fails in CI.

[dryajov]

Just a note that the event is going to fire only once, unless it's reset, so if thats what you want then fine.

[emizzle]

This feels flakey, and is difficult to understand why we need to wait for 6 Transfer events without spending the time looking through the code or logs. Maybe we could instead inspect the Transfer event for the expected values (receiver address, and value), or after each transfer event, check our conditions?

proc onTransfer(eventResult: ?!Transfer) =
  assert not eventResult.isErr
  let rewardReceived = 
    (await token.balanceOf(hostAccount)) - startBalanceHost >=
    (duration - 5 * 60).u256 * pricePerSlotPerSecond * ecNodes.u256
  let clientRefunded = ((await token.balanceOf(clientAccount)) - clientBalanceBeforeFinished > 0)
  if rewardReceived and clientRefunded:
    transferEvent.fire()

[2-towns]

after each transfer event, check our conditions?

Yeah, that's exactly what I just pushed. After each transfer event, I created an async event that triggers when the balances exactly match the expected amounts, instead of just checking if the value is greater than expected. The test now waits for those async events to complete successfully.