chore: add missed role checks (#352)

lidofinance · Oct 14, 2024 · 939482e · 939482e
1 parent ed13582
commit 939482e
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/test/fork/deployment/PostDeployment.t.sol b/test/fork/deployment/PostDeployment.t.sol
@@ -380,6 +380,14 @@ contract CSFeeOracleDeploymentTest is Test, Utilities, DeploymentFixtures {
         assertEq(oracle.getRoleMemberCount(oracle.CONTRACT_MANAGER_ROLE()), 0);
         assertEq(oracle.getRoleMemberCount(oracle.SUBMIT_DATA_ROLE()), 0);
         assertEq(oracle.getRoleMemberCount(oracle.RECOVERER_ROLE()), 0);
+        assertEq(
+            oracle.getRoleMemberCount(oracle.MANAGE_CONSENSUS_CONTRACT_ROLE()),
+            0
+        );
+        assertEq(
+            oracle.getRoleMemberCount(oracle.MANAGE_CONSENSUS_VERSION_ROLE()),
+            0
+        );
     }
 
     function test_initialState() public {

diff --git a/test/fork/invariant/Invariants.t.sol b/test/fork/invariant/Invariants.t.sol
@@ -219,6 +219,16 @@ contract CSFeeOracleInvariant is InvariantsBase {
             0,
             "recoverer"
         );
+        assertEq(
+            oracle.getRoleMemberCount(oracle.MANAGE_CONSENSUS_CONTRACT_ROLE()),
+            0,
+            "manage_consensus_contract"
+        );
+        assertEq(
+            oracle.getRoleMemberCount(oracle.MANAGE_CONSENSUS_VERSION_ROLE()),
+            0,
+            "manage_consensus_version"
+        );
     }
 }