update constitution hash

ArbitrumFoundation · Dec 20, 2023 · 6af9930 · 6af9930
1 parent 396eed4
commit 6af9930
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 2 deletions.
diff --git a/src/gov-action-contracts/AIPs/AIPXAction.sol b/src/gov-action-contracts/AIPs/AIPXAction.sol
@@ -9,23 +9,31 @@ import {
     SecurityCouncilNomineeElectionGovernor,
     GovernorSettingsUpgradeable
 } from "../../security-council-mgmt/governors/SecurityCouncilNomineeElectionGovernor.sol";
+import {IArbitrumDAOConstitution} from "../../interfaces/IArbitrumDAOConstitution.sol";
 
 contract AIPXAction {
     address public immutable proxyAdmin;
     address public immutable nomineeElectionGovernorProxy;
     address public immutable newNomineeElectionGovernorImplementation;
     uint256 public immutable votingDelay;
 
+    address public immutable constitution;
+    bytes32 public immutable newConstitutionHash;
+
     constructor(
         address _proxyAdmin,
         address _nomineeElectionGovernorProxy,
         address _newNomineeElectionGovernorImplementation,
-        uint256 _votingDelay
+        uint256 _votingDelay,
+        address _constitution,
+        bytes32 _newConstitutionHash
     ) {
         proxyAdmin = _proxyAdmin;
         nomineeElectionGovernorProxy = _nomineeElectionGovernorProxy;
         newNomineeElectionGovernorImplementation = _newNomineeElectionGovernorImplementation;
         votingDelay = _votingDelay;
+        constitution = _constitution;
+        newConstitutionHash = _newConstitutionHash;
     }
 
     function perform() external {
@@ -41,5 +49,8 @@ contract AIPXAction {
             0,
             abi.encodeWithSelector(GovernorSettingsUpgradeable.setVotingDelay.selector, votingDelay)
         );
+
+        // set the new constitution hash
+        IArbitrumDAOConstitution(constitution).setConstitutionHash(newConstitutionHash);
     }
 }
diff --git a/test/gov-actions/AIPXAction.t.sol b/test/gov-actions/AIPXAction.t.sol
@@ -12,14 +12,18 @@ contract AIPXActionTest is Test {
     ProxyAdmin proxyAdmin = ProxyAdmin(0xdb216562328215E010F819B5aBe947bad4ca961e);
     UpgradeExecutor arbOneUe = UpgradeExecutor(0xCF57572261c7c2BCF21ffD220ea7d1a27D40A827);
     address council = 0x423552c0F05baCCac5Bfa91C6dCF1dc53a0A1641;
+    IArbitrumDAOConstitution constitution = IArbitrumDAOConstitution(0x1D62fFeB72e4c360CcBbacf7c965153b00260417);
+    bytes32 newConstitutionHash = 0x0101010101010101010101010101010101010101010101010101010101010101;
 
     address newImplementation = address(new SecurityCouncilNomineeElectionGovernor());
     uint256 votingDelay = 7 days;
     AIPXAction action = new AIPXAction(
         address(proxyAdmin),
         address(gov),
         newImplementation,
-        votingDelay
+        votingDelay,
+        address(constitution),
+        newConstitutionHash
     );
 
     function testAction() external {
@@ -47,6 +51,12 @@ contract AIPXActionTest is Test {
             newImplementation,
             "implementation not set"
         );
+
+        assertEq(
+            constitution.constitutionHash(),
+            newConstitutionHash,
+            "constitution hash not set"
+        );
     }
 
     function _getImplementation() internal view returns (address) {