misc: interfaces are missing functions present in their implementations

Some interfaces in the code base are not were matching their implementation, defining only a part of the functionalities implemented in their respective contracts.

Author: abarmat

contracts/governance/Controller.sol

@@ -64,23 +64,23 @@ contract Controller is IController, Governed, Pausable {
      * @notice Change the partial paused state of the contract
      * Partial pause is intented as a partial pause of the protocol
      */
-    function setPartialPaused(bool _partialPaused) external onlyGovernorOrGuardian {
+    function setPartialPaused(bool _partialPaused) external override onlyGovernorOrGuardian {
         _setPartialPaused(_partialPaused);
     }
 
     /**
      * @notice Change the paused state of the contract
      * Full pause most of protocol functions
      */
-    function setPaused(bool _paused) external onlyGovernorOrGuardian {
+    function setPaused(bool _paused) external override onlyGovernorOrGuardian {
         _setPaused(_paused);
     }
 
     /**
      * @notice Change the Pause Guardian
      * @param _newPauseGuardian The address of the new Pause Guardian
      */
-    function setPauseGuardian(address _newPauseGuardian) external onlyGovernor {
+    function setPauseGuardian(address _newPauseGuardian) external override onlyGovernor {
         _setPauseGuardian(_newPauseGuardian);
     }
 

contracts/governance/IController.sol

@@ -3,13 +3,23 @@ pragma solidity ^0.6.12;
 interface IController {
     event SetContractProxy(bytes32 id, address contractAddress);
 
+    function getGovernor() external view returns (address);
+
+    // -- Registry --
+
     function setContractProxy(bytes32 _id, address _contractAddress) external;
 
     function updateController(bytes32 _id, address _controller) external;
 
     function getContractProxy(bytes32 _id) external view returns (address);
 
-    function getGovernor() external view returns (address);
+    // -- Pausing --
+
+    function setPartialPaused(bool _partialPaused) external;
+
+    function setPaused(bool _paused) external;
+
+    function setPauseGuardian(address _newPauseGuardian) external;
 
     function paused() external view returns (bool);
 

contracts/staking/Staking.sol

@@ -393,14 +393,6 @@ contract Staking is StakingV1Storage, GraphUpgradeable, IStaking {
         emit AssetHolderUpdate(msg.sender, _assetHolder, _allowed);
     }
 
-    /**
-     * @dev Get the GRT token used by the contract.
-     * @return GRT token contract address
-     */
-    function token() public view returns (IGraphToken) {
-        return graphToken();
-    }
-
     /**
      * @dev Return if allocationID is used.
      * @param _allocationID Address used as signer by the indexer for an allocation

contracts/token/IGraphToken.sol

@@ -3,7 +3,31 @@ pragma solidity ^0.6.12;
 import "@openzeppelin/contracts/token/ERC20/IERC20.sol";
 
 interface IGraphToken is IERC20 {
+    // -- Mint and Burn --
+
     function burn(uint256 amount) external;
 
     function mint(address _to, uint256 _amount) external;
+
+    // -- Mint Admin --
+
+    function addMinter(address _account) external;
+
+    function removeMinter(address _account) external;
+
+    function renounceMinter() external;
+
+    function isMinter(address _account) external view returns (bool);
+
+    // -- Permit --
+
+    function permit(
+        address _owner,
+        address _spender,
+        uint256 _value,
+        uint256 _deadline,
+        uint8 _v,
+        bytes32 _r,
+        bytes32 _s
+    ) external;
 }