fix: remove redundant getter functions. (OZ N-15)

packages/subgraph-service/contracts/DisputeManager.sol

@@ -46,8 +46,9 @@ contract DisputeManager is
     OwnableUpgradeable,
     GraphDirectory,
     AttestationManager,
-    DisputeManagerV1Storage,
-    IDisputeManager
+    IDisputeManager,
+    DisputeManagerV1Storage
+
 {
     using TokenUtils for IGraphToken;
     using PPMMath for uint256;
@@ -370,14 +371,6 @@ contract DisputeManager is
         return fishermanRewardCut;
     }
 
-    /**
-     * @notice Get the dispute period.
-     * @return Dispute period in seconds
-     */
-    function getDisputePeriod() external view override returns (uint64) {
-        return disputePeriod;
-    }
-
     /**
      * @notice Get the stake snapshot for an indexer.
      * @param indexer The indexer address

packages/subgraph-service/contracts/SubgraphService.sol

@@ -475,11 +475,11 @@ contract SubgraphService is
     /**
      * @notice Getter for the accepted thawing period range for provisions
      * @dev This override ensures {ProvisionManager} uses the thawing period from the {DisputeManager}
-     * @return min The minimum thawing period which is defined by {DisputeManager-getDisputePeriod}
+     * @return min The minimum thawing period which is defined by {DisputeManager-disputePeriod}
      * @return max The maximum is unbounded
      */
     function _getThawingPeriodRange() internal view override returns (uint64 min, uint64 max) {
-        uint64 disputePeriod = _disputeManager().getDisputePeriod();
+        uint64 disputePeriod = _disputeManager().disputePeriod();
         return (disputePeriod, type(uint64).max);
     }
 
@@ -549,7 +549,7 @@ contract SubgraphService is
         if (tokensCollected > 0) {
             // lock stake as economic security for fees
             uint256 tokensToLock = tokensCollected * stakeToFeesRatio;
-            uint256 unlockTimestamp = block.timestamp + _disputeManager().getDisputePeriod();
+            uint256 unlockTimestamp = block.timestamp + _disputeManager().disputePeriod();
             _lockStake(indexer, tokensToLock, unlockTimestamp);
 
             if (tokensCurators > 0) {

packages/subgraph-service/contracts/interfaces/IDisputeManager.sol

@@ -216,8 +216,6 @@ interface IDisputeManager {
 
     function getVerifierCut() external view returns (uint32);
 
-    function getDisputePeriod() external view returns (uint64);
-
     function isDisputeCreated(bytes32 disputeId) external view returns (bool);
 
     function encodeReceipt(Attestation.Receipt memory receipt) external view returns (bytes32);

packages/subgraph-service/test/subgraphService/SubgraphService.t.sol

@@ -325,7 +325,7 @@ contract SubgraphServiceTest is SubgraphServiceSharedTest {
             LinkedList.List memory claimsList = _getClaimList(_indexer);
             bytes32 claimId = _buildStakeClaimId(_indexer, claimsList.nonce - 1);
             IDataServiceFees.StakeClaim memory stakeClaim = _getStakeClaim(claimId);
-            uint64 disputePeriod = disputeManager.getDisputePeriod();
+            uint64 disputePeriod = disputeManager.disputePeriod();
             assertEq(stakeClaim.tokens, tokensToLock);
             assertEq(stakeClaim.createdAt, block.timestamp);
             assertEq(stakeClaim.releaseAt, block.timestamp + disputePeriod);