-
Notifications
You must be signed in to change notification settings - Fork 28
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
fork tests on other chains + fork test bug fix #556
Conversation
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.
2 annoying bits = RPC & block number
@@ -7,7 +7,7 @@ import {NomadTest} from "@nomad-xyz/contracts-core/contracts/test/utils/NomadTes | |||
|
|||
contract RebootTest is RebootLogic, NomadTest { | |||
string remote; | |||
string constant _domain = "ethereum"; | |||
string constant testDomain = "ethereum"; | |||
|
|||
function setUpReboot(string memory testName) public { | |||
// ALL |
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.
we could get RPC from config instead of .env, that would make it easier to switch between chains quickly, but the config RPC situation is grim because those are all public endpoints that are usually quite frail
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.
additionally, we could remove the pinned block number in order to make it more seamlessly compatible with all chains, but then the tests wouldn't cache the block which would make them even slower
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.
don't get RPC from config, we don't want to keep good RPCs in those
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.
should be multiple PRs
- test not updater sig
- config changes
- fix writeJson in reboot setup
- refactor reboots to allow non-ethereum runs
Motivation
Solution
PR Checklist