Fix bug

NeokingdomDAO · Nov 1, 2023 · b643fb6 · b643fb6
1 parent 901de5d
commit b643fb6
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/contracts/ResolutionManager/ResolutionManagerBase.sol b/contracts/ResolutionManager/ResolutionManagerBase.sol
@@ -177,6 +177,14 @@ abstract contract ResolutionManagerBase {
             executionTo.length == executionData.length,
             "Resolution: length mismatch"
         );
+        require(
+            addressedContributor == address(0) ||
+                _shareholderRegistry.isAtLeast(
+                    _shareholderRegistry.CONTRIBUTOR_STATUS(),
+                    addressedContributor
+                ),
+            "Resolution: excluded address is not a contributor"
+        );
         uint256 resolutionId = _currentResolutionId++;
         emit ResolutionCreated(msg.sender, resolutionId);
 

diff --git a/test/ResolutionManager.ts b/test/ResolutionManager.ts
@@ -247,7 +247,7 @@ describe("Resolution", async () => {
         .withArgs(managingBoard.address, resolutionId);
     });
 
-    it.only("doesn't allow to create a resolution to exclude an address that is not a contributor from votin", async () => {
+    it("doesn't allow to create a resolution to exclude an address that is not a contributor from voting", async () => {
       await expect(
         resolution
           .connect(managingBoard)
@@ -258,7 +258,7 @@ describe("Resolution", async () => {
             [],
             nonContributor.address
           )
-      ).revertedWith("Resolution: address is not a contributor");
+      ).revertedWith("Resolution: excluded address is not a contributor");
     });
 
     it("doesn't allow non contributors to create an addressable resolution", async () => {