difftreelog
CORE-325 Fix metadata name and symbol
in: master
2 files changed
pallets/nonfungible/src/erc.rsdiffbeforeafterboth--- a/pallets/nonfungible/src/erc.rs
+++ b/pallets/nonfungible/src/erc.rs
@@ -77,20 +77,14 @@
#[solidity_interface(name = "ERC721Metadata")]
impl<T: Config> NonfungibleHandle<T> {
fn name(&self) -> Result<string> {
- if let SchemaVersion::ImageURL = self.schema_version {
- Ok(decode_utf16(self.name.iter().copied())
- .map(|r| r.unwrap_or(REPLACEMENT_CHARACTER))
- .collect::<string>())
- } else {
- Err(error_unsupported_shema_version())
- }
+ Ok(decode_utf16(self.name.iter().copied())
+ .map(|r| r.unwrap_or(REPLACEMENT_CHARACTER))
+ .collect::<string>())
+
}
+
fn symbol(&self) -> Result<string> {
- if let SchemaVersion::ImageURL = self.schema_version {
- Ok(string::from_utf8_lossy(&self.token_prefix).into())
- } else {
- Err(error_unsupported_shema_version())
- }
+ Ok(string::from_utf8_lossy(&self.token_prefix).into())
}
/// Returns token's const_metadata
tests/src/eth/metadata.test.tsdiffbeforeafterboth1// Copyright 2019-2022 Unique Network (Gibraltar) Ltd.2// This file is part of Unique Network.34// Unique Network is free software: you can redistribute it and/or modify5// it under the terms of the GNU General Public License as published by6// the Free Software Foundation, either version 3 of the License, or7// (at your option) any later version.89// Unique Network is distributed in the hope that it will be useful,10// but WITHOUT ANY WARRANTY; without even the implied warranty of11// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the12// GNU General Public License for more details.1314// You should have received a copy of the GNU General Public License15// along with Unique Network. If not, see <http://www.gnu.org/licenses/>.1617import {expect} from 'chai';18import {createCollectionExpectSuccess} from '../util/helpers';19import {collectionIdToAddress, createEthAccount, createEthAccountWithBalance, GAS_ARGS, itWeb3, normalizeEvents} from './util/helpers';20import fungibleMetadataAbi from './fungibleMetadataAbi.json';21import privateKey from '../substrate/privateKey';22import {submitTransactionAsync} from '../substrate/substrate-api';23import nonFungibleAbi from './nonFungibleAbi.json';2425describe('Common metadata', () => {26 itWeb3('Returns collection name', async ({api, web3}) => {27 const collection = await createCollectionExpectSuccess({28 name: 'token name',29 mode: {type: 'NFT'},30 });31 const caller = await createEthAccountWithBalance(api, web3);3233 const address = collectionIdToAddress(collection);34 const contract = new web3.eth.Contract(fungibleMetadataAbi as any, address, {from: caller, ...GAS_ARGS});35 const name = await contract.methods.name().call();3637 expect(name).to.equal('token name');38 });3940 itWeb3('Returns symbol name', async ({api, web3}) => {41 const collection = await createCollectionExpectSuccess({42 tokenPrefix: 'TOK',43 mode: {type: 'NFT'},44 });45 const caller = await createEthAccountWithBalance(api, web3);4647 const address = collectionIdToAddress(collection);48 const contract = new web3.eth.Contract(fungibleMetadataAbi as any, address, {from: caller, ...GAS_ARGS});49 const symbol = await contract.methods.symbol().call();5051 expect(symbol).to.equal('TOK');52 });53});5455describe('Fungible metadata', () => {56 itWeb3('Returns fungible decimals', async ({api, web3}) => {57 const collection = await createCollectionExpectSuccess({58 mode: {type: 'Fungible', decimalPoints: 6},59 });60 const caller = await createEthAccountWithBalance(api, web3);6162 const address = collectionIdToAddress(collection);63 const contract = new web3.eth.Contract(fungibleMetadataAbi as any, address, {from: caller, ...GAS_ARGS});64 const decimals = await contract.methods.decimals().call();6566 expect(+decimals).to.equal(6);67 });68});6970describe.only('Support ERC721Metadata', () => {71 itWeb3('Check unsupport ERC721Metadata ShemaVersion::Unique', async ({web3, api}) => {72 const collectionId = await createCollectionExpectSuccess({73 mode: {type: 'NFT'},74 shemaVersion: 'Unique',75 });76 const collection = await api.rpc.unique.collectionById(collectionId);77 expect(collection.isSome).to.be.true;78 expect(collection.unwrap().schemaVersion.toHuman()).to.be.eq('Unique');7980 const alice = privateKey('//Alice');8182 const caller = await createEthAccountWithBalance(api, web3);83 const changeAdminTx = api.tx.unique.addCollectionAdmin(collectionId, {Ethereum: caller});84 await submitTransactionAsync(alice, changeAdminTx);8586 const address = collectionIdToAddress(collectionId);87 const contract = new web3.eth.Contract(nonFungibleAbi as any, address, {from: caller, ...GAS_ARGS});8889 await expect(contract.methods.name().call()).to.be.rejectedWith('Unsupported shema version! Support only ImageURL');90 await expect(contract.methods.symbol().call()).to.be.rejectedWith('Unsupported shema version! Support only ImageURL');9192 const receiver = createEthAccount(web3);93 const nextTokenId = await contract.methods.nextTokenId().call();94 expect(nextTokenId).to.be.equal('1');95 await expect(contract.methods.mintWithTokenURI(96 receiver,97 nextTokenId,98 'Test URI',99 ).send({from: caller})).to.be.rejected;100101 await expect(contract.methods.mintBulkWithTokenURI(102 receiver,103 [104 [nextTokenId, 'Test URI 0'],105 [+nextTokenId + 1, 'Test URI 1'],106 [+nextTokenId + 2, 'Test URI 2'],107 ],108 ).send({from: caller})).to.be.rejected;109 });110111 itWeb3('Check support ERC721Metadata for ShemaVersion::ImageURL', async ({web3, api}) => {112 const collectionId = await createCollectionExpectSuccess({113 mode: {type: 'NFT'},114 name: 'some_name',115 tokenPrefix: 'some_prefix',116 });117 const collection = await api.rpc.unique.collectionById(collectionId);118 expect(collection.isSome).to.be.true;119 expect(collection.unwrap().schemaVersion.toHuman()).to.be.eq('ImageURL');120121 const alice = privateKey('//Alice');122123 const caller = await createEthAccountWithBalance(api, web3);124 const changeAdminTx = api.tx.unique.addCollectionAdmin(collectionId, {Ethereum: caller});125 await submitTransactionAsync(alice, changeAdminTx);126127 const address = collectionIdToAddress(collectionId);128 const contract = new web3.eth.Contract(nonFungibleAbi as any, address, {from: caller, ...GAS_ARGS});129 130 expect(await contract.methods.name().call()).to.be.eq('some_name');131 expect(await contract.methods.symbol().call()).to.be.eq('some_prefix');132133 const receiver = createEthAccount(web3);134 const nextTokenId = await contract.methods.nextTokenId().call();135 expect(nextTokenId).to.be.equal('1');136 const result = await contract.methods.mintWithTokenURI(137 receiver,138 nextTokenId,139 'Test URI',140 ).send({from: caller});141 const events = normalizeEvents(result.events);142143 expect(events).to.be.deep.equal([144 {145 address,146 event: 'Transfer',147 args: {148 from: '0x0000000000000000000000000000000000000000',149 to: receiver,150 tokenId: nextTokenId,151 },152 },153 ]);154155 expect(await contract.methods.tokenURI(nextTokenId).call()).to.be.equal('Test URI');156 });157});1581// Copyright 2019-2022 Unique Network (Gibraltar) Ltd.2// This file is part of Unique Network.34// Unique Network is free software: you can redistribute it and/or modify5// it under the terms of the GNU General Public License as published by6// the Free Software Foundation, either version 3 of the License, or7// (at your option) any later version.89// Unique Network is distributed in the hope that it will be useful,10// but WITHOUT ANY WARRANTY; without even the implied warranty of11// MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the12// GNU General Public License for more details.1314// You should have received a copy of the GNU General Public License15// along with Unique Network. If not, see <http://www.gnu.org/licenses/>.1617import {expect} from 'chai';18import {createCollectionExpectSuccess} from '../util/helpers';19import {collectionIdToAddress, createEthAccount, createEthAccountWithBalance, GAS_ARGS, itWeb3, normalizeEvents} from './util/helpers';20import fungibleMetadataAbi from './fungibleMetadataAbi.json';21import privateKey from '../substrate/privateKey';22import {submitTransactionAsync} from '../substrate/substrate-api';23import nonFungibleAbi from './nonFungibleAbi.json';2425describe('Common metadata', () => {26 itWeb3('Returns collection name', async ({api, web3}) => {27 const collection = await createCollectionExpectSuccess({28 name: 'token name',29 mode: {type: 'NFT'},30 });31 const caller = await createEthAccountWithBalance(api, web3);3233 const address = collectionIdToAddress(collection);34 const contract = new web3.eth.Contract(fungibleMetadataAbi as any, address, {from: caller, ...GAS_ARGS});35 const name = await contract.methods.name().call();3637 expect(name).to.equal('token name');38 });3940 itWeb3('Returns symbol name', async ({api, web3}) => {41 const collection = await createCollectionExpectSuccess({42 tokenPrefix: 'TOK',43 mode: {type: 'NFT'},44 });45 const caller = await createEthAccountWithBalance(api, web3);4647 const address = collectionIdToAddress(collection);48 const contract = new web3.eth.Contract(fungibleMetadataAbi as any, address, {from: caller, ...GAS_ARGS});49 const symbol = await contract.methods.symbol().call();5051 expect(symbol).to.equal('TOK');52 });53});5455describe('Fungible metadata', () => {56 itWeb3('Returns fungible decimals', async ({api, web3}) => {57 const collection = await createCollectionExpectSuccess({58 mode: {type: 'Fungible', decimalPoints: 6},59 });60 const caller = await createEthAccountWithBalance(api, web3);6162 const address = collectionIdToAddress(collection);63 const contract = new web3.eth.Contract(fungibleMetadataAbi as any, address, {from: caller, ...GAS_ARGS});64 const decimals = await contract.methods.decimals().call();6566 expect(+decimals).to.equal(6);67 });68});6970describe.only('Support ERC721Metadata', () => {71 itWeb3('Check unsupport ERC721Metadata ShemaVersion::Unique', async ({web3, api}) => {72 const collectionId = await createCollectionExpectSuccess({73 mode: {type: 'NFT'},74 shemaVersion: 'Unique',75 name: 'some_name',76 tokenPrefix: 'some_prefix',77 });78 const collection = await api.rpc.unique.collectionById(collectionId);79 expect(collection.isSome).to.be.true;80 expect(collection.unwrap().schemaVersion.toHuman()).to.be.eq('Unique');8182 const alice = privateKey('//Alice');8384 const caller = await createEthAccountWithBalance(api, web3);85 const changeAdminTx = api.tx.unique.addCollectionAdmin(collectionId, {Ethereum: caller});86 await submitTransactionAsync(alice, changeAdminTx);8788 const address = collectionIdToAddress(collectionId);89 const contract = new web3.eth.Contract(nonFungibleAbi as any, address, {from: caller, ...GAS_ARGS});9091 expect(await contract.methods.name().call()).to.be.eq('some_name');92 expect(await contract.methods.symbol().call()).to.be.eq('some_prefix');9394 const receiver = createEthAccount(web3);95 const nextTokenId = await contract.methods.nextTokenId().call();96 expect(nextTokenId).to.be.equal('1');97 await expect(contract.methods.mintWithTokenURI(98 receiver,99 nextTokenId,100 'Test URI',101 ).send({from: caller})).to.be.rejected;102103 await expect(contract.methods.mintBulkWithTokenURI(104 receiver,105 [106 [nextTokenId, 'Test URI 0'],107 [+nextTokenId + 1, 'Test URI 1'],108 [+nextTokenId + 2, 'Test URI 2'],109 ],110 ).send({from: caller})).to.be.rejected;111 });112113 itWeb3('Check support ERC721Metadata for ShemaVersion::ImageURL', async ({web3, api}) => {114 const collectionId = await createCollectionExpectSuccess({115 mode: {type: 'NFT'},116 name: 'some_name',117 tokenPrefix: 'some_prefix',118 });119 const collection = await api.rpc.unique.collectionById(collectionId);120 expect(collection.isSome).to.be.true;121 expect(collection.unwrap().schemaVersion.toHuman()).to.be.eq('ImageURL');122123 const alice = privateKey('//Alice');124125 const caller = await createEthAccountWithBalance(api, web3);126 const changeAdminTx = api.tx.unique.addCollectionAdmin(collectionId, {Ethereum: caller});127 await submitTransactionAsync(alice, changeAdminTx);128129 const address = collectionIdToAddress(collectionId);130 const contract = new web3.eth.Contract(nonFungibleAbi as any, address, {from: caller, ...GAS_ARGS});131 132 expect(await contract.methods.name().call()).to.be.eq('some_name');133 expect(await contract.methods.symbol().call()).to.be.eq('some_prefix');134135 const receiver = createEthAccount(web3);136 { // mintWithTokenURI137 const nextTokenId = await contract.methods.nextTokenId().call();138 expect(nextTokenId).to.be.equal('1');139 const result = await contract.methods.mintWithTokenURI(140 receiver,141 nextTokenId,142 'Test URI',143 ).send({from: caller});144 const events = normalizeEvents(result.events);145 146 expect(events).to.be.deep.equal([147 {148 address,149 event: 'Transfer',150 args: {151 from: '0x0000000000000000000000000000000000000000',152 to: receiver,153 tokenId: nextTokenId,154 },155 },156 ]);157 158 expect(await contract.methods.tokenURI(nextTokenId).call()).to.be.equal('Test URI');159 }160161 { // mintBulkWithTokenURI162 const nextTokenId = await contract.methods.nextTokenId().call();163 expect(nextTokenId).to.be.equal('2');164 const result = await contract.methods.mintBulkWithTokenURI(165 receiver,166 [167 [nextTokenId, 'Test URI 0'],168 [+nextTokenId + 1, 'Test URI 1'],169 [+nextTokenId + 2, 'Test URI 2'],170 ],171 ).send({from: caller});172 const events = normalizeEvents(result.events);173174 expect(events).to.be.deep.equal([175 {176 address,177 event: 'Transfer',178 args: {179 from: '0x0000000000000000000000000000000000000000',180 to: receiver,181 tokenId: nextTokenId,182 },183 },184 {185 address,186 event: 'Transfer',187 args: {188 from: '0x0000000000000000000000000000000000000000',189 to: receiver,190 tokenId: String(+nextTokenId + 1),191 },192 },193 {194 address,195 event: 'Transfer',196 args: {197 from: '0x0000000000000000000000000000000000000000',198 to: receiver,199 tokenId: String(+nextTokenId + 2),200 },201 },202 ]);203204 expect(await contract.methods.tokenURI(nextTokenId).call()).to.be.equal('Test URI 0');205 expect(await contract.methods.tokenURI(+nextTokenId + 1).call()).to.be.equal('Test URI 1');206 expect(await contract.methods.tokenURI(+nextTokenId + 2).call()).to.be.equal('Test URI 2');207 }208 });209});210