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(evmstaking/keeper): add test cases for validator #66

Merged

Conversation

zsystm
Copy link
Collaborator

@zsystm zsystm commented Aug 27, 2024

coverage:

  • Increased test coverage from 0% to 86.0%
  • What's Covered
    • Most application logics written on the Story side, excluding SDK.
  • What’s Not Covered (Edge Cases Related to the SDK)
    • Failed to set collections map
    • Failed to get validator, but the reason is not ErrNoValidatorFound
    • Failed to create MsgCreateValidator
    • ...

changes:

  • added accountKeeper as a field of TestSuite for writing new test codes using accountKeeper.

issue: #64

@zsystm zsystm self-assigned this Sep 2, 2024
@zsystm zsystm force-pushed the zsystm/test/evmstaking/keeper/validator branch from 2ad4bba to 23e523b Compare September 2, 2024 07:41
coverage:
- Increased test coverage from 0% to 86.0%

changes:
- Added account keeper for testing
@limengformal limengformal merged commit 6b72ab9 into piplabs:main Sep 2, 2024
6 checks passed
@zsystm zsystm deleted the zsystm/test/evmstaking/keeper/validator branch September 25, 2024 04:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants