-
Notifications
You must be signed in to change notification settings - Fork 34
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
feat: Adds validator qgb checks tests #138
feat: Adds validator qgb checks tests #138
Conversation
…ecks # Conflicts: # client/flags/flags.go # simapp/simd/cmd/testnet.go # x/staking/simulation/operations.go
…mulation to have a deterministic way of creating it
…ecks_tests # Conflicts: # x/staking/keeper/msg_server.go # x/staking/simulation/operations.go
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.
mostly simple additions! 👍 utACK
I'm slightly concerned about upgrading a chain to use the QGB module and this change, as we'd either have to pass some address similar to the zero address, or have all of the validators eth address/orchestrator address ahead of time before upgrading.
You're right. But I thought that we will have some migration script that would change the state without depending on messages to do state transitions. If it is not, then I should change. |
Yeah, we can have an upgrade handler do whatever we need without getting that info ahead of time. I'm still worried about it though, at least until we can test upgrade, as having fake addresses as placeholders seems super hacky. |
@evan-forbes So should I remove the zero address check ? until we decide ? |
nah, let's keep it. After we get mvp working on an integration test, then we'll plan out the upgrade then |
Merging then? |
Just renamed the branch we're targeting to: release/v0.44.x-celestia-qgb. I will make sure it will always have the latest celestia branch changes (hopefuly). Then, we merge it to the default branch once we decide to merge QGB module to celestia-app. |
sounds good 👍 |
* first pass on tests fixes * fixes the rest of unit tests * remove unnecessary comments * uses default eth address when starting sim network * cosmetics * comments failing test * comments failing test * adds orchestrator/ethereum address checks for validators when creating and editing * uses correct error codes for eth/orch address errors * uncomments test and fixes it from commit 434b308 * updates tests to be able to set orch/eth address when creating validators * adds zero eth address check when creating/updating validator * attempts to fix duplicate eth address in sim network * adds validator check tests * revert squashed changes * uses unwrapped context for msg_server orch/eth validation * increase DefaultGenTxGas to accomodate new qgb validator changes * scaffolds an ethereum address from orchestrator address in staking simulation to have a deterministic way of creating it * fix eth address creation from orch address in operations * increase DefaultGenTxGas * increase DefaultGenTxGas * remove unnecessary eth address
* first pass on tests fixes * fixes the rest of unit tests * remove unnecessary comments * uses default eth address when starting sim network * cosmetics * comments failing test * comments failing test * adds orchestrator/ethereum address checks for validators when creating and editing * uses correct error codes for eth/orch address errors * uncomments test and fixes it from commit 434b308 * updates tests to be able to set orch/eth address when creating validators * adds zero eth address check when creating/updating validator * attempts to fix duplicate eth address in sim network * adds validator check tests * revert squashed changes * uses unwrapped context for msg_server orch/eth validation * increase DefaultGenTxGas to accomodate new qgb validator changes * scaffolds an ethereum address from orchestrator address in staking simulation to have a deterministic way of creating it * fix eth address creation from orch address in operations * increase DefaultGenTxGas * increase DefaultGenTxGas * remove unnecessary eth address
* first pass on tests fixes * fixes the rest of unit tests * remove unnecessary comments * uses default eth address when starting sim network * cosmetics * comments failing test * comments failing test * adds orchestrator/ethereum address checks for validators when creating and editing * uses correct error codes for eth/orch address errors * uncomments test and fixes it from commit 434b308 * updates tests to be able to set orch/eth address when creating validators * adds zero eth address check when creating/updating validator * attempts to fix duplicate eth address in sim network * adds validator check tests * revert squashed changes * uses unwrapped context for msg_server orch/eth validation * increase DefaultGenTxGas to accomodate new qgb validator changes * scaffolds an ethereum address from orchestrator address in staking simulation to have a deterministic way of creating it * fix eth address creation from orch address in operations * increase DefaultGenTxGas * increase DefaultGenTxGas * remove unnecessary eth address
Adds to #114