Skip to content
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

test: integration tests for creating and impeaching validator #86

Merged
merged 5 commits into from
Feb 16, 2023

Conversation

randyahx
Copy link
Contributor

@randyahx randyahx commented Jan 12, 2023

Description

Add additional test cases for creating and impeaching a validator.

Rationale

Add more test cases to ensure the function works right.

Example

NA

Changes

  • Add integration test cases for creating validator
  • Add integration test cases for impeaching validator

randyahx and others added 5 commits February 14, 2023 15:41
Create Validator Testcases:
Create with valid proposal
Create with invalid proposal (invalid --from account)
Create with repeated bls address
Create with repeated bls pub key

Impeach Validator Testcases:
Impeach with valid proposal
Impeach with invalid proposal (invalid --from account)
Impeach a validator that was already impeached
@keefel keefel force-pushed the randyahx/additionalUT branch from 331c715 to d44ae71 Compare February 15, 2023 07:28
@keefel keefel changed the title test: added additional testcases for create and impeach validator test: integration tests for creating and impeaching validator Feb 15, 2023
@keefel keefel added the r4r label Feb 15, 2023
@unclezoro unclezoro merged commit 97c6273 into develop Feb 16, 2023
@unclezoro unclezoro deleted the randyahx/additionalUT branch April 18, 2023 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants