git.delta.rocks / unique-network / refs/commits / d4b9e94c5b52

--- a/tests/src/eth/allowlist.test.ts
+++ b/tests/src/eth/allowlist.test.ts
@@ -92,21 +92,47 @@
   });
 
   itEth('Collection allowlist can be added and removed by [cross] address', async ({helper}) => {
-    const owner = await helper.eth.createAccountWithBalance(donor);
-    const user = donor;
+    const owner = (await helper.eth.createAccountWithBalance(donor)).toLowerCase();
+    const [userSub] = await helper.arrange.createAccounts([10n], donor);
+    const userEth = await helper.eth.createAccountWithBalance(donor);
     
     const {collectionAddress, collectionId} = await helper.eth.createNFTCollection(owner, 'A', 'B', 'C');
     const collectionEvm = helper.ethNativeContract.collection(collectionAddress, 'nft', owner);
-    const userCross = helper.ethCrossAccount.fromKeyringPair(user);
+    const userCrossSub = helper.ethCrossAccount.fromKeyringPair(userSub);
+    const userCrossEth = helper.ethCrossAccount.fromAddress(userEth);
+    const ownerCrossEth = helper.ethCrossAccount.fromAddress(owner);
+    
+    // Can addToCollectionAllowListCross:
+    expect(await helper.collection.allowed(collectionId, {Substrate: userSub.address})).to.be.false;
+    await collectionEvm.methods.addToCollectionAllowListCross(userCrossSub).send({from: owner});
+    await collectionEvm.methods.addToCollectionAllowListCross(userCrossEth).send({from: owner});
+    await collectionEvm.methods.addToCollectionAllowListCross(ownerCrossEth).send({from: owner});
+    expect(await helper.collection.allowed(collectionId, {Substrate: userSub.address})).to.be.true;
+    expect(await helper.collection.allowed(collectionId, {Ethereum: userEth})).to.be.true;
+    expect(await collectionEvm.methods.allowlistedCross(userCrossSub).call({from: owner})).to.be.true;
+    expect(await collectionEvm.methods.allowlistedCross(userCrossEth).call({from: owner})).to.be.true;
+
+    await collectionEvm.methods.mint(userEth).send(); // token #1
+    await collectionEvm.methods.mint(userEth).send(); // token #2
+    await collectionEvm.methods.setCollectionAccess(1).send();
     
-    expect(await helper.collection.allowed(collectionId, {Substrate: user.address})).to.be.false;
-    await collectionEvm.methods.addToCollectionAllowListCross(userCross).send({from: owner});
-    expect(await helper.collection.allowed(collectionId, {Substrate: user.address})).to.be.true;
-    expect(await collectionEvm.methods.allowlistedCross(userCross).call({from: owner})).to.be.true;
+    // allowlisted account can transfer and transferCross:
+    await collectionEvm.methods.transfer(owner, 1).send({from: userEth});
+    await collectionEvm.methods.transferCross(userCrossSub, 2).send({from: userEth});
+    expect(await helper.nft.getTokenOwner(collectionId, 1)).to.deep.eq({Ethereum: owner});
+    expect(await helper.nft.getTokenOwner(collectionId, 2)).to.deep.eq({Substrate: userSub.address});
     
-    await collectionEvm.methods.removeFromCollectionAllowListCross(userCross).send({from: owner});
-    expect(await helper.collection.allowed(collectionId, {Substrate: user.address})).to.be.false;
-    expect(await collectionEvm.methods.allowlistedCross(userCross).call({from: owner})).to.be.false;
+    // can removeFromCollectionAllowListCross:
+    await collectionEvm.methods.removeFromCollectionAllowListCross(userCrossSub).send({from: owner});
+    await collectionEvm.methods.removeFromCollectionAllowListCross(userCrossEth).send({from: owner});
+    expect(await helper.collection.allowed(collectionId, {Substrate: userSub.address})).to.be.false;
+    expect(await helper.collection.allowed(collectionId, {Ethereum: userEth})).to.be.false;
+    expect(await collectionEvm.methods.allowlistedCross(userCrossSub).call({from: owner})).to.be.false;
+    expect(await collectionEvm.methods.allowlistedCross(userCrossEth).call({from: owner})).to.be.false;
+
+    // cannot transfer anymore
+    await collectionEvm.methods.mint(userEth).send();
+    await expect(collectionEvm.methods.transfer(owner, 2).send({from: userEth})).to.be.rejectedWith(/Transaction has been reverted/);
   });
 
   // Soft-deprecated

