Add test & remove withdraw fund feature

bitmark-inc · Jul 17, 2023 · bd4be38 · bd4be38
1 parent e220d1b
commit bd4be38
Show file tree

Hide file tree

Showing 2 changed files with 26 additions and 5 deletions.
diff --git a/contracts/FeralfileArtworkV4.sol b/contracts/FeralfileArtworkV4.sol
@@ -607,11 +607,6 @@ contract FeralfileExhibitionV4 is
         );
     }
 
-    /// @notice withdraw all fund
-    function withdrawFunds() external onlyOwner {
-        payable(msg.sender).transfer(address(this).balance);
-    }
-
     /// @notice Event emitted when new Artwork has been minted
     event NewArtwork(
         address indexed owner,

diff --git a/test/feralfile_exhibition_v4.js b/test/feralfile_exhibition_v4.js
@@ -66,6 +66,32 @@ contract("FeralfileExhibitionV4_0", async (accounts) => {
         assert.equal(seriesTotalSupply1, 0);
     });
 
+    it("test duplicate series in constructor", async function () {
+        // Deploy contract with duplicate series defined 
+        let seriesIds = [0, 1, 2, 3, 1];
+        let seriesMaxSupply = [1, 1, 100, 1000, 10000];
+        try {
+            await FeralfileExhibitionV4.new(
+                "Feral File V4 Test",
+                "FFv4",
+                true,
+                true,
+                this.signer,
+                this.vault.address,
+                COST_RECEIVER,
+                CONTRACT_URI,
+                seriesIds,
+                seriesMaxSupply
+            );
+        } catch (error) {
+            assert.ok(
+                error.message.includes(
+                    "FeralfileExhibitionV4: duplicate seriesId"
+                )
+            );
+        }
+    })
+
     it("test mint artwork", async function () {
         const contract = this.contracts[0];