refactor: Synchronize MinGasFee

contracts/wasm/testcore/go/testcoreimpl/funcs.go

@@ -18,10 +18,10 @@ const (
 
 	MsgDoNothing     = "========== doing nothing"
 	MsgFailOnPurpose = "failing on purpose"
-	MsgFullPanic     = "========== panic FULL ENTRY POINT ========="
+	MsgFullPanic     = "========== panic FULL ENTRY POINT =========="
 	MsgJustView      = "calling empty view entry point"
 
-	MsgViewPanic = "========== panic VIEW ========="
+	MsgViewPanic = "========== panic VIEW =========="
 )
 
 func funcCallOnChain(ctx wasmlib.ScFuncContext, f *CallOnChainContext) {
@@ -242,13 +242,10 @@ func funcWithdrawFromChain(ctx wasmlib.ScFuncContext, f *WithdrawFromChainContex
 	transfer := wasmlib.ScTransferFromBalances(ctx.Allowance())
 	ctx.TransferAllowed(ctx.AccountID(), transfer)
 
-	// This is just a test contract, but normally these numbers should
-	// be parameters because there is no way for the contract to figure
-	// out the gas fees on the other chain, and it's also silly to run
-	// the costly calculation to determine storage deposit every time
-	// unless absolutely necessary. Better to just make sure that the
-	// storage deposit is large enough, since it will be returned anyway.
-	const gasFee = wasmlib.MinGasFee
+	var gasReserveTransferAccountToChain = wasmlib.MinGasFee
+	if f.Params.GasReserveTransferAccountToChain().Exists() {
+		gasReserveTransferAccountToChain = f.Params.GasReserveTransferAccountToChain().Value()
+	}
 	var gasReserve = wasmlib.MinGasFee
 	if f.Params.GasReserve().Exists() {
 		gasReserve = f.Params.GasReserve().Value()
@@ -261,7 +258,7 @@ func funcWithdrawFromChain(ctx wasmlib.ScFuncContext, f *WithdrawFromChainContex
 	// NOTE: make sure you READ THE DOCS before calling this function
 	xfer := coreaccounts.ScFuncs.TransferAccountToChain(ctx)
 	xfer.Params.GasReserve().SetValue(gasReserve)
-	xfer.Func.TransferBaseTokens(storageDeposit + gasFee + gasReserve).
+	xfer.Func.TransferBaseTokens(storageDeposit + gasReserveTransferAccountToChain + gasReserve).
 		AllowanceBaseTokens(withdrawal + storageDeposit + gasReserve).
 		PostToChain(targetChain)
 }

contracts/wasm/testcore/rs/testcoreimpl/src/funcs.rs

@@ -8,8 +8,8 @@ use crate::*;
 
 const CONTRACT_NAME_DEPLOYED: &str = "exampleDeployTR";
 const MSG_CORE_ONLY_PANIC: &str = "========== core only =========";
-const MSG_FULL_PANIC: &str = "========== panic FULL ENTRY POINT =========";
-const MSG_VIEW_PANIC: &str = "========== panic VIEW =========";
+const MSG_FULL_PANIC: &str = "========== panic FULL ENTRY POINT ==========";
+const MSG_VIEW_PANIC: &str = "========== panic VIEW ==========";
 
 pub fn func_call_on_chain(ctx: &ScFuncContext, f: &CallOnChainContext) {
     let param_int = f.params.n().value();
@@ -255,13 +255,11 @@ pub fn func_withdraw_from_chain(ctx: &ScFuncContext, f: &WithdrawFromChainContex
     let transfer = ScTransfer::from_balances(&ctx.allowance());
     ctx.transfer_allowed(&ctx.account_id(), &transfer);
 
-    // This is just a test contract, but normally these numbers should
-    // be parameters because there is no way for the contract to figure
-    // out the gas fees on the other chain, and it's also silly to run
-    // the costly calculation to determine storage deposit every time
-    // unless absolutely necessary. Better to just make sure that the
-    // storage deposit is large enough, since it will be returned anyway.
-    let gas_fee: u64 = MIN_GAS_FEE;
+    let mut gas_reserve_transfer_account_to_chain: u64 = MIN_GAS_FEE;
+    if f.params.gas_reserve_transfer_account_to_chain().exists() {
+        gas_reserve_transfer_account_to_chain =
+            f.params.gas_reserve_transfer_account_to_chain().value();
+    }
     let mut gas_reserve: u64 = MIN_GAS_FEE;
     if f.params.gas_reserve().exists() {
         gas_reserve = f.params.gas_reserve().value();
@@ -275,7 +273,7 @@ pub fn func_withdraw_from_chain(ctx: &ScFuncContext, f: &WithdrawFromChainContex
     let xfer = coreaccounts::ScFuncs::transfer_account_to_chain(ctx);
     xfer.params.gas_reserve().set_value(gas_reserve);
     xfer.func
-        .transfer_base_tokens(storage_deposit + gas_fee + gas_reserve)
+        .transfer_base_tokens(storage_deposit + gas_reserve_transfer_account_to_chain + gas_reserve)
         .allowance_base_tokens(withdrawal + storage_deposit + gas_reserve)
         .post_to_chain(target_chain);
 }

contracts/wasm/testcore/schema.yaml

@@ -94,6 +94,7 @@ funcs:
       chainID: ChainID
       baseTokens: Uint64
       gasReserve: Uint64?
+      gasReserveTransferAccountToChain: Uint64?
 
 views:
   checkContextFromViewEP:

contracts/wasm/testcore/test/2chains_test.go

@@ -125,11 +125,16 @@ func Test2Chains(t *testing.T) {
 
 		// allowance for accounts.transferAccountToChain(): SD + GAS1 + GAS2
 		xferDeposit := wasmlib.StorageDeposit
-		xferAllowance := xferDeposit + wasmlib.MinGasFee + wasmlib.MinGasFee
+		const gasFeeTransferAccountToChain = 10 * wasmlib.MinGasFee
+		const gasReserve = 10 * wasmlib.MinGasFee
+		const gasWithdrawFromChain = 10 * wasmlib.MinGasFee
+		xferAllowance := xferDeposit + gasReserve + gasFeeTransferAccountToChain
 		f := testcore.ScFuncs.WithdrawFromChain(ctx2.Sign(user))
 		f.Params.ChainID().SetValue(ctx1.CurrentChainID())
 		f.Params.BaseTokens().SetValue(withdrawalAmount)
-		f.Func.TransferBaseTokens(xferAllowance + isc.Million).
+		f.Params.GasReserve().SetValue(gasReserve)
+		f.Params.GasReserveTransferAccountToChain().SetValue(gasFeeTransferAccountToChain)
+		f.Func.TransferBaseTokens(xferAllowance + gasWithdrawFromChain).
 			AllowanceBaseTokens(xferAllowance).Post()
 		require.NoError(t, ctx2.Err)
 
@@ -153,21 +158,21 @@ func Test2Chains(t *testing.T) {
 		// chain2.testcore account will be credited with SD+GAS1+GAS2, pay actual GAS1,
 		// and be debited by SD+GAS2+'withdrawalAmount'
 		bal1.UpdateFeeBalances(ctxAcc1.GasFee)
-		bal1.Add(testcore2, xferDeposit+wasmlib.MinGasFee+wasmlib.MinGasFee-ctxAcc1.GasFee-xferDeposit-wasmlib.MinGasFee-withdrawalAmount)
+		bal1.Add(testcore2, xferDeposit+gasWithdrawFromChain+gasWithdrawFromChain-ctxAcc1.GasFee-xferDeposit-gasReserve-withdrawalAmount)
 		// verify these changes against the actual chain1 account balances
 		bal1.VerifyBalances(t)
 
-		userL1 -= xferAllowance + isc.Million
+		userL1 -= xferAllowance + gasWithdrawFromChain
 		require.Equal(t, userL1, user.Balance())
 
 		// The gas fees will be credited to chain1.Originator
 		bal2.UpdateFeeBalances(withdrawalReceipt.GasFeeCharged)
 		bal2.UpdateFeeBalances(transferReceipt.GasFeeCharged)
 		// deduct coretest.WithdrawFromChain() gas fee from user's cool million
-		bal2.Add(user, isc.Million-withdrawalReceipt.GasFeeCharged)
+		bal2.Add(user, gasWithdrawFromChain-withdrawalReceipt.GasFeeCharged)
 		// chain2.accounts1 will be credited with SD+GAS2+'withdrawalAmount', pay actual GAS2,
 		// and be debited by SD+'withdrawalAmount', leaving zero
-		bal2.Add(accounts1, xferDeposit+wasmlib.MinGasFee+withdrawalAmount-transferReceipt.GasFeeCharged-xferDeposit-withdrawalAmount)
+		bal2.Add(accounts1, xferDeposit+gasReserve+withdrawalAmount-transferReceipt.GasFeeCharged-xferDeposit-withdrawalAmount)
 		// chain2.testcore account receives the withdrawn tokens and storage deposit
 		bal2.Account += withdrawalAmount + xferDeposit
 		// verify these changes against the actual chain2 account balances

contracts/wasm/testcore/ts/testcoreimpl/funcs.ts

@@ -9,8 +9,8 @@ import * as sc from "../testcore/index";
 
 const CONTRACT_NAME_DEPLOYED = "exampleDeployTR";
 const MSG_CORE_ONLY_PANIC = "========== core only =========";
-const MSG_FULL_PANIC = "========== panic FULL ENTRY POINT =========";
-const MSG_VIEW_PANIC = "========== panic VIEW =========";
+const MSG_FULL_PANIC = "========== panic FULL ENTRY POINT ==========";
+const MSG_VIEW_PANIC = "========== panic VIEW ==========";
 
 export function funcCallOnChain(ctx: wasmlib.ScFuncContext, f: sc.CallOnChainContext): void {
     let paramInt = f.params.n().value();
@@ -228,13 +228,10 @@ export function funcWithdrawFromChain(ctx: wasmlib.ScFuncContext, f: sc.Withdraw
     const transfer = wasmlib.ScTransfer.fromBalances(ctx.allowance());
     ctx.transferAllowed(ctx.accountID(), transfer);
 
-    // This is just a test contract, but normally these numbers should
-    // be parameters because there is no way for the contract to figure
-    // out the gas fees on the other chain, and it's also silly to run
-    // the costly calculation to determine storage deposit every time
-    // unless absolutely necessary. Better to just make sure that the
-    // storage deposit is large enough, since it will be returned anyway.
-    const gasFee: u64 = wasmlib.MinGasFee;
+    let gasReserveTransferAccountToChain: u64 = wasmlib.MinGasFee;
+    if (f.params.gasReserveTransferAccountToChain().exists()) {
+        gasReserveTransferAccountToChain = f.params.gasReserveTransferAccountToChain().value();
+    }
     let gasReserve: u64 = wasmlib.MinGasFee;
     if (f.params.gasReserve().exists()) {
         gasReserve = f.params.gasReserve().value();
@@ -247,7 +244,7 @@ export function funcWithdrawFromChain(ctx: wasmlib.ScFuncContext, f: sc.Withdraw
     // NOTE: make sure you READ THE DOCS before calling this function
     const xfer = coreaccounts.ScFuncs.transferAccountToChain(ctx);
     xfer.params.gasReserve().setValue(gasReserve);
-    xfer.func.transferBaseTokens(storageDeposit + gasFee + gasReserve)
+    xfer.func.transferBaseTokens(storageDeposit + gasReserveTransferAccountToChain + gasReserve)
         .allowanceBaseTokens(withdrawal + storageDeposit + gasReserve)
         .postToChain(targetChain);
 }

packages/chain/cons/cons_test.go

@@ -34,9 +34,9 @@ import (
 	"github.com/iotaledger/wasp/packages/vm/core/accounts"
 	"github.com/iotaledger/wasp/packages/vm/core/coreprocessors"
 	"github.com/iotaledger/wasp/packages/vm/core/migrations/allmigrations"
+	"github.com/iotaledger/wasp/packages/vm/gas"
 	"github.com/iotaledger/wasp/packages/vm/processors"
 	"github.com/iotaledger/wasp/packages/vm/vmimpl"
-	"github.com/iotaledger/wasp/packages/wasmvm/wasmlib/go/wasmlib"
 )
 
 // Here we run a single consensus instance, step by step with
@@ -347,8 +347,7 @@ func testChained(t *testing.T, n, f, b int) {
 				inccounter.FuncIncCounter.Hname(),
 				dict.New(),
 				uint64(i*reqPerBlock+ii),
-				// FIXME may user other MinGasFee instead of wasmlib.MinGasFee
-				wasmlib.MinGasFee,
+				gas.LimitsDefault.MinGasPerRequest,
 			).Sign(scClient)
 			reqs = append(reqs, scRequest)
 			incTotal++

packages/solo/ledgerl1l2.go

@@ -203,7 +203,8 @@ func (fp *foundryParams) CreateFoundry() (uint32, iotago.NativeTokenID, error) {
 		user = fp.user
 	}
 	req := CallParamsFromDict(accounts.Contract.Name, accounts.FuncFoundryCreateNew.Name, par).
-		WithAllowance(isc.NewAssetsBaseTokens(allowanceForFoundryStorageDeposit))
+		WithAllowance(isc.NewAssetsBaseTokens(allowanceForFoundryStorageDeposit)).
+		AddBaseTokens(allowanceForFoundryStorageDeposit)
 
 	gas, _, err := fp.ch.EstimateGasOnLedger(req, user, true)
 	if err != nil {

packages/testutil/testchain/test_chain_ledger.go

@@ -21,7 +21,6 @@ import (
 	"github.com/iotaledger/wasp/packages/vm/core/migrations/allmigrations"
 	"github.com/iotaledger/wasp/packages/vm/core/root"
 	"github.com/iotaledger/wasp/packages/vm/gas"
-	"github.com/iotaledger/wasp/packages/wasmvm/wasmlib/go/wasmlib"
 )
 
 ////////////////////////////////////////////////////////////////////////////////
@@ -87,8 +86,7 @@ func (tcl *TestChainLedger) MakeTxAccountsDeposit(account *cryptolib.KeyPair) []
 				Metadata: &isc.SendMetadata{
 					TargetContract: accounts.Contract.Hname(),
 					EntryPoint:     accounts.FuncDeposit.Hname(),
-					// FIXME may user other MinGasFee instead of wasmlib.MinGasFee
-					GasBudget: wasmlib.MinGasFee,
+					GasBudget:      2 * gas.LimitsDefault.MinGasPerRequest,
 				},
 			},
 		},
@@ -119,8 +117,7 @@ func (tcl *TestChainLedger) MakeTxDeployIncCounterContract() []isc.Request {
 						root.ParamName:        inccounter.Contract.Name,
 						inccounter.VarCounter: 0,
 					}),
-					// FIXME may user other MinGasFee instead of wasmlib.MinGasFee
-					GasBudget: wasmlib.MinGasFee,
+					GasBudget: 2 * gas.LimitsDefault.MinGasPerRequest,
 				},
 			},
 		},

packages/vm/core/accounts/impl.go

@@ -11,7 +11,7 @@ import (
 	"github.com/iotaledger/wasp/packages/util"
 	"github.com/iotaledger/wasp/packages/vm"
 	"github.com/iotaledger/wasp/packages/vm/core/errors/coreerrors"
-	"github.com/iotaledger/wasp/packages/wasmvm/wasmlib/go/wasmlib"
+	"github.com/iotaledger/wasp/packages/vm/gas"
 )
 
 func CommonAccount() isc.AgentID {
@@ -176,7 +176,7 @@ func transferAccountToChain(ctx isc.Sandbox) dict.Dict {
 	assets := allowance.Clone()
 
 	// deduct the gas reserve GAS2 from the allowance, if possible
-	gasReserve := ctx.Params().MustGetUint64(ParamGasReserve, wasmlib.MinGasFee)
+	gasReserve := ctx.Params().MustGetUint64(ParamGasReserve, gas.LimitsDefault.MinGasPerRequest)
 	if allowance.BaseTokens < gasReserve {
 		panic(ErrNotEnoughAllowance)
 	}

packages/vm/core/evm/evmimpl/impl.go

@@ -29,7 +29,6 @@ import (
 	"github.com/iotaledger/wasp/packages/vm/core/evm/iscmagic"
 	"github.com/iotaledger/wasp/packages/vm/core/governance"
 	"github.com/iotaledger/wasp/packages/vm/gas"
-	"github.com/iotaledger/wasp/packages/wasmvm/wasmlib/go/wasmlib"
 )
 
 var Processor = evm.Contract.Processor(nil,
@@ -254,14 +253,14 @@ func registerERC20NativeTokenOnRemoteChain(ctx isc.Sandbox) dict.Dict {
 				evm.FieldFoundryTokenScheme: codec.EncodeTokenScheme(tokenScheme),
 			},
 			// FIXME why does this gas budget is higher than the allowance below
-			GasBudget: 5 * wasmlib.MinGasFee,
+			GasBudget: 50 * gas.LimitsDefault.MinGasPerRequest,
 		},
 	}
 	sd := ctx.EstimateRequiredStorageDeposit(req)
 	// this request is sent by contract account,
 	// so we move enough allowance for the gas fee below in the req.Assets.AddBaseTokens() function call
-	ctx.TransferAllowedFunds(ctx.AccountID(), isc.NewAssetsBaseTokens(sd+wasmlib.MinGasFee))
-	req.Assets.AddBaseTokens(sd + wasmlib.MinGasFee)
+	ctx.TransferAllowedFunds(ctx.AccountID(), isc.NewAssetsBaseTokens(sd+10*gas.LimitsDefault.MinGasPerRequest))
+	req.Assets.AddBaseTokens(sd + 10*gas.LimitsDefault.MinGasPerRequest)
 	ctx.Send(req)
 
 	return nil

packages/vm/core/evm/evmtest/utils_test.go

@@ -33,7 +33,6 @@ import (
 	"github.com/iotaledger/wasp/packages/vm/core/evm/iscmagic"
 	"github.com/iotaledger/wasp/packages/vm/core/governance"
 	"github.com/iotaledger/wasp/packages/vm/gas"
-	"github.com/iotaledger/wasp/packages/wasmvm/wasmlib/go/wasmlib"
 )
 
 type soloChainEnv struct {
@@ -373,9 +372,8 @@ func (e *soloChainEnv) registerERC20ExternalNativeToken(
 		evm.FieldTargetAddress:     codec.EncodeAddress(e.soloChain.ChainID.AsAddress()),
 	}).
 		// to cover sd and gas fee for the 'FuncRegisterERC20ExternalNativeToken' func call in 'FuncRegisterERC20NativeTokenOnRemoteChain'
-		WithAllowance(isc.NewAssetsBaseTokens(2*wasmlib.MinGasFee)).
-		// FIXME why this gas budget is less than the gas budget of the indirect call FuncRegisterERC20ExternalNativeToken
-		WithGasBudget(wasmlib.MinGasFee),
+		WithAllowance(isc.NewAssetsBaseTokens(20*gas.LimitsDefault.MinGasPerRequest)).
+		WithGasBudget(10*gas.LimitsDefault.MinGasPerRequest),
 		fromChain.OriginatorPrivateKey)
 	if err != nil {
 		return ret, err

packages/vm/core/testcore/custom_onledger_requests_test.go

@@ -16,7 +16,7 @@ import (
 	"github.com/iotaledger/wasp/packages/testutil/testmisc"
 	"github.com/iotaledger/wasp/packages/transaction"
 	"github.com/iotaledger/wasp/packages/vm/core/accounts"
-	"github.com/iotaledger/wasp/packages/wasmvm/wasmlib/go/wasmlib"
+	"github.com/iotaledger/wasp/packages/vm/gas"
 )
 
 func TestNoSenderFeature(t *testing.T) {
@@ -28,7 +28,9 @@ func TestNoSenderFeature(t *testing.T) {
 	// ----------------------------------------------------------------
 
 	// mint some NTs and withdraw them
-	err := ch.DepositAssetsToL2(isc.NewAssetsBaseTokens(10*isc.Million), wallet)
+	gasFee := 10 * gas.LimitsDefault.MinGasPerRequest
+	withdrawAmount := 3 * gas.LimitsDefault.MinGasPerRequest
+	err := ch.DepositAssetsToL2(isc.NewAssetsBaseTokens(withdrawAmount+gasFee), wallet)
 	require.NoError(t, err)
 	nativeTokenAmount := big.NewInt(123)
 	sn, nativeTokenID, err := ch.NewFoundryParams(1234).
@@ -40,8 +42,8 @@ func TestNoSenderFeature(t *testing.T) {
 	require.NoError(t, err)
 
 	// withdraw native tokens to L1
-	allowance := 5 * isc.Million
-	baseTokensToSend := allowance + wasmlib.MinGasFee
+	allowance := withdrawAmount
+	baseTokensToSend := allowance + gasFee
 	_, err = ch.PostRequestOffLedger(solo.NewCallParams(
 		accounts.Contract.Name, accounts.FuncWithdraw.Name,
 	).
@@ -51,7 +53,7 @@ func TestNoSenderFeature(t *testing.T) {
 			ID:     nativeTokenID,
 			Amount: nativeTokenAmount,
 		}).
-		WithGasBudget(wasmlib.MinGasFee),
+		WithGasBudget(gasFee),
 		wallet)
 	require.NoError(t, err)
 

packages/vm/core/testcore/sbtests/2chains_test.go

@@ -13,6 +13,7 @@ import (
 	"github.com/iotaledger/wasp/packages/vm/core/accounts"
 	"github.com/iotaledger/wasp/packages/vm/core/corecontracts"
 	"github.com/iotaledger/wasp/packages/vm/core/testcore/sbtests/sbtestsc"
+	"github.com/iotaledger/wasp/packages/vm/gas"
 	"github.com/iotaledger/wasp/packages/wasmvm/wasmlib/go/wasmlib"
 )
 
@@ -64,7 +65,7 @@ func test2Chains(t *testing.T, w bool) {
 
 	// send base tokens to contractAgentID2 (that is an entity of chain2) on chain1
 	const baseTokensCreditedToScOnChain1 = 10 * isc.Million
-	const creditBaseTokensToSend = baseTokensCreditedToScOnChain1 + wasmlib.MinGasFee
+	creditBaseTokensToSend := baseTokensCreditedToScOnChain1 + gas.LimitsDefault.MinGasPerRequest
 	_, err = chain1.PostRequestSync(solo.NewCallParams(
 		accounts.Contract.Name, accounts.FuncTransferAllowanceTo.Name,
 		accounts.ParamAgentID, contractAgentID2,
@@ -97,10 +98,10 @@ func test2Chains(t *testing.T, w bool) {
 	// make chain2 send a call to chain1 to withdraw base tokens
 	baseTokensToWithdrawFromChain1 := baseTokensCreditedToScOnChain1
 
-	const gasFee1 = wasmlib.MinGasFee
+	gasFeeTransferAccountToChain := 10 * gas.LimitsDefault.MinGasPerRequest
 	// gas reserve for the 'TransferAllowanceTo' func call in 'TransferAccountToChain' func call
-	const gasReserve = wasmlib.MinGasFee
-	const withdrawFeeGas = wasmlib.MinGasFee
+	gasReserve := 10 * gas.LimitsDefault.MinGasPerRequest
+	withdrawFeeGas := 10 * gas.LimitsDefault.MinGasPerRequest
 	const storageDeposit = wasmlib.StorageDeposit
 
 	// NOTE: make sure you READ THE DOCS for accounts.transferAccountToChain()
@@ -111,7 +112,7 @@ func test2Chains(t *testing.T, w bool) {
 	// the gas fees for the chain2.accounts.transferAccountToChain() request and the
 	// chain1.accounts.transferAllowanceTo() request.
 	// note that the storage deposit will be returned in the end
-	withdrawReqAllowance := storageDeposit + gasFee1 + gasReserve
+	withdrawReqAllowance := storageDeposit + gasFeeTransferAccountToChain + gasReserve
 
 	// also cover gas fee for `FuncWithdrawFromChain` on chain2
 	withdrawBaseTokensToSend := withdrawReqAllowance + withdrawFeeGas
@@ -121,6 +122,7 @@ func test2Chains(t *testing.T, w bool) {
 		sbtestsc.ParamChainID, chain1.ChainID,
 		sbtestsc.ParamBaseTokens, baseTokensToWithdrawFromChain1,
 		sbtestsc.ParamGasReserve, gasReserve,
+		sbtestsc.ParamGasReserveTransferAccountToChain, gasFeeTransferAccountToChain,
 	).
 		AddBaseTokens(withdrawBaseTokensToSend).
 		WithAllowance(isc.NewAssetsBaseTokens(withdrawReqAllowance)).
@@ -165,9 +167,11 @@ func test2Chains(t *testing.T, w bool) {
 	env.AssertL1BaseTokens(userAddress, utxodb.FundsFromFaucetAmount-creditBaseTokensToSend-withdrawBaseTokensToSend)
 	// on chain1 user only made the first transaction, so it is the same as its balance before 'WithdrawFromChain' function call
 	chain1.AssertL2BaseTokens(userAgentID, creditBaseTokensToSend-baseTokensCreditedToScOnChain1-chain1TransferAllowanceGas)
+	// gasFeeTransferAccountToChain is is used for paying the gas fee of the 'TransferAccountToChain' func call
+	// in 'WithdrawFromChain' func call
 	// gasReserve is used for paying the gas fee of the 'TransferAllowanceTo' func call in 'TransferAccountToChain' func call
 	// So the token left in contractAgentID2 on chain1 is the unused gas fee
-	chain1.AssertL2BaseTokens(contractAgentID2, gasReserve-chain1TransferAccountToChainGas)
+	chain1.AssertL2BaseTokens(contractAgentID2, gasFeeTransferAccountToChain-chain1TransferAccountToChainGas)
 	// tokens in 'withdrawBaseTokensToSend' amount are moved with the request from L1 to L2
 	// 'withdrawReqAllowance' is is the amount moved from chain1 to chain2 with the request
 	// 'baseTokensToWithdrawFromChain1' is the amount we assigned to withdraw in 'WithdrawFromChain' func call