Skip to content

test: depositShares >= withdrawableShares invariant#1201

Merged
ypatil12 merged 2 commits intotest/slashing-integration-testingfrom
yash/depositSharesInvariant
Mar 4, 2025
Merged

test: depositShares >= withdrawableShares invariant#1201
ypatil12 merged 2 commits intotest/slashing-integration-testingfrom
yash/depositSharesInvariant

Conversation

@ypatil12
Copy link
Copy Markdown
Contributor

@ypatil12 ypatil12 commented Mar 3, 2025

Motivation:

We want to validate key invariant in our system for any share changes.

Modifications:

Adds assert_DepositShares_GTE_WithdrawableShares

Result:

More comprehensive invariants

@ypatil12 ypatil12 changed the title test: depositShares >= withdrawableShares invariant test: depositShares >= withdrawableShares invariant Mar 3, 2025
@ypatil12 ypatil12 added ✨ Enhancement New feature or request. 🧪 Test Test-related changes (unit, integration, etc.). labels Mar 4, 2025
@ypatil12 ypatil12 merged commit 8e9c7dd into test/slashing-integration-testing Mar 4, 2025
10 checks passed
@ypatil12 ypatil12 deleted the yash/depositSharesInvariant branch March 4, 2025 17:03
ypatil12 added a commit that referenced this pull request Mar 5, 2025
**Motivation:**

We want to validate key invariant in our system for any share changes. 

**Modifications:**

Adds `assert_DepositShares_GTE_WithdrawableShares`

**Result:**

More comprehensive invariants
ypatil12 added a commit that referenced this pull request Mar 5, 2025
**Motivation:**

We want to validate key invariant in our system for any share changes. 

**Modifications:**

Adds `assert_DepositShares_GTE_WithdrawableShares`

**Result:**

More comprehensive invariants
ypatil12 added a commit that referenced this pull request Mar 5, 2025
**Motivation:**

Slashing Integration Testing for Competition Audit

**Modifications:**

***General State Validation***
- #1204
- #1198
- #1169
- #1158

***Upgrade Tests***
- #1187
- #1171
- #1143

***Dual Slash Tests***
- #1195
- #1153

***Rounding Tests***
- #1178

***EigenPod Tests***
- #1191
- #1188
- #1203
- #1194
- #1163

***Invariants***
- #1201
- #1176
- #1192
- #1197
- #1175
- #1189
- #1150
- #1149

**Result:**

Comprehensive Test Coverage
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Enhancement New feature or request. 🧪 Test Test-related changes (unit, integration, etc.).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants