difftreelog
feat(rmrk-rpc) rpc refactoring
in: master
6 files changed
pallets/proxy-rmrk-core/src/lib.rsdiffbeforeafterboth--- a/pallets/proxy-rmrk-core/src/lib.rs
+++ b/pallets/proxy-rmrk-core/src/lib.rs
@@ -22,7 +22,7 @@
use sp_std::vec::Vec;
use up_data_structs::*;
use pallet_common::{Pallet as PalletCommon, Error as CommonError, CollectionHandle, CommonCollectionOperations};
-use pallet_nonfungible::{Pallet as PalletNft, NonfungibleHandle};
+use pallet_nonfungible::{Pallet as PalletNft, NonfungibleHandle, TokenData};
use pallet_evm::account::CrossAccountId;
pub use pallet::*;
@@ -396,6 +396,15 @@
Ok(collection)
}
+ // should this even be here, might displace it to common/nonfungible -- but they did not need it, only rmrk does
+ pub fn collection_exists(collection_id: CollectionId) -> bool {
+ <pallet_common::CollectionById<T>>::contains_key(collection_id)
+ }
+
+ pub fn nft_exists(collection_id: CollectionId, nft_id: TokenId) -> bool {
+ <TokenData<T>>::contains_key((collection_id, nft_id))
+ }
+
pub fn get_collection_property(collection_id: CollectionId, key: RmrkProperty) -> Result<PropertyValue, DispatchError> {
let collection_property = <PalletCommon<T>>::collection_properties(collection_id)
.get(&rmrk_property!(Config=T, key)?)
@@ -414,6 +423,13 @@
Ok(collection_type)
}
+ pub fn ensure_collection_type(collection_id: CollectionId, collection_type: CollectionType) -> DispatchResult {
+ let actual_type = Self::get_collection_type(collection_id)?;
+ ensure!(actual_type == collection_type, <CommonError<T>>::NoPermission);
+
+ Ok(())
+ }
+
pub fn get_nft_property(collection_id: CollectionId, nft_id: TokenId, key: RmrkProperty) -> Result<PropertyValue, DispatchError> {
let nft_property = <PalletNft<T>>::token_properties((collection_id, nft_id))
.get(&rmrk_property!(Config=T, key)?)
@@ -423,9 +439,16 @@
Ok(nft_property)
}
- pub fn check_collection_type(collection_id: CollectionId, collection_type: CollectionType) -> DispatchResult {
- let actual_type = Self::get_collection_type(collection_id)?;
- ensure!(actual_type == collection_type, <CommonError<T>>::NoPermission);
+ pub fn get_nft_type(collection_id: CollectionId, token_id: TokenId) -> Result<NftType, DispatchError> {
+ <TokenData<T>>::get((collection_id, token_id))
+ .unwrap()
+ .rmrk_nft_type()
+ .ok_or(<Error<T>>::NoAvailableNftId.into())
+ }
+
+ pub fn ensure_nft_type(collection_id: CollectionId, token_id: TokenId, nft_type: NftType) -> DispatchResult {
+ let actual_type = Self::get_nft_type(collection_id, token_id)?;
+ ensure!(actual_type == nft_type, <CommonError<T>>::NoPermission);
Ok(())
}
@@ -434,7 +457,7 @@
collection_id: CollectionId,
collection_type: CollectionType
) -> Result<NonfungibleHandle<T>, DispatchError> {
- Self::check_collection_type(collection_id, collection_type)?;
+ Self::ensure_collection_type(collection_id, collection_type)?;
Self::get_nft_collection(collection_id)
}
pallets/proxy-rmrk-core/src/misc.rsdiffbeforeafterboth--- a/pallets/proxy-rmrk-core/src/misc.rs
+++ b/pallets/proxy-rmrk-core/src/misc.rs
@@ -82,15 +82,27 @@
}
}
-pub trait RmrkDecode<T: Decode> {
- fn decode_property(&self) -> Option<T>;
+pub trait RmrkDecode<T: Decode + Default, S> {
+ fn decode_or_default(&self) -> T;
}
-impl<T: Decode> RmrkDecode<T> for RmrkString {
- fn decode_property(&self) -> Option<T> { // todo access runtime errors? // but then rmrk_nft_type must have it too
+impl<T: Decode + Default, S> RmrkDecode<T, S> for BoundedVec<u8, S> {
+ fn decode_or_default(&self) -> T {
let mut value = self.as_slice();
- T::decode(&mut value).ok()
+ T::decode(&mut value).unwrap_or_default()
+ }
+}
+
+pub trait RmrkRebind<T, S> {
+ fn rebind(&self) -> BoundedVec<u8, S>;
+}
+
+impl<T, S> RmrkRebind<T, S> for BoundedVec<u8, T> where BoundedVec<u8, S>: TryFrom<Vec<u8>> {
+ fn rebind(&self) -> BoundedVec<u8, S> {
+ BoundedVec::<u8, S>::try_from(
+ self.clone().into_inner()
+ ).unwrap_or_default()
}
}
primitives/data-structs/src/rmrk.rsdiffbeforeafterboth--- a/primitives/data-structs/src/rmrk.rs
+++ b/primitives/data-structs/src/rmrk.rs
@@ -360,14 +360,14 @@
}
#[cfg_attr(feature = "std", derive(Serialize))]
-#[derive(Encode, Decode, Debug, TypeInfo, Clone, PartialEq, Eq, MaxEncodedLen)]
+#[derive(Encode, Decode, Debug, Default, TypeInfo, Clone, PartialEq, Eq, MaxEncodedLen)]
#[cfg_attr(
feature = "std",
serde(bound = "BoundedCollectionList: AsRef<[CollectionId]>")
)]
pub enum EquippableList<BoundedCollectionList> {
All,
- Empty,
+ #[default] Empty,
Custom(
#[cfg_attr(feature = "std", serde(with = "serialize::vec"))]
BoundedCollectionList
runtime/common/src/runtime_apis.rsdiffbeforeafterboth--- a/runtime/common/src/runtime_apis.rs
+++ b/runtime/common/src/runtime_apis.rs
@@ -38,10 +38,10 @@
keys: Option<Vec<Vec<u8>>>
) -> Result<Vec<Property>, DispatchError> {
let keys = keys.map(
- |keys| pallet_common::Pallet::<Runtime>::bytes_keys_to_property_keys(keys)
+ |keys| Common::bytes_keys_to_property_keys(keys)
).transpose()?;
- pallet_common::Pallet::<Runtime>::filter_collection_properties(collection, keys)
+ Common::filter_collection_properties(collection, keys)
}
fn token_properties(
@@ -50,7 +50,7 @@
keys: Option<Vec<Vec<u8>>>
) -> Result<Vec<Property>, DispatchError> {
let keys = keys.map(
- |keys| pallet_common::Pallet::<Runtime>::bytes_keys_to_property_keys(keys)
+ |keys| Common::bytes_keys_to_property_keys(keys)
).transpose()?;
dispatch_unique_runtime!(collection.token_properties(token_id, keys))
@@ -61,10 +61,10 @@
keys: Option<Vec<Vec<u8>>>
) -> Result<Vec<PropertyKeyPermission>, DispatchError> {
let keys = keys.map(
- |keys| pallet_common::Pallet::<Runtime>::bytes_keys_to_property_keys(keys)
+ |keys| Common::bytes_keys_to_property_keys(keys)
).transpose()?;
- pallet_common::Pallet::<Runtime>::filter_property_permissions(collection, keys)
+ Common::filter_property_permissions(collection, keys)
}
fn token_data(
@@ -144,34 +144,30 @@
fn last_collection_idx() -> Result<RmrkCollectionId, DispatchError> {
Ok(RmrkCore::last_collection_idx())
}
+
fn collection_by_id(collection_id: RmrkCollectionId) -> Result<Option<RmrkCollectionInfo<AccountId>>, DispatchError> {
- // TODO decide on displacement to palettes -- does RMRK belong there, spread across common and nonfungible?
use frame_support::BoundedVec;
use scale_info::prelude::string::String;
- use pallet_proxy_rmrk_core::{RmrkProperty, misc::CollectionType};
-
- // todo check if this is a rmrk standard collection? or simply trust and provide anyway?
- // client-is-always-right / enforce authority and order ?
+ use pallet_proxy_rmrk_core::{RmrkProperty, misc::{CollectionType, RmrkRebind, RmrkDecode}};
let collection_id = CollectionId(collection_id);
- let collection = <pallet_proxy_rmrk_core::Pallet<Runtime>>::get_typed_nft_collection(collection_id, CollectionType::Regular)?;
+ let collection = match RmrkCore::get_typed_nft_collection(collection_id, CollectionType::Regular) {
+ Ok(c) => c,
+ Err(_) => return Ok(None),
+ };
- let metadata = BoundedVec::try_from(
- <pallet_proxy_rmrk_core::Pallet<Runtime>>::get_collection_property(collection_id, RmrkProperty::Metadata)?.into_inner()
- ).map_err(|_| <pallet_common::Error<Runtime>>::PropertyKeyIsTooLong)?;
-
- let nfts_count = (dispatch_unique_runtime!(collection_id.total_supply()) as Result<u32, DispatchError>)?; // todo? <Runtime>::total_supply(collection_id)
+ let nfts_count = (dispatch_unique_runtime!(collection_id.total_supply()) as Result<u32, DispatchError>)?;
+ //<Runtime as up_rpc::UniqueApi>::total_supply(collection_id); // todo can't find UniqueApi with disabled default features
Ok(Some(RmrkCollectionInfo {
issuer: collection.owner.clone(),
- metadata,
+ metadata: RmrkCore::get_collection_property(collection_id, RmrkProperty::Metadata)?.decode_or_default(),
max: collection.limits.token_limit,
- symbol: BoundedVec::try_from(
- collection.token_prefix.clone().into_inner()
- ).map_err(|_| <pallet_common::Error<Runtime>>::PropertyKeyIsTooLong)?,
+ symbol: collection.token_prefix.rebind(), // change
nfts_count
}))
}
+
fn nft_by_id(collection_id: RmrkCollectionId, nft_by_id: RmrkNftId) -> Result<Option<RmrkInstanceInfo<AccountId>>, DispatchError> {
use frame_support::BoundedVec;
use up_data_structs::mapping::TokenAddressMapping;
@@ -179,6 +175,7 @@
let collection_id = CollectionId(collection_id);
let nft_id = TokenId(nft_by_id);
+ if !RmrkCore::nft_exists(collection_id, nft_id) { return Ok(None); }
let owner = match (dispatch_unique_runtime!(collection_id.token_owner(nft_id)) as Result<Option<CrossAccountId>, DispatchError>)? {
Some(owner) => match <Runtime as pallet_common::Config>::CrossTokenAddressMapping::address_to_token(&owner) {
@@ -187,36 +184,23 @@
},
None => return Ok(None)
};
-
- // todo displace querying property key array to rmrk proxy pallet
- let keys = [
- RmrkProperty::RoyaltyInfo,
- RmrkProperty::Metadata,
- RmrkProperty::Equipped,
- // ?? "rmrk:recipient", "rmrk:nft-type", "rmrk:resource-collection", "rmrk:resource-priorities"
- ];
-
- let properties = keys.into_iter().map(
- |key| BoundedVec::try_from(
- <pallet_proxy_rmrk_core::Pallet<Runtime>>::get_nft_property(collection_id, nft_id, key).unwrap().into_inner()
- ).unwrap()
- )
- .collect::<Vec<RmrkString>>();
-
+
let allowance = pallet_nonfungible::Allowance::<Runtime>::get((collection_id, nft_id));
Ok(Some(RmrkInstanceInfo {
owner: owner,
- //recipient: , // prop?
- royalty: properties[0].clone().decode_property().unwrap(),
- metadata: properties[1].clone(),
- equipped: properties[2].clone().decode_property().unwrap(),
+ royalty: RmrkCore::get_nft_property(collection_id, nft_id, RmrkProperty::RoyaltyInfo)?.decode_or_default(),
+ metadata: RmrkCore::get_nft_property(collection_id, nft_id, RmrkProperty::Metadata)?.decode_or_default(),
+ equipped: RmrkCore::get_nft_property(collection_id, nft_id, RmrkProperty::Equipped)?.decode_or_default(),
pending: allowance.is_some(),
}))
}
+
fn account_tokens(account_id: AccountId, collection_id: RmrkCollectionId) -> Result<Vec<RmrkNftId>, DispatchError> {
let cross_account_id = CrossAccountId::from_sub(account_id);
let collection_id = CollectionId(collection_id);
+ if !RmrkCore::collection_exists(collection_id) { return Ok(Vec::new()); }
+
Ok(
(dispatch_unique_runtime!(collection_id.account_tokens(cross_account_id)) as Result<Vec<TokenId>, DispatchError>)?
//<Runtime as up_rpc::UniqueApi<Block, CrossAccountId, AccountId>>::account_tokens(collection_id, cross_account_id)?
@@ -225,11 +209,14 @@
.collect::<Vec<_>>()
)
}
+
fn nft_children(collection_id: RmrkCollectionId, nft_id: RmrkNftId) -> Result<Vec<RmrkNftChild>, DispatchError> {
use up_data_structs::mapping::TokenAddressMapping;
let collection_id = CollectionId(collection_id);
let nft_id = TokenId(nft_id);
+ if !RmrkCore::nft_exists(collection_id, nft_id) { return Ok(Vec::new()); }
+
let cross_account_id = CrossAccountId::from_eth(
EvmTokenAddressMapping::token_to_address(collection_id, nft_id)
);
@@ -243,21 +230,25 @@
.collect()
)
}
+
fn collection_properties(collection_id: RmrkCollectionId, filter_keys: Option<Vec<RmrkPropertyKey>>) -> Result<Vec<RmrkPropertyInfo>, DispatchError> {
- use frame_support::BoundedVec;
+ use pallet_proxy_rmrk_core::misc::RmrkDecode;
let collection_id = CollectionId(collection_id);
- let properties = pallet_common::Pallet::<Runtime>::collection_properties(collection_id);
+ if !RmrkCore::collection_exists(collection_id) { return Ok(Vec::new()); }
+ let properties = Common::collection_properties(collection_id);
+
+ // todo repeated code
return Ok(match filter_keys {
Some(keys) => {
- let keys = pallet_common::Pallet::<Runtime>::bytes_keys_to_property_keys(keys)?;
+ let keys = Common::bytes_keys_to_property_keys(keys)?;
let properties = keys
.into_iter()
.filter_map(|key| {
properties.get(&key).map(|value| RmrkPropertyInfo {
- key: BoundedVec::try_from(key.into_inner()).unwrap(),
- value: BoundedVec::try_from(value.clone().into_inner()).unwrap(),
+ key: key.decode_or_default(),
+ value: value.decode_or_default(),
})
})
.collect();
@@ -268,31 +259,35 @@
properties
.iter()
.filter_map(|(key, value)| Some(RmrkPropertyInfo {
- key: BoundedVec::try_from(key.clone().into_inner()).unwrap(),
- value: BoundedVec::try_from(value.clone().into_inner()).unwrap(),
+ key: key.decode_or_default(),
+ value: value.decode_or_default(),
}))
.collect()
}
});
}
+
fn nft_properties(collection_id: RmrkCollectionId, nft_id: RmrkNftId, filter_keys: Option<Vec<RmrkPropertyKey>>) -> Result<Vec<RmrkPropertyInfo>, DispatchError> {
use frame_support::BoundedVec;
+ use pallet_proxy_rmrk_core::misc::RmrkDecode;
let collection_id = CollectionId(collection_id);
let token_id = TokenId(nft_id);
+ if !RmrkCore::nft_exists(collection_id, token_id) { return Ok(Vec::new()); }
- let properties = pallet_nonfungible::Pallet::<Runtime>::token_properties((collection_id, token_id)); // todo look into usage of pallet_nonfungible
+ let properties = Nonfungible::token_properties((collection_id, token_id));
+ // todo look into this usage of pallet_nonfungible
// todo displace to a function? redundant code piece with collection props
return Ok(match filter_keys {
Some(keys) => {
- let keys = pallet_common::Pallet::<Runtime>::bytes_keys_to_property_keys(keys)?;
+ let keys = Common::bytes_keys_to_property_keys(keys)?;
let properties = keys
.into_iter()
.filter_map(|key| {
properties.get(&key).map(|value| RmrkPropertyInfo {
- key: BoundedVec::try_from(key.into_inner()).unwrap(),
- value: BoundedVec::try_from(value.clone().into_inner()).unwrap(),
+ key: key.decode_or_default(),
+ value: value.decode_or_default(),
})
})
.collect();
@@ -303,113 +298,76 @@
properties
.iter()
.filter_map(|(key, value)| Some(RmrkPropertyInfo {
- key: BoundedVec::try_from(key.clone().into_inner()).unwrap(),
- value: BoundedVec::try_from(value.clone().into_inner()).unwrap(),
+ key: key.decode_or_default(),
+ value: value.decode_or_default(),
}))
.collect()
}
});
}
+
fn nft_resources(collection_id: RmrkCollectionId, nft_id: RmrkNftId) -> Result<Vec<RmrkResourceInfo>, DispatchError> {
use frame_support::BoundedVec;
- use pallet_proxy_rmrk_core::RmrkProperty;
+ use pallet_proxy_rmrk_core::{RmrkProperty, misc::{CollectionType, NftType}};
let collection_id = CollectionId(collection_id);
- let nft_id = TokenId(nft_id);
-
- // let keys = [
- // RmrkProperty::RoyaltyInfo,
- // RmrkProperty::Metadata,
- // RmrkProperty::Equipped,
- // RmrkProperty::Pending,
- // // ?? "rmrk:recipient", "rmrk:nft-type", "rmrk:resource-collection", "rmrk:resource-priorities"
- // ];
+ if RmrkCore::ensure_collection_type(collection_id, CollectionType::Resource).is_err() { return Ok(Vec::new()); }
- /*let resources = keys.into_iter().map(
- |key| BoundedVec::try_from(
- <pallet_proxy_rmrk_core::Pallet<Runtime>>::get_nft_property(collection_id, nft_id, key).unwrap().into_inner()
- ).unwrap()
- )
- .collect::<Vec<RmrkString>>();*/
+ let nft_id = TokenId(nft_id);
+ if RmrkCore::ensure_nft_type(collection_id, nft_id, NftType::Resource).is_err() { return Ok(Vec::new()); }
Ok(Vec::new(/*[RmrkResourceInfo {
-
+
}]*/))
}
+
fn nft_resource_priorities(collection_id: RmrkCollectionId, nft_id: RmrkNftId) -> Result<Vec<RmrkResourceId>, DispatchError> {
todo!()
}
+
fn base(base_id: RmrkBaseId) -> Result<Option<RmrkBaseInfo<AccountId>>, DispatchError> {
use frame_support::BoundedVec;
use scale_info::prelude::string::String;
- use pallet_proxy_rmrk_core::{RmrkProperty, misc::CollectionType};
+ use pallet_proxy_rmrk_core::{
+ RmrkProperty, misc::{CollectionType, RmrkRebind, RmrkDecode},
+ };
let collection_id = CollectionId(base_id);
- let collection = <pallet_proxy_rmrk_core::Pallet<Runtime>>::get_typed_nft_collection(collection_id, CollectionType::Base)?;
- // todo check prop for being a base
-
- // todo export to macro? redundancy
- let keys = [
- RmrkProperty::BaseType,
- ];
-
- let properties = keys.into_iter().map(
- |key| BoundedVec::try_from(
- <pallet_proxy_rmrk_core::Pallet<Runtime>>::get_collection_property(collection_id, key).unwrap().into_inner()
- )
- )
- // todo not-a-rmrk-collection error
- .collect::<Result<Vec<_>, _>>()
- .map_err(|_| <pallet_proxy_rmrk_core::Error<Runtime>>::CollectionUnknown)?;
+ let collection = match RmrkCore::get_typed_nft_collection(collection_id, CollectionType::Base) {
+ Ok(c) => c,
+ Err(_) => return Ok(None),
+ };
Ok(Some(RmrkBaseInfo {
issuer: collection.owner.clone(),
- base_type: properties[0].clone(),
- symbol: BoundedVec::try_from(
- collection.token_prefix.clone().into_inner()
- ).map_err(|_| <pallet_common::Error<Runtime>>::PropertyKeyIsTooLong)?,
+ base_type: RmrkCore::get_collection_property(collection_id, RmrkProperty::BaseType)?.decode_or_default(),
+ symbol: collection.token_prefix.rebind(),
}))
}
+
fn base_parts(base_id: RmrkBaseId) -> Result<Vec<RmrkPartType>, DispatchError> {
use frame_support::BoundedVec;
- use pallet_proxy_rmrk_core::{RmrkProperty, misc::{RmrkNft, RmrkDecode}};
+ use pallet_proxy_rmrk_core::{RmrkProperty, misc::{CollectionType, RmrkNft, RmrkDecode}};
let collection_id = CollectionId(base_id);
- // todo check prop for being a base
+ if RmrkCore::ensure_collection_type(collection_id, CollectionType::Base).is_err() { return Ok(Vec::new()); }
let parts = (dispatch_unique_runtime!(collection_id.collection_tokens()) as Result<Vec<TokenId>, DispatchError>)?
.iter()
.filter_map(|token_id| {
- let nft_type = <pallet_nonfungible::TokenData<Runtime>>::get((collection_id, token_id))
- //.map_err(|_| ) // no need, tis a filter_map
- .unwrap()
- .rmrk_nft_type()?;
-
- // dislocate to rmrkproxycore and simply send an array of keys
- let keys = [
- //RmrkProperty::PartId)?,
- RmrkProperty::Src,
- RmrkProperty::ZIndex,
- RmrkProperty::EquippableList,
- ];
-
- let properties = keys.into_iter().map(
- |key| BoundedVec::try_from(
- <pallet_proxy_rmrk_core::Pallet<Runtime>>::get_nft_property(collection_id, *token_id, key).unwrap().into_inner()
- ).unwrap()
- ).collect::<Vec<RmrkString>>();
+ let nft_type = RmrkCore::get_nft_type(collection_id, *token_id).unwrap();
match nft_type {
FixedPart => Some(RmrkPartType::FixedPart(RmrkFixedPart {
id: token_id.0,
- src: properties[0].clone().decode_property().unwrap(),
- z: properties[1].clone().decode_property().unwrap(),
+ src: RmrkCore::get_nft_property(collection_id, *token_id, RmrkProperty::Src).unwrap().decode_or_default(),
+ z: RmrkCore::get_nft_property(collection_id, *token_id, RmrkProperty::ZIndex).unwrap().decode_or_default(),
})),
SlotPart => Some(RmrkPartType::SlotPart(RmrkSlotPart {
id: token_id.0,
- src: properties[0].clone().decode_property().unwrap(),
- z: properties[1].clone().decode_property().unwrap(),
- equippable: properties[2].clone().decode_property().unwrap(),
+ src: RmrkCore::get_nft_property(collection_id, *token_id, RmrkProperty::Src).unwrap().decode_or_default(),
+ z: RmrkCore::get_nft_property(collection_id, *token_id, RmrkProperty::ZIndex).unwrap().decode_or_default(),
+ equippable: RmrkCore::get_nft_property(collection_id, *token_id, RmrkProperty::EquippableList).unwrap().decode_or_default(),
})),
_ => None
}
@@ -418,6 +376,7 @@
Ok(parts)
}
+
fn theme_names(base_id: RmrkBaseId) -> Result<Vec<RmrkThemeName>, DispatchError> {
use frame_support::BoundedVec;
use pallet_proxy_rmrk_core::{RmrkProperty, misc::{RmrkNft, RmrkDecode}};
@@ -428,16 +387,11 @@
let theme_names = (dispatch_unique_runtime!(collection_id.collection_tokens()) as Result<Vec<TokenId>, DispatchError>)?
.iter()
.filter_map(|token_id| {
- let nft_type = <pallet_nonfungible::TokenData<Runtime>>::get((collection_id, token_id))
- .unwrap()
- .rmrk_nft_type()?;
-
+ let nft_type = RmrkCore::get_nft_type(collection_id, *token_id).unwrap();
+
match nft_type {
Theme => Some(
- <pallet_proxy_rmrk_core::Pallet<Runtime>>::get_nft_property(
- collection_id, *token_id, RmrkProperty::ThemeName
- ).unwrap()
- .into_inner()
+ RmrkCore::get_nft_property(collection_id, *token_id, RmrkProperty::ThemeName).unwrap().decode_or_default()
),
_ => None
}
@@ -446,6 +400,7 @@
Ok(theme_names)
}
+
fn theme(base_id: RmrkBaseId, theme_name: RmrkThemeName, filter_keys: Option<Vec<RmrkPropertyKey>>) -> Result<Option<RmrkTheme>, DispatchError> {
use frame_support::BoundedVec;
@@ -457,7 +412,7 @@
let themes = (dispatch_unique_runtime!(collection_id.collection_tokens()) as Result<Vec<TokenId>, DispatchError>)?
.iter()
.filter_map(|token_id| {
- let properties = pallet_nonfungible::Pallet::<Runtime>::token_properties((collection_id, token_id));
+ let properties = Nonfungible::token_properties((collection_id, token_id));
// todo ping properties for "rmrk:nft-type"
// if none, skip, None
tests/package.jsondiffbeforeafterboth--- a/tests/package.json
+++ b/tests/package.json
@@ -37,6 +37,7 @@
"testUnnesting": "mocha --timeout 9999999 -r ts-node/register ./**/unnest.test.ts",
"testStructure": "mocha --timeout 9999999 -r ts-node/register ./**/nesting/**.test.ts",
"testProperties": "mocha --timeout 9999999 -r ts-node/register ./**/properties.test.ts",
+ "testGraphs": "mocha --timeout 9999999 -r ts-node/register ./**/graphs.test.ts",
"testMigrationStructure": "mocha --timeout 9999999 -r ts-node/register ./**/nesting/migration-check.test.ts",
"testAddCollectionAdmin": "mocha --timeout 9999999 -r ts-node/register ./**/addCollectionAdmin.test.ts",
"testSetSchemaVersion": "mocha --timeout 9999999 -r ts-node/register ./**/setSchemaVersion.test.ts",
tests/src/nesting/graphs.test.tsdiffbeforeafterboth1import {ApiPromise} from '@polkadot/api';2import {IKeyringPair} from '@polkadot/types/types';3import {expect} from 'chai';4import {tokenIdToCross} from '../eth/util/helpers';5import privateKey from '../substrate/privateKey';6import usingApi, {executeTransaction} from '../substrate/substrate-api';7import {getCreateCollectionResult, transferExpectSuccess} from '../util/helpers';89/**10 * ```dot11 * 4 -> 3 -> 2 -> 112 * 7 -> 6 -> 5 -> 213 * 8 -> 514 * ```15 */16async function buildComplexObjectGraph(api: ApiPromise, sender: IKeyringPair): Promise<number> {17 const events = await executeTransaction(api, sender, api.tx.unique.createCollectionEx({mode: 'NFT', limits: {nestingRule: 'Owner'}}));18 const {collectionId} = getCreateCollectionResult(events);1920 await executeTransaction(api, sender, api.tx.unique.createMultipleItemsEx(collectionId, {NFT: Array(8).fill({owner: {Substrate: sender.address}})}));2122 await transferExpectSuccess(collectionId, 8, sender, tokenIdToCross(collectionId, 5));2324 await transferExpectSuccess(collectionId, 7, sender, tokenIdToCross(collectionId, 6));25 await transferExpectSuccess(collectionId, 6, sender, tokenIdToCross(collectionId, 5));26 await transferExpectSuccess(collectionId, 5, sender, tokenIdToCross(collectionId, 2));2728 await transferExpectSuccess(collectionId, 4, sender, tokenIdToCross(collectionId, 3));29 await transferExpectSuccess(collectionId, 3, sender, tokenIdToCross(collectionId, 2));30 await transferExpectSuccess(collectionId, 2, sender, tokenIdToCross(collectionId, 1));3132 return collectionId;33}3435describe('Graphs', () => {36 it('Ouroboros can\'t be created in a complex graph', async () => {37 await usingApi(async api => {38 const alice = privateKey('//Alice');39 const collection = await buildComplexObjectGraph(api, alice);4041 // to self42 await expect(43 executeTransaction(api, alice, api.tx.unique.transfer(tokenIdToCross(collection, 1), collection, 1, 1)),44 'first transaction', 45 ).to.be.rejectedWith(/structure\.OuroborosDetected/);46 // to nested part of graph47 await expect(48 executeTransaction(api, alice, api.tx.unique.transfer(tokenIdToCross(collection, 5), collection, 1, 1)),49 'second transaction',50 ).to.be.rejectedWith(/structure\.OuroborosDetected/);51 await expect(52 executeTransaction(api, alice, api.tx.unique.transfer(tokenIdToCross(collection, 8), collection, 2, 1)),53 'third transaction',54 ).to.be.rejectedWith(/structure\.OuroborosDetected/);55 });56 });57});