39    });39    });
40  });40  });
41
42  // Soft-deprecated
43  itEth('Add admin by owner', async ({helper}) => {
44    const owner = await helper.eth.createAccountWithBalance(donor);
45    const {collectionAddress, collectionId} = await helper.eth.createNFTCollection(owner, 'A', 'B', 'C');
46    const collectionEvm = helper.ethNativeContract.collection(collectionAddress, 'nft', owner, true);
47
48    const newAdmin = helper.eth.createAccount();
49
50    await collectionEvm.methods.addCollectionAdmin(newAdmin).send();
51    const adminList = await helper.callRpc('api.rpc.unique.adminlist', [collectionId]);
52    expect(adminList[0].asEthereum.toString().toLocaleLowerCase())
53      .to.be.eq(newAdmin.toLocaleLowerCase());
54  });
5541
56  itEth('Add cross account admin by owner', async ({helper, privateKey}) => {42  itEth('can add account admin by owner', async ({helper, privateKey}) => {
43    // arrange
57    const owner = await helper.eth.createAccountWithBalance(donor);44    const owner = await helper.eth.createAccountWithBalance(donor);
58        45    const adminSub = await privateKey('//admin2');
46    const adminEth = helper.eth.createAccount().toLowerCase();
47
48    const adminDeprecated = helper.eth.createAccount().toLowerCase();
49    const adminCrossSub = helper.ethCrossAccount.fromKeyringPair(adminSub);
50    const adminCrossEth = helper.ethCrossAccount.fromAddress(adminEth);
51        
59    const {collectionAddress, collectionId} = await helper.eth.createNFTCollection(owner, 'A', 'B', 'C');52    const {collectionAddress, collectionId} = await helper.eth.createNFTCollection(owner, 'A', 'B', 'C');
60    const collectionEvm = helper.ethNativeContract.collection(collectionAddress, 'nft', owner);53    const collectionEvm = helper.ethNativeContract.collection(collectionAddress, 'nft', owner, true);
61    54
55    // Soft-deprecated: can addCollectionAdmin 
62    const newAdmin = await privateKey('//Bob');56    await collectionEvm.methods.addCollectionAdmin(adminDeprecated).send();
57    // Can addCollectionAdminCross for substrate and ethereum address
63    const newAdminCross = helper.ethCrossAccount.fromKeyringPair(newAdmin);58    await collectionEvm.methods.addCollectionAdminCross(adminCrossSub).send();
64    await collectionEvm.methods.addCollectionAdminCross(newAdminCross).send();59    await collectionEvm.methods.addCollectionAdminCross(adminCrossEth).send();
6560
61    // 1. Expect api.rpc.unique.adminlist returns admins:
66    const adminList = await helper.collection.getAdmins(collectionId);62    const adminListRpc = await helper.collection.getAdmins(collectionId);
63    expect(adminListRpc).to.has.length(3);
67    expect(adminList).to.be.like([{Substrate: newAdmin.address}]);64    expect(adminListRpc).to.be.deep.contain.members([{Substrate: adminSub.address}, {Ethereum: adminEth}, {Ethereum: adminDeprecated}]);
65
66    // 2. Expect methods.collectionAdmins == api.rpc.unique.adminlist
67    let adminListEth = await collectionEvm.methods.collectionAdmins().call();
68    adminListEth = adminListEth.map((element: IEthCrossAccountId) => {
69      return helper.address.convertCrossAccountFromEthCrossAccount(element);
70    });
71    expect(adminListRpc).to.be.like(adminListEth);
68  });72  });
6973
70  itEth('Check adminlist', async ({helper, privateKey}) => {74  itEth('cross account admin can mint', async ({helper}) => {
75    // arrange: create collection and accounts
71    const owner = await helper.eth.createAccountWithBalance(donor);76    const owner = await helper.eth.createAccountWithBalance(donor);
72        
73    const {collectionAddress, collectionId} = await helper.eth.createNFTCollection(owner, 'A', 'B', 'C');77    const {collectionAddress, collectionId} = await helper.eth.createERC721MetadataCompatibleNFTCollection(owner, 'Mint collection', 'a', 'b', 'uri');
74    const collectionEvm = helper.ethNativeContract.collection(collectionAddress, 'nft', owner, true);78    const adminEth = (await helper.eth.createAccountWithBalance(donor)).toLowerCase();
75
76    const admin1 = helper.eth.createAccount();79    const adminCrossEth = helper.ethCrossAccount.fromAddress(adminEth);
77    const admin2 = await privateKey('admin');80    const [adminSub] = await helper.arrange.createAccounts([100n], donor);
78    const admin2Cross = helper.ethCrossAccount.fromKeyringPair(admin2);81    const adminCrossSub = helper.ethCrossAccount.fromKeyringPair(adminSub);
79    82    const collectionEvm = helper.ethNativeContract.collection(collectionAddress, 'nft', owner, true);
80    // Soft-deprecated83    
84    // cannot mint while not admin
81    await collectionEvm.methods.addCollectionAdmin(admin1).send();85    await expect(collectionEvm.methods.mint(owner).send({from: adminEth})).to.be.rejected;
86    await expect(helper.nft.mintToken(adminSub, {collectionId, owner: {Ethereum: owner}})).to.be.rejectedWith(/common.PublicMintingNotAllowed/);
87    
88    // admin (sub and eth) can mint token:
82    await collectionEvm.methods.addCollectionAdminCross(admin2Cross).send();89    await collectionEvm.methods.addCollectionAdminCross(adminCrossEth).send();
83
84    const adminListRpc = await helper.collection.getAdmins(collectionId);90    await collectionEvm.methods.addCollectionAdminCross(adminCrossSub).send();
85    let adminListEth = await collectionEvm.methods.collectionAdmins().call();91    await collectionEvm.methods.mint(owner).send({from: adminEth});
86    adminListEth = adminListEth.map((element: IEthCrossAccountId) => {92    await helper.nft.mintToken(adminSub, {collectionId, owner: {Ethereum: owner}});
87      return helper.address.convertCrossAccountFromEthCrossAcoount(element);93
88    });
89    expect(adminListRpc).to.be.like(adminListEth);94    expect(await helper.collection.getLastTokenId(collectionId)).to.eq(2);
90  });95  });
9196
92  // Soft-deprecated
93  itEth('Verify owner or admin', async ({helper}) => {97  itEth('cannot add invalid cross account admin', async ({helper}) => {
94    const owner = await helper.eth.createAccountWithBalance(donor);98    const owner = await helper.eth.createAccountWithBalance(donor);
99    const [admin] = await helper.arrange.createAccounts([100n, 100n], donor);
100
95    const {collectionAddress} = await helper.eth.createNFTCollection(owner, 'A', 'B', 'C');101    const {collectionAddress} = await helper.eth.createNFTCollection(owner, 'A', 'B', 'C');
96
97    const newAdmin = helper.eth.createAccount();
98    const collectionEvm = helper.ethNativeContract.collection(collectionAddress, 'nft', owner, true);102    const collectionEvm = helper.ethNativeContract.collection(collectionAddress, 'nft', owner);
99  103
100    expect(await collectionEvm.methods.isOwnerOrAdmin(newAdmin).call()).to.be.false;104    const adminCross = {
101    await collectionEvm.methods.addCollectionAdmin(newAdmin).send();105      eth: helper.address.substrateToEth(admin.address),
106      sub: admin.addressRaw,
107    };
102    expect(await collectionEvm.methods.isOwnerOrAdmin(newAdmin).call()).to.be.true;108    await expect(collectionEvm.methods.addCollectionAdminCross(adminCross).send()).to.be.rejected;
103  });109  });
104110
105  itEth('Verify owner or admin cross', async ({helper, privateKey}) => {111  itEth('can verify owner with methods.isOwnerOrAdmin[Cross]', async ({helper, privateKey}) => {
106    const owner = await helper.eth.createAccountWithBalance(donor);112    const owner = await helper.eth.createAccountWithBalance(donor);
107    const {collectionAddress} = await helper.eth.createNFTCollection(owner, 'A', 'B', 'C');113    const {collectionAddress} = await helper.eth.createNFTCollection(owner, 'A', 'B', 'C');
108114
115    const adminDeprecated = helper.eth.createAccount();
109    const newAdmin = await privateKey('admin');116    const admin1Cross = helper.ethCrossAccount.fromKeyringPair(await privateKey('admin'));
110    const newAdminCross = helper.ethCrossAccount.fromKeyringPair(newAdmin);117    const admin2Cross = helper.ethCrossAccount.fromAddress(helper.address.substrateToEth((await privateKey('admin3')).address));
111    const collectionEvm = helper.ethNativeContract.collection(collectionAddress, 'nft', owner);118    const collectionEvm = helper.ethNativeContract.collection(collectionAddress, 'nft', owner, true);
112  119  
120    // Soft-deprecated:
121    expect(await collectionEvm.methods.isOwnerOrAdmin(adminDeprecated).call()).to.be.false;
113    expect(await collectionEvm.methods.isOwnerOrAdminCross(newAdminCross).call()).to.be.false;122    expect(await collectionEvm.methods.isOwnerOrAdminCross(admin1Cross).call()).to.be.false;
123    expect(await collectionEvm.methods.isOwnerOrAdminCross(admin2Cross).call()).to.be.false;
124
125    await collectionEvm.methods.addCollectionAdmin(adminDeprecated).send();
114    await collectionEvm.methods.addCollectionAdminCross(newAdminCross).send();126    await collectionEvm.methods.addCollectionAdminCross(admin1Cross).send();
127    await collectionEvm.methods.addCollectionAdminCross(admin2Cross).send();
128
129    // Soft-deprecated: isOwnerOrAdmin returns true
130    expect(await collectionEvm.methods.isOwnerOrAdmin(adminDeprecated).call()).to.be.true;
131    // Expect isOwnerOrAdminCross return true
115    expect(await collectionEvm.methods.isOwnerOrAdminCross(newAdminCross).call()).to.be.true;132    expect(await collectionEvm.methods.isOwnerOrAdminCross(admin1Cross).call()).to.be.true;
133    expect(await collectionEvm.methods.isOwnerOrAdminCross(admin2Cross).call()).to.be.true;
116  });134  });
117135
118  // Soft-deprecated136  // Soft-deprecated
154    const owner = await helper.eth.createAccountWithBalance(donor);172    const owner = await helper.eth.createAccountWithBalance(donor);
155    const {collectionAddress, collectionId} = await helper.eth.createNFTCollection(owner, 'A', 'B', 'C');173    const {collectionAddress, collectionId} = await helper.eth.createNFTCollection(owner, 'A', 'B', 'C');
156174
157    const [admin] = await helper.arrange.createAccounts([10n], donor);175    const [admin, notAdmin] = await helper.arrange.createAccounts([10n, 10n], donor);
158    const adminCross = helper.ethCrossAccount.fromKeyringPair(admin);176    const adminCross = helper.ethCrossAccount.fromKeyringPair(admin);
159    const collectionEvm = helper.ethNativeContract.collection(collectionAddress, 'nft', owner);177    const collectionEvm = helper.ethNativeContract.collection(collectionAddress, 'nft', owner);
160    await collectionEvm.methods.addCollectionAdminCross(adminCross).send();178    await collectionEvm.methods.addCollectionAdminCross(adminCross).send();
161179
162    const [notAdmin] = await helper.arrange.createAccounts([10n], donor);
163    const notAdminCross = helper.ethCrossAccount.fromKeyringPair(notAdmin);180    const notAdminCross = helper.ethCrossAccount.fromKeyringPair(notAdmin);
164    await expect(collectionEvm.methods.addCollectionAdminCross(notAdminCross).call({from: adminCross.eth}))181    await expect(collectionEvm.methods.addCollectionAdminCross(notAdminCross).call({from: adminCross.eth}))
165      .to.be.rejectedWith('NoPermission');182      .to.be.rejectedWith('NoPermission');

