Scoru: fix/reactivate tezt refutation tests
Context
Closes #4020 (closed)
There were two issues with the tests:
- The messages were not posted at the expected levels, so the
empty_levelsin tests' config were broken - The failing ticks were broken in the node by !6478 (merged)
- The fuel mechanism in the rollup node as well?
There is still a problem about the publish commitment's gas in the rollup-node. Not fixing it in this merge request.
The error proto.alpha.gas_exhausted.operation is raised during the wasm tests. This merge request can be merged
only if the error is not raised during the arith ones. Apparently, this error is not raised during arith refutation tests,
it would be anyway detected by the check added by @mebsout but I don't know if it's flaky:
Arith refutation tests: 4 runs and no gas_exhausted error.
Wasm refutation tests: 1 run and the error was raised.
Fixed by @mebsout in the mean time.
Manually testing the MR
- You can check that the arith tests are green (the CI should do it for you)
- You can also test that the wasm tests are not all green, because of #3653 (closed)
Checklist
- n/a Document the interface of any function added or modified (see the coding guidelines)
- n/a Document any change to the user interface, including configuration parameters (see node configuration)
-
Provide automatic testing (see the testing guide). - n/a For new features and bug fixes, add an item in the appropriate changelog (
docs/protocols/alpha.rstfor the protocol and the environment,CHANGES.rstat the root of the repository for everything else). -
Select suitable reviewers using the Reviewersfield below. -
Select as Assigneethe next person who should take action on that MR
Edited by Valentin Chaboche