From 9d9aa738dbe654e84e322d630f5de140a440e0b0 Mon Sep 17 00:00:00 2001 From: Jacek Sieka Date: Wed, 29 May 2024 11:26:27 +0200 Subject: [PATCH] general state test: don't swallow asserts --- tests/test_generalstate_json.nim | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/tests/test_generalstate_json.nim b/tests/test_generalstate_json.nim index 37fa65529..26451465d 100644 --- a/tests/test_generalstate_json.nim +++ b/tests/test_generalstate_json.nim @@ -110,7 +110,15 @@ proc testFixtureIndexes(ctx: var TestCtx, testStatusIMPL: var TestStatus) = # during the next call to `getComittedStorage` db.persist() - defer: + let rc = vmState.processTransaction( + ctx.tx, sender, ctx.header, fork) + if rc.isOk: + gasUsed = rc.value + + let miner = ctx.header.coinbase + coinbaseStateClearing(vmState, miner, fork) + + block post: let obtainedHash = vmState.readOnlyStateDB.rootHash check obtainedHash == ctx.expectedHash let logEntries = vmState.getAndClearLogEntries() @@ -120,14 +128,6 @@ proc testFixtureIndexes(ctx: var TestCtx, testStatusIMPL: var TestStatus) = let success = ctx.expectedLogs == actualLogsHash and obtainedHash == ctx.expectedHash ctx.dumpDebugData(vmState, gasUsed, success) - let rc = vmState.processTransaction( - ctx.tx, sender, ctx.header, fork) - if rc.isOk: - gasUsed = rc.value - - let miner = ctx.header.coinbase - coinbaseStateClearing(vmState, miner, fork) - proc testFixture(fixtures: JsonNode, testStatusIMPL: var TestStatus, trace = false, debugMode = false) = var ctx: TestCtx