Remove unnecessary finalizeEpoch modifier from migrateTo

Fixes #109
logos-co · Sep 11, 2024 · 80456b8 · 80456b8
1 parent b312c59
commit 80456b8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/.gas-snapshot b/.gas-snapshot
@@ -24,7 +24,7 @@ MigrateTest:testDeployment() (gas: 28720)
 MigrateTest:test_RevertWhen_NoPendingMigration() (gas: 1152399)
 MigrateTest:test_RevertWhen_SenderIsNotVault() (gas: 10750)
 MigrationInitializeTest:testDeployment() (gas: 28720)
-MigrationInitializeTest:test_RevertWhen_MigrationPending() (gas: 5716968)
+MigrationInitializeTest:test_RevertWhen_MigrationPending() (gas: 5495325)
 MigrationStakeManagerTest:testDeployment() (gas: 28720)
 MigrationStakeManagerTest:testNewDeployment() (gas: 30859)
 MigrationStakeManagerTest:test_ExecuteEpochShouldNotIncreaseEpochInMigration() (gas: 105686)
@@ -49,7 +49,7 @@ UnstakeTest:test_UnstakeShouldBurnMultiplierPoints() (gas: 5497531)
 UnstakeTest:test_UnstakeShouldReturnFund_NoLockUp() (gas: 1026695)
 UnstakeTest:test_UnstakeShouldReturnFund_WithLockUp() (gas: 1115820)
 UserFlowsTest:testDeployment() (gas: 28720)
-UserFlowsTest:test_PendingMPToBeMintedCannotBeGreaterThanTotalSupplyMP(uint8) (runs: 1001, μ: 69673434, ~: 32945058)
+UserFlowsTest:test_PendingMPToBeMintedCannotBeGreaterThanTotalSupplyMP(uint8) (runs: 1001, μ: 68857964, ~: 30150189)
 UserFlowsTest:test_StakeWithLockUpTimeLocksStake() (gas: 1116708)
 UserFlowsTest:test_StakedSupplyShouldIncreaseAndDecreaseAgain() (gas: 1951147)
 VaultFactoryTest:testDeployment() (gas: 9774)
diff --git a/contracts/StakeManager.sol b/contracts/StakeManager.sol
@@ -392,7 +392,6 @@ contract StakeManager is Ownable {
         onlyVault
         onlyAccountInitialized(msg.sender)
         onlyPendingMigration
-        finalizeEpoch
         returns (StakeManager newManager)
     {
         _processAccount(accounts[msg.sender], currentEpoch);