[go: up one dir, main page]

Scoru: fix/reactivate tezt refutation tests

Context

Closes #4020 (closed)

There were two issues with the tests:

  1. The messages were not posted at the expected levels, so the empty_levels in tests' config were broken
  2. The failing ticks were broken in the node by !6478 (merged)
  3. 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

  1. You can check that the arith tests are green (the CI should do it for you)
  2. 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.rst for the protocol and the environment, CHANGES.rst at the root of the repository for everything else).
  • Select suitable reviewers using the Reviewers field below.
  • Select as Assignee the next person who should take action on that MR
Edited by Valentin Chaboche

Merge request reports

Loading