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

feat(cheatcodes): add --fork-retry + --fork-retry-backoff equivalent to vm.create*Fork #9162

Open
anajuliabit opened this issue Oct 22, 2024 · 1 comment
Labels
A-cheatcodes Area: cheatcodes T-feature Type: feature

Comments

@anajuliabit
Copy link

Component

Forge

Describe the feature you would like

Most of our integration tests are multichain, meaning we use vm.createFork to set up the forks. We can't use the --fork-url option because we want to ensure that the fork IDs have the same order, regardless of which chain the tests belong to.

For example, we always instantiate the forks in the following order:

  • ID 0: Moonbeam fork
  • ID 1: Base fork
  • ID 2: Optimism fork

When running a Base integration test, the setUp function first fetches the Moonbeam governor and executes any live proposals (that can be cross-chain).

This is why we cannot use --fork-url with forge test.

Please allow the --fork-retry and --fork-retry-backoff flags to be used without requiring the --fork-url.

Additional context

No response

@anajuliabit anajuliabit added T-feature Type: feature T-needs-triage Type: this issue needs to be labelled labels Oct 22, 2024
@zerosnacks
Copy link
Member

Hi @anajuliabit thanks for your suggestion,

Supportive of the idea, wondering how we can best implement this.
At the moment it doesn't seem like simply allowing --fork-retry / --fork-retry-backoff to be ran without --fork-url would have any effect as the two flows are quite distinct.

We could overload the vm.createFork / vm.createSelectFork cheatcode to pass in additional configuration. This would allow you to configure a per-fork retries and backoff which seems desirable.

@zerosnacks zerosnacks removed the T-needs-triage Type: this issue needs to be labelled label Oct 23, 2024
@zerosnacks zerosnacks changed the title Allow --fork-retry and --fork-retry-backoff without requiring --fork-url feat(cheatcodes): add --fork-retry + --fork-retry-backoff equivalent to vm.create*Fork Oct 23, 2024
@zerosnacks zerosnacks added the A-cheatcodes Area: cheatcodes label Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-cheatcodes Area: cheatcodes T-feature Type: feature
Projects
Status: Todo
Development

No branches or pull requests

2 participants