-
Notifications
You must be signed in to change notification settings - Fork 7
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
Get all existing tests passing #1
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* updated_sys_contracts: Update test sysio.system contract with latest build from wire-system-contracts
@valthon I am getting same results as last time fresh pull, recursive clone and build (from your branch)
|
Did a fresh checkout and now I'm also seeing a bunch of failures 😭 Moving this into draft status until I get it happy locally. Sorry for the false-start. I think I have a bad merge that picked up wrong renames or something (nodeos vs nodeop?) |
This reverts commit ca517e9.
Update hashes for the feature descriptions affected by the sysio rename
These tests are fragile as the numbers are arbitrary constants, but you have to start somewhere...
* Add fix to have sysio vote on behalf of b1 (as required by sysio.system contract) * Update to use latest sysio.system contract that correctly checks b1 vesting
Add slight delay before loading sysio.system wasm. Without this change, the set contract command would frequently fail.
Relax number of skipped blocks and incorporate tuning from antelope
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request includes several changes to the test scripts and configuration files, focusing on modifying test cases to match latest assumptions and adjusting test net initialization scripts.