update snapshot module (#2254)

* update snapshot module * bump version * comment * comment --------- Co-authored-by: dbeal <[email protected]> Co-authored-by: Moss <[email protected]>
Synthetixio · Sep 3, 2024 · d03a0ed · d03a0ed
1 parent 09b8614
commit d03a0ed
Show file tree

Hide file tree

Showing 3 changed files with 31 additions and 31 deletions.
diff --git a/protocol/governance/contracts/modules/core/SnapshotVotePowerModule.sol b/protocol/governance/contracts/modules/core/SnapshotVotePowerModule.sol
@@ -29,7 +29,7 @@ contract SnapshotVotePowerModule is ISnapshotVotePowerModule {
         bool enabled
     ) external override {
         OwnableStorage.onlyOwner();
-        Council.onlyInPeriod(Epoch.ElectionPeriod.Administration);
+        Council.onlyInPeriods(Epoch.ElectionPeriod.Administration, Epoch.ElectionPeriod.Nomination);
 
         SnapshotVotePower.Data storage snapshotVotePower = SnapshotVotePower.load(snapshotContract);
         snapshotVotePower.enabled = enabled;

diff --git a/protocol/governance/package.json b/protocol/governance/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@synthetixio/governance",
-  "version": "3.0.7",
+  "version": "3.0.8",
   "description": "On-Chain elections for all Synthetix councils",
   "private": true,
   "scripts": {

diff --git a/protocol/governance/test/integration/Elections.test.ts b/protocol/governance/test/integration/Elections.test.ts
@@ -237,35 +237,35 @@ describe('SynthetixElectionModule - Elections', () => {
           }
         });
 
-        describe('when trying to set the snapshot contract', () => {
-          it('reverts', async () => {
-            const { mothership, satellite1, satellite2 } = chains;
-            await assertRevert(
-              mothership.GovernanceProxy.setSnapshotContract(
-                mothership.SnapshotRecordMock.address,
-                0,
-                true
-              ),
-              'NotCallableInCurrentPeriod'
-            );
-            await assertRevert(
-              satellite1.GovernanceProxy.setSnapshotContract(
-                satellite1.SnapshotRecordMock.address,
-                0,
-                true
-              ),
-              'NotCallableInCurrentPeriod'
-            );
-            await assertRevert(
-              satellite2.GovernanceProxy.setSnapshotContract(
-                satellite2.SnapshotRecordMock.address,
-                0,
-                true
-              ),
-              'NotCallableInCurrentPeriod'
-            );
-          });
-        });
+        // describe('when trying to set the snapshot contract', () => {
+        //   it('reverts', async () => {
+        //     const { mothership, satellite1, satellite2 } = chains;
+        //     await assertRevert(
+        //       mothership.GovernanceProxy.setSnapshotContract(
+        //         mothership.SnapshotRecordMock.address,
+        //         0,
+        //         true
+        //       ),
+        //       'NotCallableInCurrentPeriod'
+        //     );
+        //     await assertRevert(
+        //       satellite1.GovernanceProxy.setSnapshotContract(
+        //         satellite1.SnapshotRecordMock.address,
+        //         0,
+        //         true
+        //       ),
+        //       'NotCallableInCurrentPeriod'
+        //     );
+        //     await assertRevert(
+        //       satellite2.GovernanceProxy.setSnapshotContract(
+        //         satellite2.SnapshotRecordMock.address,
+        //         0,
+        //         true
+        //       ),
+        //       'NotCallableInCurrentPeriod'
+        //     );
+        //   });
+        // });
 
         it('simulate debt share data', async () => {
           const { mothership, satellite1, satellite2 } = chains;