diff --git a/pallets/nfts/src/tests.rs b/pallets/nfts/src/tests.rs index f6e25b64..5c78450d 100644 --- a/pallets/nfts/src/tests.rs +++ b/pallets/nfts/src/tests.rs @@ -2119,10 +2119,10 @@ fn cancel_approval_ensures_no_active_collection_approval() { delegate.clone() )); assert_ok!(Nfts::cancel_approval( - RuntimeOrigin::signed(owner.clone()), + RuntimeOrigin::signed(owner), collection_id, item_id, - delegate.clone() + delegate )); }); } @@ -2500,7 +2500,7 @@ fn clear_all_transfer_approvals_ensures_no_active_collection_approval() { delegate )); assert_ok!(Nfts::clear_all_transfer_approvals( - RuntimeOrigin::signed(owner.clone()), + RuntimeOrigin::signed(owner), collection_id, item_id )); @@ -4441,7 +4441,7 @@ fn clear_collection_approvals_works() { assert_eq!(Balances::free_balance(&item_owner), balance); assert!(events().contains(&Event::::ApprovalsCancelled { collection: collection_id, - owner: item_owner.clone(), + owner: item_owner, approvals: 0 })); @@ -4568,7 +4568,7 @@ fn force_clear_collection_approvals_works() { assert_eq!(Balances::free_balance(&item_owner), balance); assert!(events().contains(&Event::::ApprovalsCancelled { collection: collection_id, - owner: item_owner.clone(), + owner: item_owner, approvals: 0 })); @@ -4687,13 +4687,13 @@ fn approve_collection_transfer_works() { // Set collection settings to non transferable. assert_ok!(Nfts::lock_collection( - RuntimeOrigin::signed(collection_owner.clone()), + RuntimeOrigin::signed(collection_owner), collection_id, CollectionSettings::from_disabled(CollectionSetting::TransferableItems.into()) )); assert_noop!( Nfts::approve_collection_transfer( - RuntimeOrigin::signed(item_owner.clone()), + RuntimeOrigin::signed(item_owner), collection_id, delegate, None @@ -4813,16 +4813,16 @@ fn force_approve_collection_transfer_works() { // Set collection settings to non transferable. assert_ok!(Nfts::lock_collection( - RuntimeOrigin::signed(collection_owner.clone()), + RuntimeOrigin::signed(collection_owner), collection_id, CollectionSettings::from_disabled(CollectionSetting::TransferableItems.into()) )); assert_noop!( Nfts::force_approve_collection_transfer( root(), - item_owner.clone(), + item_owner, collection_id, - delegate.clone(), + delegate, None ), Error::::ItemsNonTransferable @@ -4894,8 +4894,8 @@ fn cancel_collection_approval_works() { Event::::ApprovalCancelled { collection: collection_id, item: None, - owner: item_owner.clone(), - delegate: delegate.clone(), + owner: item_owner, + delegate, } .into(), ); @@ -4974,8 +4974,8 @@ fn force_cancel_collection_approval_works() { Event::::ApprovalCancelled { collection: collection_id, item: None, - owner: item_owner.clone(), - delegate: delegate.clone(), + owner: item_owner, + delegate, } .into(), );