refactor: make weight argument in xtokens transfers optional (#841)

* refactor: make weight argument in xtokens be of type WeightLimit

* refactor: rename dest_weight -> dest_weight_limit

Author: sander2

asset-registry/src/tests.rs

@@ -118,7 +118,7 @@ fn send_self_parachain_asset_to_sibling() {
 				)
 				.into()
 			),
-			40,
+			WeightLimit::Unlimited,
 		));
 
 		assert_eq!(ParaTokens::free_balance(CurrencyId::RegisteredAsset(1), &ALICE), 500);
@@ -204,7 +204,7 @@ fn send_sibling_asset_to_non_reserve_sibling() {
 				)
 				.into()
 			),
-			40
+			WeightLimit::Unlimited
 		));
 		assert_eq!(ParaTokens::free_balance(CurrencyId::RegisteredAsset(1), &ALICE), 500);
 	});
@@ -302,7 +302,7 @@ fn test_fixed_rate_asset_trader() {
 				)
 				.into()
 			),
-			40,
+			WeightLimit::Unlimited,
 		));
 	});
 
@@ -353,7 +353,7 @@ fn test_fixed_rate_asset_trader() {
 				)
 				.into()
 			),
-			40,
+			WeightLimit::Unlimited,
 		));
 	});
 

traits/src/xcm_transfer.rs

@@ -1,5 +1,5 @@
 use frame_support::dispatch::DispatchResult;
-use xcm::latest::{prelude::*, Weight};
+use xcm::latest::prelude::*;
 
 /// Abstraction over cross-chain token transfers.
 pub trait XcmTransfer<AccountId, Balance, CurrencyId> {
@@ -9,14 +9,14 @@ pub trait XcmTransfer<AccountId, Balance, CurrencyId> {
 		currency_id: CurrencyId,
 		amount: Balance,
 		dest: MultiLocation,
-		dest_weight: Weight,
+		dest_weight_limit: WeightLimit,
 	) -> DispatchResult;
 
 	/// Transfer `MultiAsset`
 	fn transfer_multi_asset(
 		who: AccountId,
 		asset: MultiAsset,
 		dest: MultiLocation,
-		dest_weight: Weight,
+		dest_weight_limit: WeightLimit,
 	) -> DispatchResult;
 }