fix eventually symbol resolution

Because `eventuallySafe` calls the symbol `eventually`, it should be declared before `proc eventually` is declared to avoid ambiguous symbol lookups.
This commit is contained in:
Eric 2025-03-24 19:41:56 +11:00
parent 75047f52be
commit b060c8d520
No known key found for this signature in database
4 changed files with 5 additions and 8 deletions

View File

@ -2,12 +2,12 @@ import pkg/asynctest/chronos/unittest2
export unittest2 except eventually
template eventually*(expression: untyped, timeout = 5000, pollInterval = 10): bool =
## Fast defaults, do not use with HTTP connections!
eventually(expression, timeout, pollInterval)
template eventuallySafe*(
expression: untyped, timeout = 5000, pollInterval = 1000
): bool =
## More sane defaults, for use with HTTP connections
eventually(expression, timeout = timeout, pollInterval = pollInterval)
eventually(expression, timeout, pollInterval)
template eventually*(expression: untyped, timeout = 5000, pollInterval = 10): bool =
## Fast defaults, do not use with HTTP connections!
eventually(expression, timeout, pollInterval)

View File

@ -6,7 +6,6 @@ import ../contracts/deployment
import ./marketplacesuite
import ./twonodes
import ./nodeconfigs
from ../helpers import eventuallySafe
marketplacesuite "Marketplace":
let marketplaceConfig = NodeConfigs(

View File

@ -7,7 +7,6 @@ import ../codex/helpers
import ../examples
import ./marketplacesuite
import ./nodeconfigs
from ../helpers import eventuallySafe
export logutils

View File

@ -8,7 +8,6 @@ import ../contracts/time
import ./codexconfig
import ./codexclient
import ./nodeconfigs
from ../helpers import eventuallySafe
proc findItem[T](items: seq[T], item: T): ?!T =
for tmp in items: