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: state validation integration testing #1169

Open
wants to merge 5 commits into
base: test/slashing-integration-testing
Choose a base branch
from

Conversation

0xClandestine
Copy link
Member

Motivation:

Explain here the context, and why you're making that change. What is the problem you're trying to solve.

Modifications:

Describe the modifications you've done.

Result:

After your change, what will change.

@0xClandestine 0xClandestine added 🗡️ Slashing Release Changes for the slashing release. 🧪 Test Test-related changes (unit, integration, etc.). labels Feb 25, 2025
@0xClandestine 0xClandestine changed the title test: upgrade integration testing test: state validation integration testing Feb 25, 2025
@0xClandestine 0xClandestine marked this pull request as draft February 25, 2025 20:48
- Add TestState struct to manage test state consistently
- Extract setup into _init_ and _setupAllocation functions
- Add test stubs for various slashing paths:
  * Partial slash with redeposit
  * Undelegate with partial slash
  * Deallocate with partial slash
  * Deregister with partial slash
@0xClandestine 0xClandestine force-pushed the 0xclandestine/slashing-integration-testing-2 branch from 5ae42b2 to e7189ef Compare February 26, 2025 00:38
@0xClandestine
Copy link
Member Author

Curious what state validation we're looking to do, the doc didn't specify? @ypatil12

@0xClandestine 0xClandestine marked this pull request as ready for review February 26, 2025 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🗡️ Slashing Release Changes for the slashing release. 🧪 Test Test-related changes (unit, integration, etc.).
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant