fix: readd manual checks (#996)

* fix: readd manual checks

* chore: forge fmt

Author: 0xClandestine

src/contracts/core/RewardsCoordinator.sol

@@ -371,7 +371,7 @@ contract RewardsCoordinator is
      */
     function _setOperatorSplit(OperatorSplit storage operatorSplit, uint16 split, uint32 activatedAt) internal {
         require(split <= ONE_HUNDRED_IN_BIPS, SplitExceedsMax());
-        
+
         require(block.timestamp > operatorSplit.activatedAt, PreviousSplitPending());
 
         if (operatorSplit.activatedAt == 0) {
@@ -555,20 +555,20 @@ contract RewardsCoordinator is
      * @param operatorSplit The split struct for an Operator
      * @return The split in basis points.
      */
-    function _getOperatorSplit(OperatorSplit memory operatorSplit) internal view returns (uint16) {
+    function _getOperatorSplit(
+        OperatorSplit memory operatorSplit
+    ) internal view returns (uint16) {
         if (
-            (operatorSplit.activatedAt == 0) ||
-            (operatorSplit.oldSplitBips == type(uint16).max && block.timestamp < operatorSplit.activatedAt)
+            (operatorSplit.activatedAt == 0)
+                || (operatorSplit.oldSplitBips == type(uint16).max && block.timestamp < operatorSplit.activatedAt)
         ) {
             // Return the Default Operator Split if the operator split has not been initialized.
             // Also return the Default Operator Split if the operator split has been initialized but not activated yet. (i.e the first initialization)
             return defaultOperatorSplitBips;
         } else {
             // Return the new split if the new split has been activated, else return the old split.
             return
-                (block.timestamp >= operatorSplit.activatedAt)
-                    ? operatorSplit.newSplitBips
-                    : operatorSplit.oldSplitBips;
+                (block.timestamp >= operatorSplit.activatedAt) ? operatorSplit.newSplitBips : operatorSplit.oldSplitBips;
         }
     }
 

src/test/unit/AVSDirectoryUnit.t.sol

@@ -50,6 +50,8 @@ contract AVSDirectoryUnitTests is EigenLayerUnitTestSetup, IAVSDirectoryEvents,
             )
         );
         isExcludedFuzzAddress[address(avsd)] = true;
+
+        assertTrue(avsd.domainSeparator() != bytes32(0), "sanity check");
     }
 
     function _newOperatorRegistrationSignature(

src/test/unit/DelegationUnit.t.sol

@@ -1335,6 +1335,7 @@ contract DelegationManagerUnitTests is EigenLayerUnitTestSetup, IDelegationManag
 
 contract DelegationManagerUnitTests_Initialization_Setters is DelegationManagerUnitTests {
     function test_initialization() public view {
+        assertTrue(delegationManager.domainSeparator() != bytes32(0), "sanity check");
         assertEq(
             address(delegationManager.strategyManager()),
             address(strategyManagerMock),

src/test/unit/StrategyManagerUnit.t.sol

@@ -251,6 +251,7 @@ contract StrategyManagerUnitTests_initialize is StrategyManagerUnitTests {
     }
 
     function test_InitializedStorageProperly() public view {
+        assertTrue(strategyManager.domainSeparator() != bytes32(0), "sanity check");
         assertEq(strategyManager.owner(), initialOwner, "strategyManager.owner() != initialOwner");
         assertEq(
             strategyManager.strategyWhitelister(),