Update comment

Author: raymondkfcheung

cumulus/parachains/runtimes/assets/asset-hub-westend/src/lib.rs

@@ -2166,40 +2166,46 @@ impl_runtime_apis! {
 
 					let wnd: Location = Parent.into();
 					let (account, _) = pallet_xcm_benchmarks::account_and_location::<Runtime>(1);
+					let signed_origin = RuntimeOrigin::signed(account.clone());
 
 					// Simulate the maximum possible assets for 'give' and 'receive'
 					let mut give_assets = XcmAssets::new();
 					let mut receive_assets = XcmAssets::new();
 
-					// Setup AssetHubWestend assets
+					// Setup assets and pools
 					for i in 0..MAX_ITEMS_IN_ASSETS {
 						let asset_id: AssetId = Location::new(1, [Parachain((2000 + i) as u32)]).into();
 
-						// Create pool
+						// Mint foreign asset
 						assert_ok!(ForeignAssets::mint(
-							RuntimeOrigin::root(),
+							signed_origin.clone(),
 							asset_id.clone().into(),
 							account.clone().into(),
 							3 * UNITS,
 						));
+
+						// Create pool
 						assert_ok!(AssetConversion::create_pool(
-							RuntimeOrigin::root(),
+							signed_origin.clone(),
 							wnd.clone().into(),
 							asset_id.clone().into(),
 						));
+
+						// Add liquidity
 						assert_ok!(AssetConversion::add_liquidity(
-							RuntimeOrigin::root(),
+							signed_origin.clone(),
 							wnd.clone().into(),
 							asset_id.clone().into(),
-							1 * UNITS,
-							2 * UNITS,
-							1,
-							1,
+							1 * UNITS,  // wnd amount
+							2 * UNITS,  // asset_id amount
+							1,          // min wnd
+							1,          // min asset_id
 							account.clone().into(),
 						));
 
-						give_assets.push((asset_id.clone(), 1 * UNITS).into());
-						receive_assets.push((asset_id, 2 * UNITS).into());
+						// Adjust give/receive to match pool ratio (1 wnd : 2 asset_id)
+						give_assets.push((wnd.clone().into(), 1 * UNITS).into());      // Give 1 wnd
+						receive_assets.push((asset_id.clone(), 2 * UNITS).into());    // Receive 2 asset_id
 					}
 
 					Ok((give_assets.into(), receive_assets.into()))