Remove assertions, the state isn't changed.

zkemail · Aug 6, 2024 · 4850ef3 · 4850ef3
1 parent d0b56b5
commit 4850ef3
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 12 deletions.
diff --git a/test/unit/EmailRecoveryManager/completeRecovery.t.sol b/test/unit/EmailRecoveryManager/completeRecovery.t.sol
@@ -134,9 +134,6 @@ contract EmailRecoveryManager_completeRecovery_Test is UnitBase {
         assertEq(recoveryRequest.executeBefore, 0);
         assertEq(recoveryRequest.currentWeight, 0);
         assertEq(recoveryRequest.calldataHash, "");
-
-        bool isActivated = emailRecoveryModule.isActivated(accountAddress);
-        assertTrue(isActivated);
     }
 
     function test_CompleteRecovery_SucceedsAlmostExpiry() public {
@@ -158,8 +155,5 @@ contract EmailRecoveryManager_completeRecovery_Test is UnitBase {
         assertEq(recoveryRequest.executeBefore, 0);
         assertEq(recoveryRequest.currentWeight, 0);
         assertEq(recoveryRequest.calldataHash, "");
-
-        bool isActivated = emailRecoveryModule.isActivated(accountAddress);
-        assertTrue(isActivated);
     }
 }
diff --git a/test/unit/EmailRecoveryManager/updateRecoveryConfig.t.sol b/test/unit/EmailRecoveryManager/updateRecoveryConfig.t.sol
@@ -98,9 +98,6 @@ contract EmailRecoveryManager_updateRecoveryConfig_Test is UnitBase {
         recoveryConfig = emailRecoveryModule.getRecoveryConfig(accountAddress);
         assertEq(recoveryConfig.delay, newDelay);
         assertEq(recoveryConfig.expiry, newExpiry);
-
-        bool isActivated = emailRecoveryModule.isActivated(accountAddress);
-        assertTrue(isActivated);
     }
 
     function test_UpdateRecoveryConfig_Succeeds() public {
@@ -120,8 +117,5 @@ contract EmailRecoveryManager_updateRecoveryConfig_Test is UnitBase {
         recoveryConfig = emailRecoveryModule.getRecoveryConfig(accountAddress);
         assertEq(recoveryConfig.delay, newDelay);
         assertEq(recoveryConfig.expiry, newExpiry);
-
-        bool isActivated = emailRecoveryModule.isActivated(accountAddress);
-        assertTrue(isActivated);
     }
 }