You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The ReleaseGold unit tests, which use a "mock tunnel" interface for calling the initialize function, should test for the exact expected error messages.
This can be done with recoverErrorString, introduced in #11263.
Additionally, logic common to tunnels should be abstracted away into a separate contract to inherit from.
Current Behavior
MockInitialize reverts with "unsuccessful tunnel call", not the original error message.
Logic is duplicated between the two tunnel contracts.
The text was updated successfully, but these errors were encountered:
Expected Behavior
The ReleaseGold unit tests, which use a "mock tunnel" interface for calling the initialize function, should test for the exact expected error messages.
This can be done with
recoverErrorString
, introduced in #11263.Additionally, logic common to tunnels should be abstracted away into a separate contract to inherit from.
Current Behavior
MockInitialize
reverts with "unsuccessful tunnel call", not the original error message.Logic is duplicated between the two tunnel contracts.
The text was updated successfully, but these errors were encountered: