Fix validate in xcm-bridge-hub-router

Snowfork · Apr 15, 2024 · 7973ce3 · 7973ce3
1 parent 68ab248
commit 7973ce3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/bridges/modules/xcm-bridge-hub-router/src/lib.rs b/bridges/modules/xcm-bridge-hub-router/src/lib.rs
@@ -341,8 +341,8 @@ impl<T: Config<I>, I: 'static> SendXcm for Pallet<T, I> {
 		// versioned message to the sibling bridge hub. However, the local bridge hub may have a
 		// higher XCM version than the remote `dest`. Once again, it is better to discard such
 		// messages here than at the bridge hub (e.g., to avoid losing funds).
-		let destination_version = T::DestinationVersion::get_version_for(dest_ref)
-			.ok_or(SendError::DestinationUnsupported)?;
+		let destination_version =
+			T::DestinationVersion::get_version_for(dest_ref).ok_or(SendError::NotApplicable)?;
 		let _ = VersionedXcm::from(xcm_ref.clone())
 			.into_version(destination_version)
 			.map_err(|()| SendError::DestinationUnsupported)?;

diff --git a/...chains/integration-tests/emulated/tests/bridges/bridge-hub-rococo/src/tests/snowbridge.rs b/...chains/integration-tests/emulated/tests/bridges/bridge-hub-rococo/src/tests/snowbridge.rs
@@ -388,10 +388,6 @@ fn send_weth_asset_from_asset_hub_to_ethereum() {
 
 	AssetHubRococo::force_default_xcm_version(Some(XCM_VERSION));
 	BridgeHubRococo::force_default_xcm_version(Some(XCM_VERSION));
-	AssetHubRococo::force_xcm_version(
-		Location::new(2, [GlobalConsensus(Ethereum { chain_id: CHAIN_ID })]),
-		XCM_VERSION,
-	);
 
 	BridgeHubRococo::fund_accounts(vec![(assethub_sovereign.clone(), INITIAL_FUND)]);
 	AssetHubRococo::fund_accounts(vec![(AssetHubRococoReceiver::get(), INITIAL_FUND)]);