feat: consistent transaction fees

Cargo.toml

@@ -38,6 +38,7 @@ members = [
   'pallets/democracy',
   'runtime/hydradx/src/evm/evm-utility/macro',
   'pallets/referrals',
+  'pallets/dynamic-evm-fee'
 ]
 
 [workspace.dependencies]
@@ -60,6 +61,7 @@ pallet-currencies = { path = "pallets/currencies", default-features = false }
 pallet-dca = { path = "pallets/dca", default-features = false }
 pallet-duster = { path = "pallets/duster", default-features = false }
 pallet-dynamic-fees = { path = "pallets/dynamic-fees", default-features = false }
+pallet-dynamic-evm-fee = { path = "pallets/dynamic-evm-fee", default-features = false }
 pallet-ema-oracle = { path = "pallets/ema-oracle", default-features = false }
 pallet-genesis-history = { path = "pallets/genesis-history", default-features = false }
 pallet-liquidity-mining = { path = "pallets/liquidity-mining", default-features = false }

Makefile

@@ -1,3 +1,10 @@
+UNAME_S := $(shell uname -s)
+ifeq ($(UNAME_S),Darwin)
+	SHASUM = shasum -a 256
+else
+	SHASUM = sha256sum
+endif
+
 .PHONY: build
 build:
 	cargo build --release --locked
@@ -56,9 +63,9 @@ docker:
 	docker build -t hydra-dx .
 
 checksum:
-	sha256sum target/release/hydradx > target/release/hydradx.sha256
+	$(SHASUM) target/release/hydradx > target/release/hydradx.sha256
 	cp target/release/wbuild/hydradx-runtime/hydradx_runtime.compact.compressed.wasm target/release/
-	sha256sum target/release/hydradx_runtime.compact.compressed.wasm > target/release/hydradx_runtime.compact.compressed.wasm.sha256
+	$(SHASUM) target/release/hydradx_runtime.compact.compressed.wasm > target/release/hydradx_runtime.compact.compressed.wasm.sha256
 
 release: build checksum
 

integration-tests/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "runtime-integration-tests"
-version = "1.17.2"
+version = "1.17.3"
 description = "Integration tests"
 authors = ["GalacticCouncil"]
 edition = "2021"
@@ -35,6 +35,7 @@ pallet-relaychain-info = { workspace = true }
 pallet-route-executor = { workspace = true}
 pallet-dca = { workspace = true}
 pallet-dynamic-fees = { workspace = true }
+pallet-dynamic-evm-fee = { workspace = true }
 pallet-staking = { workspace = true}
 pallet-lbp = { workspace = true}
 pallet-xyk = { workspace = true}
@@ -127,8 +128,9 @@ polkadot-runtime = { workspace = true }
 [dev-dependencies]
 hex-literal = "0.4.1"
 pretty_assertions = "1.2.1"
+pallet-relaychain-info = { workspace = true }
 xcm-emulator = { workspace = true }
-
+test-utils = { workspace = true }
 [features]
 default = ["std"]
 std = [
@@ -194,6 +196,7 @@ std = [
     "hydradx-runtime/std",
     "pallet-staking/std",
     "scraper/std",
+    "pallet-dynamic-evm-fee/std"
 ]
 
 # we don't include integration tests when benchmarking feature is enabled

integration-tests/src/cross_chain_transfer.rs

@@ -53,16 +53,16 @@ fn hydra_should_receive_asset_when_transferred_from_polkadot_relay_chain() {
 		);
 	});
 
-	let fees = 401884032343;
+	let fees = 66980672057;
 	Hydra::execute_with(|| {
-		assert_eq!(
-			hydradx_runtime::Tokens::free_balance(1, &AccountId::from(BOB)),
-			BOB_INITIAL_NATIVE_BALANCE + 300 * UNITS - fees
-		);
 		assert_eq!(
 			hydradx_runtime::Tokens::free_balance(1, &hydradx_runtime::Treasury::account_id()),
 			fees
 		);
+		assert_eq!(
+			hydradx_runtime::Tokens::free_balance(1, &AccountId::from(BOB)),
+			BOB_INITIAL_NATIVE_BALANCE + 300 * UNITS - fees
+		);
 	});
 }
 
@@ -143,7 +143,7 @@ fn hydra_should_receive_asset_when_transferred_from_acala() {
 		);
 	});
 
