fix: remove setDeniedMany since it's unused (OZ N-03)

Author: Maikol

packages/contracts/contracts/rewards/IRewardsManager.sol

@@ -25,9 +25,6 @@ interface IRewardsManager {
 
     function setDenied(bytes32 _subgraphDeploymentID, bool _deny) external;
 
-    function setDeniedMany(bytes32[] calldata _subgraphDeploymentID, bool[] calldata _deny)
-        external;
-
     function isDenied(bytes32 _subgraphDeploymentID) external view returns (bool);
 
     // -- Getters --

packages/contracts/contracts/rewards/RewardsManager.sol

@@ -144,23 +144,6 @@ contract RewardsManager is RewardsManagerV4Storage, GraphUpgradeable, IRewardsMa
         _setDenied(_subgraphDeploymentID, _deny);
     }
 
-    /**
-     * @dev Denies to claim rewards for multiple subgraph.
-     * NOTE: Can only be called by the subgraph availability oracle
-     * @param _subgraphDeploymentID Array of subgraph deployment ID
-     * @param _deny Array of denied status for claiming rewards for each subgraph
-     */
-    function setDeniedMany(bytes32[] calldata _subgraphDeploymentID, bool[] calldata _deny)
-        external
-        override
-        onlySubgraphAvailabilityOracle
-    {
-        require(_subgraphDeploymentID.length == _deny.length, "!length");
-        for (uint256 i = 0; i < _subgraphDeploymentID.length; i++) {
-            _setDenied(_subgraphDeploymentID[i], _deny[i]);
-        }
-    }
-
     /**
      * @dev Internal: Denies to claim rewards for a subgraph.
      * @param _subgraphDeploymentID Subgraph deployment ID

packages/contracts/test/rewards/rewards.test.ts

@@ -213,21 +213,6 @@ describe('Rewards', () => {
           .withArgs(subgraphDeploymentID1, blockNum + 1)
         expect(await rewardsManager.isDenied(subgraphDeploymentID1)).eq(true)
       })
-
-      it('reject deny subgraph w/ many if not the oracle', async function () {
-        const deniedSubgraphs = [subgraphDeploymentID1, subgraphDeploymentID2]
-        const tx = rewardsManager.connect(oracle).setDeniedMany(deniedSubgraphs, [true, true])
-        await expect(tx).revertedWith('Caller must be the subgraph availability oracle')
-      })
-
-      it('should deny subgraph w/ many', async function () {
-        await rewardsManager.connect(governor).setSubgraphAvailabilityOracle(oracle.address)
-
-        const deniedSubgraphs = [subgraphDeploymentID1, subgraphDeploymentID2]
-        await rewardsManager.connect(oracle).setDeniedMany(deniedSubgraphs, [true, true])
-        expect(await rewardsManager.isDenied(subgraphDeploymentID1)).eq(true)
-        expect(await rewardsManager.isDenied(subgraphDeploymentID2)).eq(true)
-      })
     })
   })