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
Staking v4: integration tests for unStake nodes #4121
Staking v4: integration tests for unStake nodes #4121
Conversation
…aking-v4-integration-tests-unstake
…aking-v4-integration-tests-unstake # Conflicts: # integrationTests/vm/staking/stakingV4_test.go
for _, blsKey := range nodesData.BLSKeys { | ||
signature := append([]byte("signature-"), blsKey...) | ||
//TODO: | ||
// - Do the same for unJail |
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.
do not forget to implement the TODO
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.
Already implemented in #4170
MinDelegationAmount: big.NewInt(1), | ||
} | ||
marshaledData, err := marshaller.Marshal(managementData) | ||
log.LogIfError(err) |
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.
instead of logging the error maybe you can pass the t *testing.T
in order to stop the execution
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.
or return error and stop the test where the call happens
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.
When creating a newTestMetaProcessor
I do not have any t *testing.T
pointer to pass, so I'll leave it as it is
…aking-v4-integration-tests-unstake
What's new
TestStakingV4_UnStakeNodes
test which tests unstaking nodes along the staking v4 activation chainSaveDelegationManagerConfig
test func which savessystemSmartContracts.DelegationManagement
config intestscommon/stakingcommon/stakingCommon.go
Bugfixes
RegisterValidatorKey
&AddKeysToWaitingList
execution order to setStaked
flag to falsew_
prefix ingetWaitingListKeys
when saving the staking queue in current nodes config in testsRefactor
createSCRsFromStakingSCOutput
func which generates scrs related to staking instead of "hard-codding" them