-	let fee = 321507225875;
+	let fee = 53584537646;
 	Hydra::execute_with(|| {
 		assert_eq!(
 			hydradx_runtime::Tokens::free_balance(ACA, &AccountId::from(BOB)),
@@ -336,7 +336,7 @@ fn claim_trapped_asset_should_work() {
 	Hydra::execute_with(|| {
 		assert_eq!(
 			hydradx_runtime::Tokens::free_balance(1, &AccountId::from(BOB)),
-			1000 * UNITS + 29_758_869_580_594
+			1_029_959_811_596_766 //1000 * UNITS + 30 * UNITS - fee
 		);
 
 		let origin = MultiLocation::new(1, X1(Parachain(ACALA_PARA_ID)));

integration-tests/src/dca.rs

@@ -443,7 +443,7 @@ mod omnipool {
 			run_to_block(11, 40);
 
 			//Assert
-			assert_balance!(ALICE.into(), DAI, ALICE_INITIAL_DAI_BALANCE + 600 * UNITS);
+			assert_balance!(ALICE.into(), DAI, ALICE_INITIAL_DAI_BALANCE + 700 * UNITS);
 
 			//Because the last trade is not enough for a whole trade, it is returned to the user
 			let amount_in = 140_421_094_431_120;

integration-tests/src/evm.rs

@@ -22,7 +22,7 @@ use sp_runtime::{traits::SignedExtension, FixedU128, Permill};
 use std::borrow::Cow;
 use xcm_emulator::TestExt;
 
-const TREASURY_ACCOUNT_INIT_BALANCE: Balance = 1000 * UNITS;
+pub const TREASURY_ACCOUNT_INIT_BALANCE: Balance = 1000 * UNITS;
 
 mod currency_precompile {
 	use super::*;
@@ -604,12 +604,16 @@ fn dispatch_should_respect_call_filter() {
 		);
 	});
 }
-
 #[test]
 fn compare_fee_between_evm_and_native_omnipool_calls() {
 	TestNet::reset();
 
 	Hydra::execute_with(|| {
+		//Set up to idle state where the chain is not utilized at all
+		pallet_transaction_payment::pallet::NextFeeMultiplier::<hydradx_runtime::Runtime>::put(
+			hydradx_runtime::MinimumMultiplier::get(),
+		);
+
 		//Set alice with as fee currency and fund it
 		assert_ok!(hydradx_runtime::MultiTransactionPayment::set_currency(
 			hydradx_runtime::RuntimeOrigin::signed(ALICE.into()),
@@ -619,7 +623,7 @@ fn compare_fee_between_evm_and_native_omnipool_calls() {
 			hydradx_runtime::RuntimeOrigin::root(),
 			ALICE.into(),
 			WETH,
-			100 * UNITS as i128,
+			1000000000 * UNITS as i128,
 		));
 
 		//Fund evm account with HDX to dispatch omnipool sell
@@ -644,6 +648,8 @@ fn compare_fee_between_evm_and_native_omnipool_calls() {
 			});
 
 		let gas_limit = 1000000;
+		let gas_price = hydradx_runtime::DynamicEvmFee::min_gas_price();
+
 		//Execute omnipool via EVM
 		assert_ok!(EVM::call(
 			evm_signed_origin(evm_address()),
@@ -652,21 +658,20 @@ fn compare_fee_between_evm_and_native_omnipool_calls() {
 			omni_sell.encode(),
 			U256::from(0),
 			gas_limit,
-			gas_price(),
+			gas_price.0 * 10,
 			None,
 			Some(U256::zero()),
 			[].into(),
 		));
 
 		//Pre dispatch the native omnipool call - so withdrawring only the fees for the execution
 		let info = omni_sell.get_dispatch_info();
-		let len: usize = 1;
 		assert_ok!(
 			pallet_transaction_payment::ChargeTransactionPayment::<hydradx_runtime::Runtime>::from(0).pre_dispatch(
 				&AccountId::from(ALICE),
 				&omni_sell,
 				&info,
-				len,
+				crate::fee_calculation::SWAP_ENCODED_LEN as usize,
 			)
 		);
 
@@ -690,7 +695,7 @@ fn compare_fee_between_evm_and_native_omnipool_calls() {
 	})
 }
 
-fn init_omnipool_with_oracle_for_block_10() {
+pub fn init_omnipool_with_oracle_for_block_10() {
 	init_omnipol();
 	//do_trade_to_populate_oracle(DAI, HDX, UNITS);
 	set_relaychain_block_number(10);
@@ -743,10 +748,10 @@ pub fn init_omnipol() {
 
 // TODO: test that we charge approximatelly same fee on evm as with extrinsics directly
 
-const DISPATCH_ADDR: H160 = addr(1025);
+pub const DISPATCH_ADDR: H160 = addr(1025);
 
-fn gas_price() -> U256 {
-	U256::from(8 * 10_u128.pow(7))
+pub fn gas_price() -> U256 {
+	U256::from(hydradx_runtime::evm::DEFAULT_BASE_FEE_PER_GAS)
 }
 
 fn create_dispatch_handle(data: Vec<u8>) -> MockHandle {