-
Notifications
You must be signed in to change notification settings - Fork 116
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
Dpn test fixes #1224
base: v6
Are you sure you want to change the base?
Dpn test fixes #1224
Conversation
…o prioritise other failing tests.
…#1212) * Implementing ol-genesis recover option (create JSON file with account data from snapshot) * Fix Move error when "Set the validator operator for each validator owner" in fn recovery_owners_operators * Implementing ol-genesis "recover from JSON" option (create genesis blob from JSON file with accounts) * Do not ignore mv files * Diem-framework must be built in diem-move/diem-framework dir. * Build artifacts * Reverting 0L change which causes the panic "Err value: unable to parse AccoutAddress" * Refactoring * Update Readme.md
This is a great fix. Especially ones related to roles. Could you rebase with the latest changes, I wonder if there was some drift in the version you were working from. Then please use this test command:
|
Getting errors running the test command you provided. Something to do with missing arg |
Are you running this commandsfrom the project root? |
Motivation
Most of the tests in
DPN
(which should probably be renamed tool-framework
) fail. This PR addresses some of the more trivial fixes but others warrant a more thorough review (e.g.DiemAccount
changes).Test Plan
cargo test --release
inDPN
root.Related PRs
(not sure)