Skip to content

Commit cb5524d

Browse files
committed
remove stPROVE from Unstake event
1 parent b0059cd commit cb5524d

File tree

5 files changed

+66
-75
lines changed

5 files changed

+66
-75
lines changed

contracts/.gas-snapshot

Lines changed: 61 additions & 61 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ SuccinctStakingCancelSlashTests:test_CancelSlash_WhenExactBoundaryTiming() (gas:
1010
SuccinctStakingCancelSlashTests:test_CancelSlash_WhenGovernanceCanFinishImmediately() (gas: 240295)
1111
SuccinctStakingCancelSlashTests:test_CancelSlash_WhenMultipleSlashesCanBeCancelledIndependently() (gas: 218874)
1212
SuccinctStakingCancelSlashTests:test_CancelSlash_WhenNonOwnerCancelsAfterDeadline() (gas: 138154)
13-
SuccinctStakingCancelSlashTests:test_CancelSlash_WhenStakersCanUnstakeAfterCancellation() (gas: 381327)
13+
SuccinctStakingCancelSlashTests:test_CancelSlash_WhenStakersCanUnstakeAfterCancellation() (gas: 363049)
1414
SuccinctStakingCancelSlashTests:test_RevertCancelSlash_WhenAlreadyResolved() (gas: 137407)
1515
SuccinctStakingCancelSlashTests:test_RevertCancelSlash_WhenNonOwnerCancelsEarly() (gas: 132940)
1616
SuccinctStakingDispenseTests:test_Dispense_WhenConsecutiveWithWaiting() (gas: 580194)
@@ -30,76 +30,76 @@ SuccinctStakingDispenseTests:test_Dispense_WhenRateIsZero() (gas: 133785)
3030
SuccinctStakingDispenseTests:test_Dispense_WhenRateLimit() (gas: 510282)
3131
SuccinctStakingDispenseTests:test_Dispense_WhenReentrancyProtection() (gas: 172061)
3232
SuccinctStakingDispenseTests:test_Dispense_WhenTotalEarnedEqualsDispensedPlusAvailable() (gas: 295244)
33-
SuccinctStakingDispenseTests:test_Dispense_WhenValid() (gas: 600445)
33+
SuccinctStakingDispenseTests:test_Dispense_WhenValid() (gas: 582168)
3434
SuccinctStakingDispenseTests:test_Dispense_WhenVeryLargeTimePeriod() (gas: 576052)
3535
SuccinctStakingDispenseTests:test_RevertDispense_WhenNotEnoughPROVE() (gas: 56900)
3636
SuccinctStakingDispenseTests:test_Revert_WhenSetDispenseRate_WhenNotDispenser() (gas: 13531)
3737
SuccinctStakingDispenseTests:test_Revert_WhenSetDispenser_WhenNotOwner() (gas: 15778)
3838
SuccinctStakingDispenseTests:test_SetDispenseRate_WhenValid() (gas: 37299)
3939
SuccinctStakingDispenseTests:test_SetDispenser_WhenValid() (gas: 22253)
4040
SuccinctStakingFulfillmentTests:test_RevertReward_WhenProverNotFound() (gas: 12788)
41-
SuccinctStakingFulfillmentTests:test_Reward_WhenPartialUnstake() (gas: 924663)
42-
SuccinctStakingFulfillmentTests:test_Reward_WhenTwoStakersOneProverRewardDistributedAfterBothStake() (gas: 1078446)
43-
SuccinctStakingFulfillmentTests:test_Reward_WhenTwoStakersOneProverRewardOnlyDistributedAfterFirstStake() (gas: 1096968)
44-
SuccinctStakingFulfillmentTests:test_Reward_WhenTwoStakersTwoProversBothRewarded() (gas: 1327528)
45-
SuccinctStakingFulfillmentTests:test_Reward_WhenTwoStakersTwoProversOneRewarded() (gas: 1206436)
46-
SuccinctStakingFulfillmentTests:test_Reward_WhenUnstaked() (gas: 774378)
41+
SuccinctStakingFulfillmentTests:test_Reward_WhenPartialUnstake() (gas: 889788)
42+
SuccinctStakingFulfillmentTests:test_Reward_WhenTwoStakersOneProverRewardDistributedAfterBothStake() (gas: 1041891)
43+
SuccinctStakingFulfillmentTests:test_Reward_WhenTwoStakersOneProverRewardOnlyDistributedAfterFirstStake() (gas: 1060412)
44+
SuccinctStakingFulfillmentTests:test_Reward_WhenTwoStakersTwoProversBothRewarded() (gas: 1290973)
45+
SuccinctStakingFulfillmentTests:test_Reward_WhenTwoStakersTwoProversOneRewarded() (gas: 1169880)
46+
SuccinctStakingFulfillmentTests:test_Reward_WhenUnstaked() (gas: 756100)
4747
SuccinctStakingFulfillmentTests:test_Reward_WhenValid() (gas: 590301)
4848
SuccinctStakingInitalizationTests:test_RevertInitialize_WhenAlreadyInitialized() (gas: 27195)
49-
SuccinctStakingInitalizationTests:test_RevertInitialize_WhenNotOwner() (gas: 4632743)
49+
SuccinctStakingInitalizationTests:test_RevertInitialize_WhenNotOwner() (gas: 4618694)
5050
SuccinctStakingMiscellaneousTests:test_Misc_BelowMinimumStake() (gas: 145046)
51-
SuccinctStakingMiscellaneousTests:test_Misc_CannotAvoidSlashingByUnstaking() (gas: 782064)
51+
SuccinctStakingMiscellaneousTests:test_Misc_CannotAvoidSlashingByUnstaking() (gas: 763787)
5252
SuccinctStakingMiscellaneousTests:test_Misc_CannotSwitchProvers() (gas: 459678)
53-
SuccinctStakingMiscellaneousTests:test_Misc_ConcurrentOperations() (gas: 1194917)
53+
SuccinctStakingMiscellaneousTests:test_Misc_ConcurrentOperations() (gas: 1158362)
5454
SuccinctStakingMiscellaneousTests:test_Misc_DispenseOneWei() (gas: 160184)
5555
SuccinctStakingMiscellaneousTests:test_Misc_DispenseOverflow() (gas: 160045)
5656
SuccinctStakingMiscellaneousTests:test_Misc_DispenseZeroTime() (gas: 163513)
57-
SuccinctStakingMiscellaneousTests:test_Misc_DustAmountsCanBeUnstaked() (gas: 765715)
58-
SuccinctStakingMiscellaneousTests:test_Misc_ExactMinimumStake() (gas: 611260)
59-
SuccinctStakingMiscellaneousTests:test_Misc_ExactTimingBoundaries() (gas: 517282)
57+
SuccinctStakingMiscellaneousTests:test_Misc_DustAmountsCanBeUnstaked() (gas: 730840)
58+
SuccinctStakingMiscellaneousTests:test_Misc_ExactMinimumStake() (gas: 592982)
59+
SuccinctStakingMiscellaneousTests:test_Misc_ExactTimingBoundaries() (gas: 499004)
6060
SuccinctStakingMiscellaneousTests:test_Misc_ExchangeRateManipulation() (gas: 722604)
6161
SuccinctStakingMiscellaneousTests:test_Misc_FinishUnstakeNoClaims() (gas: 369919)
6262
SuccinctStakingMiscellaneousTests:test_Misc_FrontrunningRewardDistribution() (gas: 614396)
6363
SuccinctStakingMiscellaneousTests:test_Misc_MaxUint256Operations() (gas: 390861)
64-
SuccinctStakingMiscellaneousTests:test_Misc_MaximumStakeValues() (gas: 610700)
65-
SuccinctStakingMiscellaneousTests:test_Misc_MultiplePartialUnstakesExceedBalance() (gas: 572436)
66-
SuccinctStakingMiscellaneousTests:test_Misc_MultipleUnstakeRequestsCannotDrainFunds() (gas: 525876)
64+
SuccinctStakingMiscellaneousTests:test_Misc_MaximumStakeValues() (gas: 592423)
65+
SuccinctStakingMiscellaneousTests:test_Misc_MultiplePartialUnstakesExceedBalance() (gas: 550303)
66+
SuccinctStakingMiscellaneousTests:test_Misc_MultipleUnstakeRequestsCannotDrainFunds() (gas: 507598)
6767
SuccinctStakingMiscellaneousTests:test_Misc_NonOwnerCannotSlash() (gas: 469677)
6868
SuccinctStakingMiscellaneousTests:test_Misc_PermitExpiredDeadline() (gas: 34217)
69-
SuccinctStakingMiscellaneousTests:test_Misc_RestakeAfterFullUnstake() (gas: 867580)
69+
SuccinctStakingMiscellaneousTests:test_Misc_RestakeAfterFullUnstake() (gas: 849302)
7070
SuccinctStakingMiscellaneousTests:test_Misc_RewardAndSlashSameBlock() (gas: 619246)
71-
SuccinctStakingMiscellaneousTests:test_Misc_RoundingErrorsDoNotAccumulate() (gas: 3844862)
71+
SuccinctStakingMiscellaneousTests:test_Misc_RoundingErrorsDoNotAccumulate() (gas: 3677206)
7272
SuccinctStakingMiscellaneousTests:test_Misc_SlashExactTimingBoundary() (gas: 548832)
7373
SuccinctStakingMiscellaneousTests:test_Misc_SlashInvalidIndex() (gas: 472039)
7474
SuccinctStakingMiscellaneousTests:test_Misc_UnstakeMoreThanBalance() (gas: 372440)
75-
SuccinctStakingMiscellaneousTests:test_Misc_UnstakeQueueDoS() (gas: 4937965)
75+
SuccinctStakingMiscellaneousTests:test_Misc_UnstakeQueueDoS() (gas: 4755189)
7676
SuccinctStakingMiscellaneousTests:test_Misc_VaultReturnsZero() (gas: 457157)
7777
SuccinctStakingMiscellaneousTests:test_Misc_ZeroProverAddress() (gas: 19892)
7878
SuccinctStakingRegistryTests:test_CreateProver_WhenValid() (gas: 1645914)
7979
SuccinctStakingRegistryTests:test_RevertCreateProver_WhenAlreadyCreated() (gas: 1640377)
8080
SuccinctStakingSetupTests:test_SetUp() (gas: 127994)
8181
SuccinctStakingSlashTests:test_RevertSlash_WhenProverNotFound() (gas: 413960)
82-
SuccinctStakingSlashTests:test_Slash_Events() (gas: 848228)
82+
SuccinctStakingSlashTests:test_Slash_Events() (gas: 826095)
8383
SuccinctStakingSlashTests:test_Slash_Events_WhenDeactivated() (gas: 582573)
84-
SuccinctStakingSlashTests:test_Slash_WhenCancelled() (gas: 663404)
85-
SuccinctStakingSlashTests:test_Slash_WhenCumulativeSlashAmountExceedsBalance() (gas: 714094)
86-
SuccinctStakingSlashTests:test_Slash_WhenDispenseAfterFullSlash() (gas: 701577)
87-
SuccinctStakingSlashTests:test_Slash_WhenDispenseBeforeFullSlash() (gas: 685770)
88-
SuccinctStakingSlashTests:test_Slash_WhenFactorZeroResetWithDeactivation() (gas: 670569)
89-
SuccinctStakingSlashTests:test_Slash_WhenFullAmount() (gas: 663779)
84+
SuccinctStakingSlashTests:test_Slash_WhenCancelled() (gas: 645126)
85+
SuccinctStakingSlashTests:test_Slash_WhenCumulativeSlashAmountExceedsBalance() (gas: 695814)
86+
SuccinctStakingSlashTests:test_Slash_WhenDispenseAfterFullSlash() (gas: 683297)
87+
SuccinctStakingSlashTests:test_Slash_WhenDispenseBeforeFullSlash() (gas: 667490)
88+
SuccinctStakingSlashTests:test_Slash_WhenFactorZeroResetWithDeactivation() (gas: 652289)
89+
SuccinctStakingSlashTests:test_Slash_WhenFullAmount() (gas: 645499)
9090
SuccinctStakingSlashTests:test_Slash_WhenFullSlashDeactivatesProver() (gas: 498582)
91-
SuccinctStakingSlashTests:test_Slash_WhenMultipleSlashesBeforeUnstakeFinish() (gas: 906770)
92-
SuccinctStakingSlashTests:test_Slash_WhenPartialAmount() (gas: 707847)
91+
SuccinctStakingSlashTests:test_Slash_WhenMultipleSlashesBeforeUnstakeFinish() (gas: 888492)
92+
SuccinctStakingSlashTests:test_Slash_WhenPartialAmount() (gas: 689569)
9393
SuccinctStakingSlashTests:test_Slash_WhenPartialSlashEventuallyDeactivates() (gas: 779779)
9494
SuccinctStakingSlashTests:test_Slash_WhenRequestingWhenProverHasNoStake() (gas: 143769)
9595
SuccinctStakingSlashTests:test_Slash_WhenRewardAfterFullSlash() (gas: 606196)
96-
SuccinctStakingSlashTests:test_Slash_WhenSingleSlashAmountExceedsBalance() (gas: 650349)
97-
SuccinctStakingSlashTests:test_Slash_WhenSlashExceedsVaultButNotTotal() (gas: 794472)
96+
SuccinctStakingSlashTests:test_Slash_WhenSingleSlashAmountExceedsBalance() (gas: 632069)
97+
SuccinctStakingSlashTests:test_Slash_WhenSlashExceedsVaultButNotTotal() (gas: 776194)
9898
SuccinctStakingSlashTests:test_Slash_WhenStakeBlockedAfterDeactivation() (gas: 530040)
99-
SuccinctStakingSlashTests:test_Slash_WhenTwoProversDispenseAfterOneFullSlash() (gas: 1121821)
100-
SuccinctStakingSlashTests:test_Slash_WhenTwoStakersDispenseBeforeFullSlash() (gas: 941839)
101-
SuccinctStakingSlashTests:test_Slash_WhenTwoStakersFullSlash() (gas: 905827)
102-
SuccinctStakingSlashTests:test_Slash_WhenVaultDrainedToZero() (gas: 658125)
99+
SuccinctStakingSlashTests:test_Slash_WhenTwoProversDispenseAfterOneFullSlash() (gas: 1085264)
100+
SuccinctStakingSlashTests:test_Slash_WhenTwoStakersDispenseBeforeFullSlash() (gas: 905279)
101+
SuccinctStakingSlashTests:test_Slash_WhenTwoStakersFullSlash() (gas: 869267)
102+
SuccinctStakingSlashTests:test_Slash_WhenVaultDrainedToZero() (gas: 639848)
103103
SuccinctStakingStakeTests:test_PermitAndStake_WhenAttackerFrontruns() (gas: 402373)
104104
SuccinctStakingStakeTests:test_PermitAndStake_WhenLessThanApproved() (gas: 419932)
105105
SuccinctStakingStakeTests:test_PermitAndStake_WhenValid() (gas: 415464)
@@ -126,34 +126,34 @@ SuccinctStakingTransferTests:test_RevertWithdraw_WhenDirectiPROVEFromProver() (g
126126
SuccinctStakingTransferTests:test_RevertWithdraw_WhenDirectiPROVEFromStaker() (gas: 387776)
127127
SuccinctStakingTransferTests:test_Transfer_WhenProve() (gas: 23892)
128128
SuccinctStakingUnstakeTests:test_RevertUnstake_WhenAmountExceedsBalance() (gas: 410596)
129-
SuccinctStakingUnstakeTests:test_RevertUnstake_WhenFinishUnstakeWhileProverHasSlashRequest() (gas: 605901)
129+
SuccinctStakingUnstakeTests:test_RevertUnstake_WhenFinishUnstakeWhileProverHasSlashRequest() (gas: 583768)
130130
SuccinctStakingUnstakeTests:test_RevertUnstake_WhenRequestUnstakeWhileProverHasSlashRequest() (gas: 505344)
131-
SuccinctStakingUnstakeTests:test_RevertUnstake_WhenTooManyUnstakeRequests() (gas: 1672082)
131+
SuccinctStakingUnstakeTests:test_RevertUnstake_WhenTooManyUnstakeRequests() (gas: 1448867)
132132
SuccinctStakingUnstakeTests:test_RevertUnstake_WhenZeroAmount() (gas: 401029)
133-
SuccinctStakingUnstakeTests:test_Unstake_WhenEarlyAttempt() (gas: 551619)
134-
SuccinctStakingUnstakeTests:test_Unstake_WhenManySmallStakesUnstakes() (gas: 1299169)
135-
SuccinctStakingUnstakeTests:test_Unstake_WhenManySmallStakesUnstakesTimeBetween() (gas: 1303129)
136-
SuccinctStakingUnstakeTests:test_Unstake_WhenManySmallUnstakes() (gas: 1525989)
137-
SuccinctStakingUnstakeTests:test_Unstake_WhenMaxRequestsReset() (gas: 2700392)
138-
SuccinctStakingUnstakeTests:test_Unstake_WhenMultipleRequestsSumToFull() (gas: 619245)
139-
SuccinctStakingUnstakeTests:test_Unstake_WhenMultipleRewards() (gas: 892193)
140-
SuccinctStakingUnstakeTests:test_Unstake_WhenMultipleUnstakes() (gas: 715453)
141-
SuccinctStakingUnstakeTests:test_Unstake_WhenOneStakerTwoProversRewardNonDelegated() (gas: 1029996)
142-
SuccinctStakingUnstakeTests:test_Unstake_WhenProverFullSlashAfterEscrow() (gas: 652264)
143-
SuccinctStakingUnstakeTests:test_Unstake_WhenProverFullSlashBeforeEscrow() (gas: 606556)
144-
SuccinctStakingUnstakeTests:test_Unstake_WhenProverPartialSlashAfterEscrow() (gas: 649274)
145-
SuccinctStakingUnstakeTests:test_Unstake_WhenProverPartialSlashBeforeEscrow() (gas: 670177)
146-
SuccinctStakingUnstakeTests:test_Unstake_WhenRequestAfterSlash() (gas: 1057186)
147-
SuccinctStakingUnstakeTests:test_Unstake_WhenRewardDuringOngoingUnstake() (gas: 888856)
148-
SuccinctStakingUnstakeTests:test_Unstake_WhenRewardDuringOngoingUnstakeOutOfOrder() (gas: 888840)
149-
SuccinctStakingUnstakeTests:test_Unstake_WhenRewardDuringUnstakePeriod() (gas: 865085)
150-
SuccinctStakingUnstakeTests:test_Unstake_WhenSlashDuringUnstakePeriod() (gas: 799001)
151-
SuccinctStakingUnstakeTests:test_Unstake_WhenSomeClaimsReadyOthersNot() (gas: 1083041)
152-
SuccinctStakingUnstakeTests:test_Unstake_WhenSomeoneElseFinishesUnstake() (gas: 563772)
153-
SuccinctStakingUnstakeTests:test_Unstake_WhenTwoClaimsMixedReady() (gas: 636471)
154-
SuccinctStakingUnstakeTests:test_Unstake_WhenTwoStakersOneProverReward() (gas: 1107176)
155-
SuccinctStakingUnstakeTests:test_Unstake_WhenTwoStakersTwoProversDispenseOnly() (gas: 1008382)
156-
SuccinctStakingUnstakeTests:test_Unstake_WhenValid() (gas: 635836)
133+
SuccinctStakingUnstakeTests:test_Unstake_WhenEarlyAttempt() (gas: 533341)
134+
SuccinctStakingUnstakeTests:test_Unstake_WhenManySmallStakesUnstakes() (gas: 1280792)
135+
SuccinctStakingUnstakeTests:test_Unstake_WhenManySmallStakesUnstakesTimeBetween() (gas: 1284752)
136+
SuccinctStakingUnstakeTests:test_Unstake_WhenManySmallUnstakes() (gas: 1474218)
137+
SuccinctStakingUnstakeTests:test_Unstake_WhenMaxRequestsReset() (gas: 2354172)
138+
SuccinctStakingUnstakeTests:test_Unstake_WhenMultipleRequestsSumToFull() (gas: 574979)
139+
SuccinctStakingUnstakeTests:test_Unstake_WhenMultipleRewards() (gas: 873763)
140+
SuccinctStakingUnstakeTests:test_Unstake_WhenMultipleUnstakes() (gas: 680378)
141+
SuccinctStakingUnstakeTests:test_Unstake_WhenOneStakerTwoProversRewardNonDelegated() (gas: 993441)
142+
SuccinctStakingUnstakeTests:test_Unstake_WhenProverFullSlashAfterEscrow() (gas: 633984)
143+
SuccinctStakingUnstakeTests:test_Unstake_WhenProverFullSlashBeforeEscrow() (gas: 588276)
144+
SuccinctStakingUnstakeTests:test_Unstake_WhenProverPartialSlashAfterEscrow() (gas: 630996)
145+
SuccinctStakingUnstakeTests:test_Unstake_WhenProverPartialSlashBeforeEscrow() (gas: 651900)
146+
SuccinctStakingUnstakeTests:test_Unstake_WhenRequestAfterSlash() (gas: 1020036)
147+
SuccinctStakingUnstakeTests:test_Unstake_WhenRewardDuringOngoingUnstake() (gas: 851999)
148+
SuccinctStakingUnstakeTests:test_Unstake_WhenRewardDuringOngoingUnstakeOutOfOrder() (gas: 851982)
149+
SuccinctStakingUnstakeTests:test_Unstake_WhenRewardDuringUnstakePeriod() (gas: 846655)
150+
SuccinctStakingUnstakeTests:test_Unstake_WhenSlashDuringUnstakePeriod() (gas: 780419)
151+
SuccinctStakingUnstakeTests:test_Unstake_WhenSomeClaimsReadyOthersNot() (gas: 990403)
152+
SuccinctStakingUnstakeTests:test_Unstake_WhenSomeoneElseFinishesUnstake() (gas: 545494)
153+
SuccinctStakingUnstakeTests:test_Unstake_WhenTwoClaimsMixedReady() (gas: 598860)
154+
SuccinctStakingUnstakeTests:test_Unstake_WhenTwoStakersOneProverReward() (gas: 1070621)
155+
SuccinctStakingUnstakeTests:test_Unstake_WhenTwoStakersTwoProversDispenseOnly() (gas: 971827)
156+
SuccinctStakingUnstakeTests:test_Unstake_WhenValid() (gas: 619438)
157157
SuccinctTokenTest:test_InitialOwner() (gas: 12700)
158158
SuccinctTokenTest:test_Mint_WhenOwner() (gas: 60886)
159159
SuccinctTokenTest:test_RevertMint_WhenNotOwner() (gas: 10986)

contracts/src/SuccinctStaking.sol

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -308,7 +308,6 @@ contract SuccinctStaking is
308308
// Record the unstake request.
309309
unstakeClaims[msg.sender].push(
310310
UnstakeClaim({
311-
stPROVE: _stPROVE,
312311
iPROVEEscrow: iPROVEEscrow,
313312
slashFactor: pool.slashFactor,
314313
timestamp: block.timestamp
@@ -611,7 +610,7 @@ contract SuccinctStaking is
611610
PROVE = IERC4626(iProve).redeem(iPROVEScaled, _staker, address(this));
612611
}
613612

614-
emit Unstake(_staker, _prover, PROVE, iPROVEScaled, _claim.stPROVE);
613+
emit Unstake(_staker, _prover, PROVE, iPROVEScaled);
615614
}
616615

617616
/// @dev Iterate over the unstake claims, processing each one that has passed the unstake

contracts/src/interfaces/ISuccinctStaking.sol

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,11 @@ import {IProverRegistry} from "./IProverRegistry.sol";
55

66
interface ISuccinctStaking is IProverRegistry {
77
/// @dev Represents a claim for unstaking.
8-
/// @param stPROVE The share amount of $stPROVE requested to be unstaked.
98
/// @param iPROVEEscrow The escrowed amount of $iPROVE at request time.
109
/// @param slashFactor The slash factor for the prover when the claim was created (1e27 fp).
1110
/// @param timestamp The timestamp when the unstake was requested. Used for comparing against
1211
/// the `unstakePeriod()` to determine if the claim can be finished.
1312
struct UnstakeClaim {
14-
uint256 stPROVE;
1513
uint256 iPROVEEscrow;
1614
uint256 slashFactor;
1715
uint256 timestamp;
@@ -61,13 +59,7 @@ interface ISuccinctStaking is IProverRegistry {
6159
);
6260

6361
/// @dev Emitted when a staker unstakes from a prover.
64-
event Unstake(
65-
address indexed staker,
66-
address indexed prover,
67-
uint256 PROVE,
68-
uint256 iPROVE,
69-
uint256 stPROVE
70-
);
62+
event Unstake(address indexed staker, address indexed prover, uint256 PROVE, uint256 iPROVE);
7163

7264
/// @dev Emitted when a prover is requested to be slashed.
7365
event SlashRequest(address indexed prover, uint256 iPROVE, uint256 index);

contracts/src/mocks/MockStaking.sol

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,6 @@ contract MockStaking is ProverRegistry, ISuccinctStaking {
112112

113113
unstakeClaims[msg.sender].push(
114114
UnstakeClaim({
115-
stPROVE: _stPROVE,
116115
iPROVEEscrow: _stPROVE,
117116
slashFactor: pool.slashFactor,
118117
timestamp: block.timestamp

contracts/test/SuccinctStaking.unstake.t.sol

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,8 @@ contract SuccinctStakingUnstakeTests is SuccinctStakingTest {
3636
assertEq(poolBefore.slashFactor, 0, "Slash factor should be 0 before init");
3737

3838
// Step 1: Submit unstake request
39+
vm.expectEmit(true, true, true, true);
40+
emit ISuccinctStaking.UnstakeRequest(STAKER_1, ALICE_PROVER, stakeAmount, stakeAmount);
3941
_requestUnstake(STAKER_1, stakeAmount);
4042

4143
// After request unstake: stPROVE burned, iPROVE escrowed
@@ -65,7 +67,7 @@ contract SuccinctStakingUnstakeTests is SuccinctStakingTest {
6567
skip(UNSTAKE_PERIOD);
6668

6769
vm.expectEmit(true, true, true, true);
68-
emit ISuccinctStaking.Unstake(STAKER_1, ALICE_PROVER, stakeAmount, stakeAmount, stakeAmount);
70+
emit ISuccinctStaking.Unstake(STAKER_1, ALICE_PROVER, stakeAmount, stakeAmount);
6971
vm.expectEmit(true, true, true, true);
7072
emit ISuccinctStaking.ProverUnbound(STAKER_1, ALICE_PROVER);
7173

@@ -1225,7 +1227,6 @@ contract SuccinctStakingUnstakeTests is SuccinctStakingTest {
12251227
SuccinctStaking.UnstakeClaim[] memory claimsAfter =
12261228
SuccinctStaking(STAKING).unstakeRequests(STAKER_1);
12271229
assertEq(claimsAfter.length, 1, "Should have 1 claim remaining");
1228-
assertEq(claimsAfter[0].stPROVE, secondClaim, "Second claim should remain intact");
12291230

12301231
// Wait for second claim and finish
12311232
skip(UNSTAKE_PERIOD / 2);

0 commit comments

Comments
 (0)