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: bc/avs slash tests #1153

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

Conversation

ypatil12
Copy link
Collaborator

@ypatil12 ypatil12 commented Feb 21, 2025

Motivation:

Tests that handle slashes by both the BC and AVS onto Native ETH.

Modifications:

Added assertions for BC/AVS slashings, no full slashed included

  • testFuzz_avsSlash_bcSlash_checkpoint
  • testFuzz_bcSlash_checkpoint_avsSlash
  • testFuzz_avsSlash_verifyValidator_bcSlash_checkpoint
  • testFuzz_avsSlash_bcSlash_verifyValidator_checkpoint
  • testFuzz_avsSlash_bcSlash_checkpoint_verifyValidator
  • testFuzz_avsSlash_bcSlash_balanceIncrease_checkpoint
  • testFuzz_avsSlash_bcSlash_checkpoint_balanceIncrease

Result:

Test coverage for BC/AVS edge cases.

Need to properly bound a lot of the AVS/BC slashing cases, but this is a start

@ypatil12 ypatil12 added 🗡️ Slashing Release Changes for the slashing release. 🧪 Test Test-related changes (unit, integration, etc.). labels Feb 21, 2025
@0xClandestine 0xClandestine force-pushed the yash/bc-avs-ordering-tests branch from 76877a7 to b97878a Compare February 21, 2025 19:40
@0xClandestine 0xClandestine marked this pull request as ready for review February 21, 2025 19:48
@0xClandestine 0xClandestine force-pushed the yash/bc-avs-ordering-tests branch from f1ec995 to b97878a Compare February 21, 2025 20:07
@bowenli86 bowenli86 force-pushed the yash/bc-avs-ordering-tests branch from e3db760 to b97878a Compare February 22, 2025 02:51
@0xClandestine
Copy link
Member

Cannot understand why ci is failing...

@ypatil12 ypatil12 force-pushed the yash/bc-avs-ordering-tests branch from afdef81 to b26a646 Compare February 25, 2025 00:05
@bowenli86
Copy link
Member

image
i'm getting the same error locally

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.

3 participants