aptos-labs / aptos-core

Aptos is a layer 1 blockchain built to support the widespread use of blockchain through better technology and user experience.
https://aptosfoundation.org
Other
5.85k stars 3.54k forks source link

[Bug] prover global timeout causes inconsistent outputs with CI #13188

Closed brmataptos closed 2 weeks ago

brmataptos commented 2 weeks ago

🐛 Bug

In PR#11384, there is currently a global timeout which can be seen in third_party/move/move-prover/tests/sources/functional/mut_ref.v2_exp. This is hard to sync with CI tests, as timeout is doubled if MVP_TEST_ON_CI is set. There seems to be some intended mechanism to mask the actual timeout from the prover tests in boogie_wrapper.rs:769 but it does nothing.

What's funny is that the timeout is only adjusted for the error message, not for actually timing anything.