fix: tests using fetchMerkleRoot (#3513)

* Add waitFor for fetchMerkleRoot function in test_rln_group_manager_onchain
This commit is contained in:
Tanya S 2025-07-18 15:34:14 +02:00 committed by GitHub
parent 61171ed551
commit 2cff70d158
2 changed files with 10 additions and 13 deletions

View File

@ -135,7 +135,7 @@ suite "Onchain group manager":
(waitFor manager.init()).isOkOr: (waitFor manager.init()).isOkOr:
raiseAssert $error raiseAssert $error
let merkleRootBefore = manager.fetchMerkleRoot() let merkleRootBefore = waitFor manager.fetchMerkleRoot()
try: try:
waitFor manager.register(credentials, UserMessageLimit(20)) waitFor manager.register(credentials, UserMessageLimit(20))
@ -144,7 +144,7 @@ suite "Onchain group manager":
discard waitFor withTimeout(trackRootChanges(manager), 15.seconds) discard waitFor withTimeout(trackRootChanges(manager), 15.seconds)
let merkleRootAfter = manager.fetchMerkleRoot() let merkleRootAfter = waitFor manager.fetchMerkleRoot()
let metadataSetRes = manager.setMetadata() let metadataSetRes = manager.setMetadata()
assert metadataSetRes.isOk(), metadataSetRes.error assert metadataSetRes.isOk(), metadataSetRes.error
@ -170,7 +170,7 @@ suite "Onchain group manager":
(waitFor manager.init()).isOkOr: (waitFor manager.init()).isOkOr:
raiseAssert $error raiseAssert $error
let merkleRootBefore = manager.fetchMerkleRoot() let merkleRootBefore = waitFor manager.fetchMerkleRoot()
try: try:
for i in 0 ..< credentials.len(): for i in 0 ..< credentials.len():
@ -180,7 +180,7 @@ suite "Onchain group manager":
except Exception, CatchableError: except Exception, CatchableError:
assert false, "exception raised: " & getCurrentExceptionMsg() assert false, "exception raised: " & getCurrentExceptionMsg()
let merkleRootAfter = manager.fetchMerkleRoot() let merkleRootAfter = waitFor manager.fetchMerkleRoot()
check: check:
merkleRootBefore != merkleRootAfter merkleRootBefore != merkleRootAfter
@ -205,20 +205,16 @@ suite "Onchain group manager":
(waitFor manager.init()).isOkOr: (waitFor manager.init()).isOkOr:
raiseAssert $error raiseAssert $error
let idCommitment = generateCredentials(manager.rlnInstance).idCommitment let idCredentials = generateCredentials(manager.rlnInstance)
let merkleRootBefore = manager.fetchMerkleRoot() let merkleRootBefore = waitFor manager.fetchMerkleRoot()
try: try:
waitFor manager.register( waitFor manager.register(idCredentials, UserMessageLimit(20))
RateCommitment(
idCommitment: idCommitment, userMessageLimit: UserMessageLimit(20)
)
)
except Exception, CatchableError: except Exception, CatchableError:
assert false, assert false,
"exception raised when calling register: " & getCurrentExceptionMsg() "exception raised when calling register: " & getCurrentExceptionMsg()
let merkleRootAfter = manager.fetchMerkleRoot() let merkleRootAfter = waitFor manager.fetchMerkleRoot()
check: check:
merkleRootAfter != merkleRootBefore merkleRootAfter != merkleRootBefore

View File

@ -315,7 +315,8 @@ hence would have reachability issues.""",
.}: seq[string] .}: seq[string]
keepAlive* {. keepAlive* {.
desc: "Deprecated since >=v0.37. This param is ignored and keep alive is always active", desc:
"Deprecated since >=v0.37. This param is ignored and keep alive is always active",
defaultValue: true, defaultValue: true,
name: "keep-alive" name: "keep-alive"
.}: bool .}: bool