Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Uncomment retryEnvelope Forwarder test #13

Merged
merged 1 commit into from
Apr 27, 2024
Merged

Conversation

Karrq
Copy link

@Karrq Karrq commented Apr 25, 2024

We were mocking the contract at that address but for another function, and the return value wasn't really used (it was fine for it to be empty).
To fix, we mock the function being called as well, so zkVM won't error out.

@Karrq Karrq requested a review from nbaztec April 25, 2024 15:16
@Karrq Karrq merged commit 8d0bfb1 into feat/zksync Apr 27, 2024
@Karrq Karrq deleted the fix/forwarder-test branch April 30, 2024 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants