-
Notifications
You must be signed in to change notification settings - Fork 6
refactor: update tests with SignetStd for easy local/fork testing #84
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
Conversation
90fa4d1
to
6b57a4f
Compare
this is based on #83 ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: signet std could be immutables, but i don't think it matters
i made them mutable because of the pattern i used in the fork testing -- setting up the fork and then re-running i am actually realizing while writing this comment though that the fork testing strategy could be changed to
(which is cool 👀) nevertheless i think having the configuration mutable leaves things a bit more flexible re: setting up forking in the code? 🤔 |
Refactor local tests to run against values in SignetStd contract, so that all local tests can be run as fork tests against any chain.