Hadrien Croubois 2 years ago
parent
commit
dfafd79692
1 changed files with 5 additions and 1 deletions
  1. 5 1
      certora/specs/GovernorPreventLateQuorum.spec

+ 5 - 1
certora/specs/GovernorPreventLateQuorum.spec

@@ -26,6 +26,10 @@ rule deadlineChangeToPreventLateQuorum(uint256 pId, env e, method f, calldataarg
     requireInvariant proposalStateConsistency(pId);
     requireInvariant votesImplySnapshotPassed(pId);
 
+    // This is not (easily) provable because the prover think `_totalSupplyCheckpoints` can arbitrarily change,
+    // which causes the quorum() to change. Not sure how to fix that.
+    require !quorumReached(pId) <=> getExtendedDeadline(pId) == 0;
+
     uint256 deadlineBefore         = proposalDeadline(pId);
     bool    deadlineExtendedBefore = getExtendedDeadline(pId) > 0;
     bool    quorumReachedBefore    = quorumReached(pId);
@@ -48,7 +52,7 @@ rule deadlineChangeToPreventLateQuorum(uint256 pId, env e, method f, calldataarg
         ) || (
             !deadlineExtendedBefore &&
             deadlineExtendedAfter &&
-            // !quorumReachedBefore && // Not sure how to prove that
+            !quorumReachedBefore && // Not sure how to prove that
             quorumReachedAfter &&
             deadlineAfter == clock(e) + lateQuorumVoteExtension() &&
             votingAll(f)