difftreelog
Merge pull request #954 from UniqueNetwork/feature/update_market_v2_contract
in: master
4 files changed
tests/package.jsondiffbeforeafterboth--- a/tests/package.json
+++ b/tests/package.json
@@ -47,6 +47,7 @@
"testEthNesting": "yarn _test './**/eth/nesting/**/*.*test.ts'",
"testEthFractionalizer": "yarn _test './**/eth/fractionalizer/**/*.*test.ts'",
"testEthMarketplace": "yarn _test './**/eth/marketplace/**/*.*test.ts'",
+ "testEthMarket": "yarn _test './**/eth/marketplace-v2/**/*.*test.ts'",
"testSub": "yarn _test './**/sub/**/*.*test.ts'",
"testSubNesting": "yarn _test './**/sub/nesting/**/*.*test.ts'",
"testEvent": "yarn _test ./src/check-event/*.*test.ts",
tests/src/eth/marketplace-v2/Market.soldiffbeforeafterboth--- a/tests/src/eth/marketplace-v2/Market.sol
+++ b/tests/src/eth/marketplace-v2/Market.sol
@@ -1,13 +1,16 @@
// SPDX-License-Identifier: UNLICENSED
pragma solidity 0.8.17;
+import "@openzeppelin/contracts/security/ReentrancyGuard.sol";
import "@openzeppelin/contracts/utils/introspection/ERC165Checker.sol";
import "@openzeppelin/contracts/token/ERC721/IERC721.sol";
+import "@openzeppelin/contracts/access/Ownable.sol";
import { UniqueNFT, CrossAddress } from "@unique-nft/solidity-interfaces/contracts/UniqueNFT.sol";
+import { UniqueFungible, CrossAddress as CrossAddressF } from "@unique-nft/solidity-interfaces/contracts/UniqueFungible.sol";
import "@unique-nft/solidity-interfaces/contracts/CollectionHelpers.sol";
import "./royalty/UniqueRoyaltyHelper.sol";
-contract Market {
+contract Market is Ownable, ReentrancyGuard {
using ERC165Checker for address;
struct Order {
@@ -20,6 +23,7 @@
}
uint32 public constant version = 0;
+ uint32 public constant buildVersion = 3;
bytes4 private constant InterfaceId_ERC721 = 0x80ac58cd;
bytes4 private constant InterfaceId_ERC165 = 0x5755c3f2;
CollectionHelpers private constant collectionHelpers =
@@ -29,7 +33,6 @@
uint32 private idCount = 1;
uint32 public marketFee;
uint64 public ctime;
- address selfAddress;
address public ownerAddress;
mapping(address => bool) public admins;
@@ -55,15 +58,11 @@
error OrderNotFound();
error TooManyAmountRequested();
error NotEnoughMoneyError();
+ error InvalidRoyaltiesError(uint256 totalRoyalty);
error FailTransferToken(string reason);
-
- modifier onlyOwner() {
- require(msg.sender == ownerAddress, "Only owner can");
- _;
- }
modifier onlyAdmin() {
- require(msg.sender == ownerAddress || admins[msg.sender], "Only admin can");
+ require(msg.sender == this.owner() || admins[msg.sender], "Only admin can");
_;
}
@@ -83,13 +82,22 @@
marketFee = fee;
ctime = timestamp;
- if (marketFee == 0 || marketFee >= 100) {
+ if (marketFee >= 100) {
revert InvalidMarketFee();
}
+ }
+
+ /**
+ * Fallback that allows this contract to receive native token.
+ * We need this for self-sponsoring
+ */
+ fallback() external payable {}
- ownerAddress = msg.sender;
- selfAddress = address(this);
- }
+ /**
+ * Receive also allows this contract to receive native token.
+ * We need this for self-sponsoring
+ */
+ receive() external payable {}
function getErc721(uint32 collectionId) private view returns (IERC721) {
address collectionAddress = collectionHelpers.collectionAddress(
@@ -110,36 +118,35 @@
}
return IERC721(collectionAddress);
- }
-
- // ################################################################
- // Set new contract owner #
- // ################################################################
-
- function setOwner() public onlyOwner {
- ownerAddress = msg.sender;
}
-
- // ################################################################
- // Add new admin #
- // ################################################################
+ /**
+ * Add new admin. Only owner or an existing admin can add admins.
+ *
+ * @param admin: Address of a new admin to add
+ */
function addAdmin(address admin) public onlyAdmin {
admins[admin] = true;
}
- // ################################################################
- // Remove admin #
- // ################################################################
-
+ /**
+ * Remove an admin. Only owner or an existing admin can remove admins.
+ *
+ * @param admin: Address of a new admin to add
+ */
function removeAdmin(address admin) public onlyAdmin {
delete admins[admin];
}
- // ################################################################
- // Place a token for sale #
- // ################################################################
-
+ /**
+ * Place an NFT or RFT token for sale. It must be pre-approved for transfers by this contract address.
+ *
+ * @param collectionId: ID of the token collection
+ * @param tokenId: ID of the token
+ * @param price: Price (with proper network currency decimals)
+ * @param amount: Number of token fractions to list (must always be 1 for NFT)
+ * @param seller: The seller cross-address (the beneficiary account to receive payment, may be different from transaction sender)
+ */
function put(
uint32 collectionId,
uint32 tokenId,
@@ -164,7 +171,7 @@
revert SellerIsNotOwner();
}
- if (erc721.getApproved(tokenId) != selfAddress) {
+ if (erc721.getApproved(tokenId) != address(this)) {
revert TokenIsNotApproved();
}
@@ -182,11 +189,14 @@
emit TokenIsUpForSale(version, order);
}
-
- // ################################################################
- // Get order #
- // ################################################################
+ /**
+ * Get information about the listed token order
+ *
+ * @param collectionId: ID of the token collection
+ * @param tokenId: ID of the token
+ * @return The order information
+ */
function getOrder(
uint32 collectionId,
uint32 tokenId
@@ -194,10 +204,13 @@
return orders[collectionId][tokenId];
}
- // ################################################################
- // Revoke the token from the sale #
- // ################################################################
-
+ /**
+ * Revoke the token from the sale. Only the original lister can use this method.
+ *
+ * @param collectionId: ID of the token collection
+ * @param tokenId: ID of the token
+ * @param amount: Number of token fractions to de-list (must always be 1 for NFT)
+ */
function revoke(
uint32 collectionId,
uint32 tokenId,
@@ -239,10 +252,12 @@
emit TokenRevoke(version, order, amount);
}
- // ################################################################
- // Check approved #
- // ################################################################
-
+ /**
+ * Test if the token is still approved to be transferred by this contract and delete the order if not.
+ *
+ * @param collectionId: ID of the token collection
+ * @param tokenId: ID of the token
+ */
function checkApproved(uint32 collectionId, uint32 tokenId) public onlyAdmin {
Order memory order = orders[collectionId][tokenId];
if (order.price == 0) {
@@ -251,7 +266,7 @@
IERC721 erc721 = getErc721(collectionId);
- if (erc721.getApproved(tokenId) != selfAddress) {
+ if (erc721.getApproved(tokenId) != address(this) || erc721.ownerOf(tokenId) != getAddressFromCrossAccount(order.seller)) {
uint32 amount = order.amount;
order.amount = 0;
emit TokenRevoke(version, order, amount);
@@ -262,16 +277,47 @@
}
}
- // ################################################################
- // Buy a token #
- // ################################################################
+ function getAddressFromCrossAccount(CrossAddress memory account) private pure returns (address) {
+ if (account.eth != address(0)) {
+ return account.eth;
+ } else {
+ return address(uint160(account.sub >> 96));
+ }
+ }
+
+ /**
+ * Revoke the token from the sale. Only the contract admin can use this method.
+ *
+ * @param collectionId: ID of the token collection
+ * @param tokenId: ID of the token
+ */
+ function revokeAdmin(uint32 collectionId, uint32 tokenId) public onlyAdmin {
+ Order memory order = orders[collectionId][tokenId];
+ if (order.price == 0) {
+ revert OrderNotFound();
+ }
+ uint32 amount = order.amount;
+ order.amount = 0;
+ emit TokenRevoke(version, order, amount);
+
+ delete orders[collectionId][tokenId];
+ }
+
+ /**
+ * Buy a token (partially for an RFT).
+ *
+ * @param collectionId: ID of the token collection
+ * @param tokenId: ID of the token
+ * @param amount: Number of token fractions to buy (must always be 1 for NFT)
+ * @param buyer: Cross-address of the buyer, eth part must be equal to the transaction signer address
+ */
function buy(
uint32 collectionId,
uint32 tokenId,
uint32 amount,
CrossAddress memory buyer
- ) public payable validCrossAddress(buyer.eth, buyer.sub) {
+ ) public payable validCrossAddress(buyer.eth, buyer.sub) nonReentrant {
if (msg.value == 0) {
revert InvalidArgument("msg.value must not be zero");
}
@@ -296,7 +342,7 @@
}
IERC721 erc721 = getErc721(order.collectionId);
- if (erc721.getApproved(tokenId) != selfAddress) {
+ if (erc721.getApproved(tokenId) != address(this)) {
revert TokenIsNotApproved();
}
@@ -316,26 +362,30 @@
order.tokenId
);
- (uint256 totalRoyalty, RoyaltyAmount[] memory royalties) = sendRoyalties(collectionAddress, tokenId, totalValue);
+ (uint256 totalRoyalty, RoyaltyAmount[] memory royalties) = sendRoyalties(collectionAddress, tokenId, totalValue - feeValue);
+ if (totalRoyalty >= totalValue - feeValue) {
+ revert InvalidRoyaltiesError(totalRoyalty);
+ }
+
sendMoney(order.seller, totalValue - feeValue - totalRoyalty);
if (msg.value > totalValue) {
- // todo, send money to signer or buyer ?
- payable(msg.sender).transfer(msg.value - totalValue);
+ sendMoney(buyer, msg.value - totalValue);
}
emit TokenIsPurchased(version, order, amount, buyer, royalties);
}
function sendMoney(CrossAddress memory to, uint256 money) private {
- address payable eth;
- if (to.eth != address(0)) {
- eth = payable(to.eth);
- } else {
- eth = payable(address(uint160(to.sub >> 96)));
- }
- eth.transfer(money);
+ address collectionAddress = collectionHelpers.collectionAddress(0);
+
+ UniqueFungible fungible = UniqueFungible(collectionAddress);
+
+ CrossAddressF memory fromF = CrossAddressF(address(this), 0);
+ CrossAddressF memory toF = CrossAddressF(to.eth, to.sub);
+
+ fungible.transferFromCross(fromF, toF, money);
}
function sendRoyalties(address collection, uint tokenId, uint sellPrice) private returns (uint256, RoyaltyAmount[] memory) {
@@ -355,7 +405,7 @@
}
function withdraw(address transferTo) public onlyOwner {
- uint256 balance = selfAddress.balance;
+ uint256 balance = address(this).balance;
if (balance > 0) {
payable(transferTo).transfer(balance);
tests/src/eth/marketplace-v2/marketplace.test.tsdiffbeforeafterboth161617import {IKeyringPair} from '@polkadot/types/types';17import {IKeyringPair} from '@polkadot/types/types';18import {readFile} from 'fs/promises';18import {readFile} from 'fs/promises';19import {EthUniqueHelper, itEth, usingEthPlaygrounds} from '../util';19import {EthUniqueHelper, SponsoringMode, itEth, usingEthPlaygrounds} from '../util';20import {makeNames} from '../../util';20import {makeNames} from '../../util';21import {expect} from 'chai';21import {expect} from 'chai';22import Web3 from 'web3';22import Web3 from 'web3';232324const {dirname} = makeNames(import.meta.url);24const {dirname} = makeNames(import.meta.url);2526const MARKET_FEE = 1;252726describe('Market V2 Contract', () => {28describe('Market V2 Contract', () => {27 let donor: IKeyringPair;29 let donor: IKeyringPair;42 solPath: '@unique-nft/solidity-interfaces/contracts/UniqueNFT.sol',44 solPath: '@unique-nft/solidity-interfaces/contracts/UniqueNFT.sol',43 fsPath: `${dirname}/../api/UniqueNFT.sol`,45 fsPath: `${dirname}/../api/UniqueNFT.sol`,44 },46 },47 {48 solPath: '@unique-nft/solidity-interfaces/contracts/UniqueFungible.sol',49 fsPath: `${dirname}/../api/UniqueFungible.sol`,50 },45 {51 {46 solPath: '@openzeppelin/contracts/utils/introspection/IERC165.sol',52 solPath: '@openzeppelin/contracts/utils/introspection/IERC165.sol',47 fsPath: `${dirname}/../../../node_modules/@openzeppelin/contracts/utils/introspection/IERC165.sol`,53 fsPath: `${dirname}/../../../node_modules/@openzeppelin/contracts/utils/introspection/IERC165.sol`,48 },54 },55 {56 solPath: '@openzeppelin/contracts/access/Ownable.sol',57 fsPath: `${dirname}/../../../node_modules/@openzeppelin/contracts/access/Ownable.sol`,58 },59 {60 solPath: '@openzeppelin/contracts/utils/Context.sol',61 fsPath: `${dirname}/../../../node_modules/@openzeppelin/contracts/utils/Context.sol`,62 },63 {64 solPath: '@openzeppelin/contracts/security/ReentrancyGuard.sol',65 fsPath: `${dirname}/../../../node_modules/@openzeppelin/contracts/security/ReentrancyGuard.sol`,66 },49 {67 {50 solPath: '@openzeppelin/contracts/utils/introspection/ERC165Checker.sol',68 solPath: '@openzeppelin/contracts/utils/introspection/ERC165Checker.sol',51 fsPath: `${dirname}/../../../node_modules/@openzeppelin/contracts/utils/introspection/ERC165Checker.sol`,69 fsPath: `${dirname}/../../../node_modules/@openzeppelin/contracts/utils/introspection/ERC165Checker.sol`,72 },90 },73 ],91 ],74 15000000,92 15000000,75 [1, 0],93 [MARKET_FEE, 0],76 );94 );77 }95 }789690 });108 });9110992 itEth('Put + Buy [eth]', async ({helper}) => {110 itEth('Put + Buy [eth]', async ({helper}) => {111 const ONE_TOKEN = helper.balance.getOneTokenNominal();112 const PRICE = 2n * ONE_TOKEN; // 2 UNQ93 const marketOwner = await helper.eth.createAccountWithBalance(donor, 600n);113 const marketOwner = await helper.eth.createAccountWithBalance(donor, 60000n);94 const market = await deployMarket(helper, marketOwner);114 const market = await deployMarket(helper, marketOwner);115 const contractHelpers = helper.ethNativeContract.contractHelpers(marketOwner);116117 // Set external sponsoring118 await contractHelpers.methods.setSponsor(market.options.address, marketOwner).send({from: marketOwner});119 await contractHelpers.methods.confirmSponsorship(market.options.address).send({from: marketOwner});120121 // Configure sponsoring122 await contractHelpers.methods.setSponsoringMode(market.options.address, SponsoringMode.Generous).send({from: marketOwner});123 await contractHelpers.methods.setSponsoringRateLimit(market.options.address, 0).send({from: marketOwner});9512496 const {collectionId, collectionAddress} = await helper.eth.createNFTCollection(marketOwner, 'Sponsor', 'absolutely anything', 'ROC');125 const {collectionId, collectionAddress} = await helper.eth.createNFTCollection(marketOwner, 'Sponsor', 'absolutely anything', 'ROC');97 const collection = await helper.ethNativeContract.collection(collectionAddress, 'nft', marketOwner);126 const collection = helper.ethNativeContract.collection(collectionAddress, 'nft', marketOwner, true);127128 // Set collection sponsoring129 await collection.methods.setCollectionSponsor(marketOwner).send({from: marketOwner});130 await collection.methods.confirmCollectionSponsorship().send({from: marketOwner});9813199 const sellerCross = await helper.ethCrossAccount.createAccountWithBalance(donor, 600n);132 const sellerCross = helper.ethCrossAccount.createAccount();100 const result = await collection.methods.mintCross(sellerCross, []).send();133 const result = await collection.methods.mintCross(sellerCross, []).send();101 const tokenId = result.events.Transfer.returnValues.tokenId;134 const tokenId = result.events.Transfer.returnValues.tokenId;102 await collection.methods.approve(market.options.address, tokenId).send({from: sellerCross.eth});135 await collection.methods.approve(market.options.address, tokenId).send({from: sellerCross.eth});136137 // Seller has no funds at all, his transactions are sponsored138 const sellerBalance = await helper.balance.getEthereum(sellerCross.eth);139 expect(sellerBalance).to.be.eq(0n);103140104 const putResult = await market.methods.put(collectionId, tokenId, 1, 1, sellerCross).send({from: sellerCross.eth});141 const putResult = await market.methods.put(collectionId, tokenId, PRICE.toString(), 1, sellerCross).send({142 from: sellerCross.eth, gasLimit: 1_000_000,143 });105 expect(putResult.events.TokenIsUpForSale).is.not.undefined;144 expect(putResult.events.TokenIsUpForSale).is.not.undefined;145146 // Seller balance are still 0147 const sellerBalanceAfter = await helper.balance.getEthereum(sellerCross.eth);148 expect(sellerBalanceAfter).to.be.eq(0n);149106 let ownerCross = await collection.methods.ownerOfCross(tokenId).call();150 let ownerCross = await collection.methods.ownerOfCross(tokenId).call();107 expect(ownerCross.eth).to.be.eq(sellerCross.eth);151 expect(ownerCross.eth).to.be.eq(sellerCross.eth);108 expect(ownerCross.sub).to.be.eq(sellerCross.sub);152 expect(ownerCross.sub).to.be.eq(sellerCross.sub);109153110 const buyerCross = await helper.ethCrossAccount.createAccountWithBalance(donor, 600n);154 const buyerCross = await helper.ethCrossAccount.createAccountWithBalance(donor, 10n);155156 // Buyer has only 10 UNQ157 const buyerBalance = await helper.balance.getEthereum(buyerCross.eth);158 expect(buyerBalance).to.be.eq(10n * ONE_TOKEN);159111 const buyResult = await market.methods.buy(collectionId, tokenId, 1, buyerCross).send({from: buyerCross.eth, value: 1});160 const buyResult = await market.methods.buy(collectionId, tokenId, 1, buyerCross).send({from: buyerCross.eth, value: PRICE.toString(), gasLimit: 1_000_000});112 expect(buyResult.events.TokenIsPurchased).is.not.undefined;161 expect(buyResult.events.TokenIsPurchased).is.not.undefined;162163 // Buyer pays only value, transaction use sponsoring164 const buyerBalanceAfter = await helper.balance.getEthereum(buyerCross.eth);165 expect(buyerBalanceAfter).to.be.eq(10n * ONE_TOKEN - PRICE);166113 ownerCross = await collection.methods.ownerOfCross(tokenId).call();167 ownerCross = await collection.methods.ownerOfCross(tokenId).call();114 expect(ownerCross.eth).to.be.eq(buyerCross.eth);168 expect(ownerCross.eth).to.be.eq(buyerCross.eth);115 expect(ownerCross.sub).to.be.eq(buyerCross.sub);169 expect(ownerCross.sub).to.be.eq(buyerCross.sub);116 });170 });117171118 itEth('Put + Buy [sub]', async ({helper}) => {172 itEth('Put + Buy [sub]', async ({helper}) => {173 const ONE_TOKEN = helper.balance.getOneTokenNominal();119 const PRICE = 1n;174 const PRICE = 2n * ONE_TOKEN; // 2 UNQ120 const web3 = helper.getWeb3();175 const web3 = helper.getWeb3();121 const marketOwner = await helper.eth.createAccountWithBalance(donor, 600n);176 const marketOwner = await helper.eth.createAccountWithBalance(donor, 600n);122 const market = await deployMarket(helper, marketOwner);177 const market = await deployMarket(helper, marketOwner);178 const contractHelpers = helper.ethNativeContract.contractHelpers(marketOwner);179180 // Set self sponsoring from contract balance181 await contractHelpers.methods.selfSponsoredEnable(market.options.address).send({from: marketOwner});182 await helper.eth.transferBalanceFromSubstrate(donor, market.options.address, 10n);183184 // Configure sponsoring185 await contractHelpers.methods.setSponsoringMode(market.options.address, SponsoringMode.Generous).send({from: marketOwner});186 await contractHelpers.methods.setSponsoringRateLimit(market.options.address, 0).send({from: marketOwner});123187124 const {collectionId, collectionAddress} = await helper.eth.createNFTCollection(marketOwner, 'Sponsor', 'absolutely anything', 'ROC');188 const {collectionId, collectionAddress} = await helper.eth.createNFTCollection(marketOwner, 'Sponsor', 'absolutely anything', 'ROC');125 const collection = await helper.ethNativeContract.collection(collectionAddress, 'nft', marketOwner);189 const collection = helper.ethNativeContract.collection(collectionAddress, 'nft', marketOwner, true);126190191 // Set collection sponsoring127 const [seller] = await helper.arrange.createAccounts([600n], donor);192 await collection.methods.setCollectionSponsor(marketOwner).send({from: marketOwner});193 await collection.methods.confirmCollectionSponsorship().send({from: marketOwner});194128 const sellerMirror = helper.address.substrateToEth(seller.address);195 const seller = helper.util.fromSeed(`//Market-seller-${(new Date()).getTime()}`);129 const sellerCross = helper.ethCrossAccount.fromKeyringPair(seller);196 const sellerCross = helper.ethCrossAccount.fromKeyringPair(seller);197198 // Seller has no funds at all, his transactions are sponsored199 {200 const sellerBalance = await helper.balance.getSubstrate(seller.address);201 expect(sellerBalance).to.be.eq(0n);202 }203130 const result = await collection.methods.mintCross(sellerCross, []).send();204 const result = await collection.methods.mintCross(sellerCross, []).send();131 const tokenId = result.events.Transfer.returnValues.tokenId;205 const tokenId = result.events.Transfer.returnValues.tokenId;132 await helper.nft.approveToken(seller, collectionId, tokenId, {Ethereum: market.options.address}, 1n);206 await helper.nft.approveToken(seller, collectionId, tokenId, {Ethereum: market.options.address});133207134 await helper.eth.sendEVM(seller, market.options.address, market.methods.put(collectionId, tokenId, PRICE, 1, sellerCross).encodeABI(), '0');208 await helper.eth.sendEVM(seller, market.options.address, market.methods.put(collectionId, tokenId, PRICE, 1, sellerCross).encodeABI(), '0');209 // Seller balance is still zero210 {211 const sellerBalance = await helper.balance.getSubstrate(seller.address);212 expect(sellerBalance).to.be.eq(0n);213 }135 let ownerCross = await collection.methods.ownerOfCross(tokenId).call();214 let ownerCross = await collection.methods.ownerOfCross(tokenId).call();136 expect(ownerCross.eth).to.be.eq(sellerCross.eth);215 expect(ownerCross.eth).to.be.eq(sellerCross.eth);137 expect(substrateAddressToHex(ownerCross.sub, web3)).to.be.eq(substrateAddressToHex(sellerCross.sub, web3));216 expect(substrateAddressToHex(ownerCross.sub, web3)).to.be.eq(substrateAddressToHex(sellerCross.sub, web3));138217139 const [buyer] = await helper.arrange.createAccounts([600n], donor);218 const [buyer] = await helper.arrange.createAccounts([600n], donor);219 // Buyer has only expected balance220 {221 const buyerBalance = await helper.balance.getSubstrate(buyer.address);222 expect(buyerBalance).to.be.eq(600n * ONE_TOKEN);223 }140 const buyerMirror = helper.address.substrateToEth(buyer.address);224 const buyerMirror = helper.address.substrateToEth(buyer.address);141 const buyerCross = helper.ethCrossAccount.fromKeyringPair(buyer);225 const buyerCross = helper.ethCrossAccount.fromKeyringPair(buyer);142 await helper.eth.transferBalanceFromSubstrate(donor, buyerMirror, 1n);226 await helper.eth.transferBalanceFromSubstrate(donor, buyerMirror, PRICE, false);143 //TODO: change balance check to helper.balance.getSubstrate when implementation of sendMoney will be fixed in contract227144 const sellerBalance = BigInt(await web3.eth.getBalance(sellerMirror));228 const buyerBalanceBefore = await helper.balance.getSubstrate(buyer.address);145 await helper.eth.sendEVM(buyer, market.options.address, market.methods.buy(collectionId, tokenId, 1, buyerCross).encodeABI(), PRICE.toString());229 await helper.eth.sendEVM(buyer, market.options.address, market.methods.buy(collectionId, tokenId, 1, buyerCross).encodeABI(), PRICE.toString());230 const buyerBalanceAfter = await helper.balance.getSubstrate(buyer.address);231 // Buyer balance not changed: transaction is sponsored232 expect(buyerBalanceBefore).to.be.eq(buyerBalanceAfter);233146 const sellerBalanceAfterBuy = BigInt(await web3.eth.getBalance(sellerMirror));234 const sellerBalanceAfterBuy = BigInt(await helper.balance.getSubstrate(seller.address));147 ownerCross = await collection.methods.ownerOfCross(tokenId).call();235 ownerCross = await collection.methods.ownerOfCross(tokenId).call();148 expect(ownerCross.eth).to.be.eq(buyerCross.eth);236 expect(ownerCross.eth).to.be.eq(buyerCross.eth);149 expect(substrateAddressToHex(ownerCross.sub, web3)).to.be.eq(substrateAddressToHex(buyerCross.sub, web3));237 expect(substrateAddressToHex(ownerCross.sub, web3)).to.be.eq(substrateAddressToHex(buyerCross.sub, web3));238239 // Seller got only PRICE - MARKET_FEE150 expect(sellerBalance + PRICE).to.be.equal(sellerBalanceAfterBuy);240 expect(sellerBalanceAfterBuy).to.be.eq(PRICE * BigInt(100 - MARKET_FEE) / 100n);151 });241 });152});242});153243tests/src/eth/tokenProperties.test.tsdiffbeforeafterboth--- a/tests/src/eth/tokenProperties.test.ts
+++ b/tests/src/eth/tokenProperties.test.ts
@@ -561,10 +561,10 @@
itEth.ifWithPallets(`[${testCase.mode}] Can't be multiple set/read for non-existent token`, testCase.requiredPallets, async({helper}) => {
const caller = await helper.eth.createAccountWithBalance(donor);
- const properties = Array(5).fill(0).map((_, i) => { return {key: `key_${i}`, value: Buffer.from(`value_${i}`)}; });
- const permissions: ITokenPropertyPermission[] = properties.map(p => { return {key: p.key, permission: {tokenOwner: true,
+ const properties = Array(5).fill(0).map((_, i) => ({key: `key_${i}`, value: Buffer.from(`value_${i}`)}));
+ const permissions: ITokenPropertyPermission[] = properties.map(p => ({key: p.key, permission: {tokenOwner: true,
collectionAdmin: true,
- mutable: true}}; });
+ mutable: true}}));
const collection = await helper[testCase.mode].mintCollection(alice, {
tokenPrefix: 'ethp',