From 8834a9bf7dcb1e5a7498b7148787c544136609b2 Mon Sep 17 00:00:00 2001
From: Clara van Staden <claravanstaden64@gmail.com>
Date: Tue, 4 Feb 2025 14:55:44 +0200
Subject: [PATCH] Snowbridge: Remove fee amount check from tests (#7436)
Remove the specific fee amount checks in integration tests, since it
changes every time weights are regenerated.
---
.../bridge-hub-rococo/src/tests/snowbridge.rs | 16 ++++++++--------
.../bridge-hub-westend/src/tests/snowbridge.rs | 8 ++++----
2 files changed, 12 insertions(+), 12 deletions(-)
diff --git a/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-rococo/src/tests/snowbridge.rs b/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-rococo/src/tests/snowbridge.rs
index 6364ff9fe95..7f242bab5a9 100644
--- a/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-rococo/src/tests/snowbridge.rs
+++ b/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-rococo/src/tests/snowbridge.rs
@@ -498,8 +498,8 @@ fn send_weth_asset_from_asset_hub_to_ethereum() {
assert!(
events.iter().any(|event| matches!(
event,
- RuntimeEvent::Balances(pallet_balances::Event::Minted { who, amount })
- if *who == TREASURY_ACCOUNT.into() && *amount == 16903333
+ RuntimeEvent::Balances(pallet_balances::Event::Minted { who, amount: _amount })
+ if *who == TREASURY_ACCOUNT.into()
)),
"Snowbridge sovereign takes local fee."
);
@@ -507,8 +507,8 @@ fn send_weth_asset_from_asset_hub_to_ethereum() {
assert!(
events.iter().any(|event| matches!(
event,
- RuntimeEvent::Balances(pallet_balances::Event::Minted { who, amount })
- if *who == assethub_sovereign && *amount == 2680000000000,
+ RuntimeEvent::Balances(pallet_balances::Event::Minted { who, amount: _amount })
+ if *who == assethub_sovereign
)),
"AssetHub sovereign takes remote fee."
);
@@ -668,8 +668,8 @@ fn send_eth_asset_from_asset_hub_to_ethereum_and_back() {
assert!(
events.iter().any(|event| matches!(
event,
- RuntimeEvent::Balances(pallet_balances::Event::Minted { who, amount })
- if *who == TREASURY_ACCOUNT.into() && *amount == 16903333
+ RuntimeEvent::Balances(pallet_balances::Event::Minted { who, amount: _amount })
+ if *who == TREASURY_ACCOUNT.into()
)),
"Snowbridge sovereign takes local fee."
);
@@ -677,8 +677,8 @@ fn send_eth_asset_from_asset_hub_to_ethereum_and_back() {
assert!(
events.iter().any(|event| matches!(
event,
- RuntimeEvent::Balances(pallet_balances::Event::Minted { who, amount })
- if *who == assethub_sovereign && *amount == 2680000000000,
+ RuntimeEvent::Balances(pallet_balances::Event::Minted { who, amount: _amount })
+ if *who == assethub_sovereign
)),
"AssetHub sovereign takes remote fee."
);
diff --git a/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/snowbridge.rs b/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/snowbridge.rs
index 15ca3a5cf1b..6789aae83ff 100644
--- a/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/snowbridge.rs
+++ b/cumulus/parachains/integration-tests/emulated/tests/bridges/bridge-hub-westend/src/tests/snowbridge.rs
@@ -902,8 +902,8 @@ fn send_weth_asset_from_asset_hub_to_ethereum() {
assert!(
events.iter().any(|event| matches!(
event,
- RuntimeEvent::Balances(pallet_balances::Event::Minted { who, amount })
- if *who == TreasuryAccount::get().into() && *amount == 5071000000
+ RuntimeEvent::Balances(pallet_balances::Event::Minted { who, amount: _amount })
+ if *who == TreasuryAccount::get().into()
)),
"Snowbridge sovereign takes local fee."
);
@@ -911,8 +911,8 @@ fn send_weth_asset_from_asset_hub_to_ethereum() {
assert!(
events.iter().any(|event| matches!(
event,
- RuntimeEvent::Balances(pallet_balances::Event::Minted { who, amount })
- if *who == assethub_sovereign && *amount == 2680000000000,
+ RuntimeEvent::Balances(pallet_balances::Event::Minted { who, amount: _amount })
+ if *who == assethub_sovereign
)),
"AssetHub sovereign takes remote fee."
);
--
GitLab