--- a/tests/src/eth/fungible.test.ts
+++ b/tests/src/eth/fungible.test.ts
@@ -314,7 +314,7 @@
     }
   });
 
-  itEth('Can perform transferFromCross()', async ({helper, privateKey}) => {
+  itEth('Can perform transferFromCross()', async ({helper}) => {
     const sender = await helper.eth.createAccountWithBalance(donor, 100n);
 
     const collection = await helper.ft.mintCollection(owner, {name: 'A', description: 'B', tokenPrefix: 'C'}, 0);
@@ -508,7 +508,7 @@
     expect(event.returnValues.value).to.be.equal('51');
   });
 
-  itEth('Events emitted for transferFromCross()', async ({helper, privateKey}) => {
+  itEth('Events emitted for transferFromCross()', async ({helper}) => {
     const sender = await helper.eth.createAccountWithBalance(donor, 100n);
 
     const collection = await helper.ft.mintCollection(owner, {name: 'A', description: 'B', tokenPrefix: 'C'}, 0);

--- a/tests/src/eth/nonFungible.test.ts
+++ b/tests/src/eth/nonFungible.test.ts
@@ -17,7 +17,6 @@
 import {itEth, usingEthPlaygrounds, expect, EthUniqueHelper} from './util';
 import {IKeyringPair} from '@polkadot/types/types';
 import {Contract} from 'web3-eth-contract';
-import exp from 'constants';
 
 
 describe('NFT: Information getting', () => {
@@ -280,30 +279,51 @@
   });
 
   itEth('Can perform approveCross()', async ({helper}) => {
+    // arrange: create accounts
+    const owner = await helper.eth.createAccountWithBalance(donor, 100n);
+    const ownerCross = helper.ethCrossAccount.fromAddress(owner);
+    const receiverSub = charlie;
+    const recieverCrossSub = helper.ethCrossAccount.fromKeyringPair(receiverSub);
+    const receiverEth = await helper.eth.createAccountWithBalance(donor, 100n);
+    const receiverCrossEth = helper.ethCrossAccount.fromAddress(receiverEth);
+
+    // arrange: create collection and tokens:
     const collection = await helper.nft.mintCollection(minter, {name: 'A', description: 'B', tokenPrefix: 'C'});
+    const token1 = await collection.mintToken(minter, {Ethereum: owner});
+    const token2 = await collection.mintToken(minter, {Ethereum: owner});
 
-    const owner = await helper.eth.createAccountWithBalance(donor, 100n);
-    const receiver = charlie;
+    const collectionEvm = helper.ethNativeContract.collection(helper.ethAddress.fromCollectionId(collection.collectionId), 'nft');
 
-    const token = await collection.mintToken(minter, {Ethereum: owner});
-
-    const address = helper.ethAddress.fromCollectionId(collection.collectionId);
-    const contract = helper.ethNativeContract.collection(address, 'nft');
+    // Can approveCross substrate and ethereum address:
+    const resultSub = await collectionEvm.methods.approveCross(recieverCrossSub, token1.tokenId).send({from: owner});
+    const resultEth = await collectionEvm.methods.approveCross(receiverCrossEth, token2.tokenId).send({from: owner});
+    const eventSub = resultSub.events.Approval;
+    const eventEth = resultEth.events.Approval;
+    expect(eventSub).to.be.like({
+      address: helper.ethAddress.fromCollectionId(collection.collectionId),
+      event: 'Approval',
+      returnValues: {
+        owner,
+        approved: helper.address.substrateToEth(receiverSub.address),
+        tokenId: token1.tokenId.toString(),
+      },
+    });
+    expect(eventEth).to.be.like({
+      address: helper.ethAddress.fromCollectionId(collection.collectionId),
+      event: 'Approval',
+      returnValues: {
+        owner,
+        approved: receiverEth,
+        tokenId: token2.tokenId.toString(),
+      },
+    });
 
-    {
-      const recieverCross = helper.ethCrossAccount.fromKeyringPair(receiver);
-      const result = await contract.methods.approveCross(recieverCross, token.tokenId).send({from: owner});
-      const event = result.events.Approval;
-      expect(event).to.be.like({
-        address: helper.ethAddress.fromCollectionId(collection.collectionId),
-        event: 'Approval',
-        returnValues: {
-          owner,
-          approved: helper.address.substrateToEth(receiver.address),
-          tokenId: token.tokenId.toString(),
-        },
-      });
-    }
+    // Substrate address can transferFrom approved tokens:
+    await helper.nft.transferTokenFrom(receiverSub, collection.collectionId, token1.tokenId, {Ethereum: owner}, {Substrate: receiverSub.address});
+    expect(await helper.nft.getTokenOwner(collection.collectionId, token1.tokenId)).to.deep.eq({Substrate: receiverSub.address});
+    // Ethereum address can transferFromCross approved tokens:
+    await collectionEvm.methods.transferFromCross(ownerCross, receiverCrossEth, token2.tokenId).send({from: receiverEth});
+    expect(await helper.nft.getTokenOwner(collection.collectionId, token2.tokenId)).to.deep.eq({Ethereum: receiverEth.toLowerCase()});
   });
 
   itEth('Can perform transferFrom()', async ({helper}) => {
@@ -340,13 +360,11 @@
     }
   });
 
-  itEth('Can perform transferFromCross()', async ({helper, privateKey}) => {
-    const minter = await privateKey('//Alice');
+  itEth('Can perform transferFromCross()', async ({helper}) => {
     const collection = await helper.nft.mintCollection(minter, {name: 'A', description: 'B', tokenPrefix: 'C'});
 
-    const owner = await privateKey('//Bob');
+    const [owner, receiver] = await helper.arrange.createAccounts([100n, 100n], donor);
     const spender = await helper.eth.createAccountWithBalance(donor);
-    const receiver = await privateKey('//Charlie');
 
     const token = await collection.mintToken(minter, {Substrate: owner.address});
 
@@ -501,7 +519,7 @@
     expect(cost < BigInt(0.2 * Number(helper.balance.getOneTokenNominal())));
   });
 
-  itEth('Can perform transferFromCross()', async ({helper, privateKey}) => {
+  itEth('Can perform transferFromCross()', async ({helper}) => {
     const collectionMinter = alice;
     const owner = bob;
     const receiver = charlie;

--- a/tests/src/util/playgrounds/unique.ts
+++ b/tests/src/util/playgrounds/unique.ts
@@ -2429,7 +2429,7 @@
    * @param ethCrossAccount etherium cross account
    * @returns substrate cross account id
    */
-  convertCrossAccountFromEthCrossAcoount(ethCrossAccount: IEthCrossAccountId): ICrossAccountId {
+  convertCrossAccountFromEthCrossAccount(ethCrossAccount: IEthCrossAccountId): ICrossAccountId {
     if (ethCrossAccount.sub === '0') {
       return {Ethereum: ethCrossAccount.eth.toLocaleLowerCase()};
     }