Merge pull request #108 from Erebos0506/master

Collection max

Author: Yuripetusko

src/rmrk2.0.0/tools/consolidator/interactions/mint.ts

@@ -23,6 +23,14 @@ export const validateMintNFT = async (
       `Attempted issue of NFT in non-owned collection. Issuer: ${nftParentCollection.issuer}, caller: ${remark.caller}`
     );
   }
+  
+  const nfts = await dbAdapter.getNFTsByCollection(nftParentCollection.id);
+  const unburnedNfts = nfts ? nfts.filter((nft) => nft.burned === "") : [];
+  if (nftParentCollection.max <= unburnedNfts.length && nftParentCollection.max !== 0) {
+    throw new Error(
+      `Attempted to mint into maxed out collection ${nftParentCollection.id}`
+    );
+  }
 
   if (nft.owner) {
     if (!isValidAddressPolkadotAddress(nft.owner)) {

test/2.0.0/consolidator/mint.test.ts

@@ -2,6 +2,7 @@ import { Consolidator, NFT } from "../../../src/rmrk2.0.0";
 import {
   createCollectionMock,
   getBlockCallsMock,
+  getAliceKey,
   getBobKey,
   getRemarksFromBlocksMock,
   mintNftMock,
@@ -100,4 +101,20 @@ describe("rmrk2.0.0 Consolidator: MINT", () => {
     const consolidator = new Consolidator();
     expect(await consolidator.consolidate(remarks)).toMatchSnapshot();
   });
+  
+  it("Should prevent minting NFT into locked collection", async () => {
+    const remarks = getRemarksFromBlocksMock([
+      ...getSetupRemarks(),
+      ...getBlockCallsMock(
+        createCollectionMock(2).lock(),
+        getAliceKey().address
+      ),
+      ...getBlockCallsMock(mintNftMock2().mint()),
+    ]);
+    const consolidator = new Consolidator();
+    const consolidated = await consolidator.consolidate(remarks);
+    expect(consolidated.invalid[0].message).toEqual(
+      "Attempted to mint into maxed out collection d43593c715a56da27d-KANARIABIRDS"
+    );
+  });
 });