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

run all scenarios in CI tests #113

Open
zeroXbrock opened this issue Feb 4, 2025 · 0 comments
Open

run all scenarios in CI tests #113

zeroXbrock opened this issue Feb 4, 2025 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@zeroXbrock
Copy link
Member

Is your feature request related to a problem? Please describe.

If there's ever a breaking change to the TOML spec, we have to update all the builtin scenarios. Running them in CI will make sure we don't miss any, and will also alert us if we don't realize we've made a breaking change.

Describe the solution you'd like

Add a rust test that loads the scenario configs from the files in ./scenarios/ and runs setup and a short spam run on a spawned anvil instance.

Describe alternatives you've considered

I considered writing a shell script that would invoke the CLI, but that would be kind of muddy, because now we'd be testing both the CLI and the scenario files. This test only needs to consider whether the scenario files are valid, both syntactically and functionally.

@zeroXbrock zeroXbrock added enhancement New feature or request good first issue Good for newcomers labels Feb 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant