From f081d8013ce37a2109750424d1ed615411d9056a Mon Sep 17 00:00:00 2001 From: Lasse Herskind <16536249+LHerskind@users.noreply.github.com> Date: Tue, 26 Nov 2024 23:36:32 +0000 Subject: [PATCH] fix: use current base fee for public fee payment (#10230) As #10172 got merged right after #10176 it had not run the new test against the updated master so the issue with fee payment was not caught. To fix it, we fetch the current base fees and use those in our settings. --- .../end-to-end/src/e2e_fees/public_payments.test.ts | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/yarn-project/end-to-end/src/e2e_fees/public_payments.test.ts b/yarn-project/end-to-end/src/e2e_fees/public_payments.test.ts index 1440629072f..83f4a4ef68b 100644 --- a/yarn-project/end-to-end/src/e2e_fees/public_payments.test.ts +++ b/yarn-project/end-to-end/src/e2e_fees/public_payments.test.ts @@ -1,5 +1,5 @@ import { type AccountWallet, type AztecAddress, PublicFeePaymentMethod } from '@aztec/aztec.js'; -import { type GasSettings } from '@aztec/circuits.js'; +import { GasSettings } from '@aztec/circuits.js'; import { type TokenContract as BananaCoin, type FPCContract } from '@aztec/noir-contracts.js'; import { expectMapping } from '../fixtures/utils.js'; @@ -38,12 +38,11 @@ describe('e2e_fees public_payment', () => { let initialSequencerGas: bigint; - let maxFee: bigint; - beforeEach(async () => { - maxFee = BigInt(20e9); - - expect(gasSettings.getFeeLimit().toBigInt()).toEqual(maxFee); + gasSettings = GasSettings.from({ + ...gasSettings, + maxFeesPerGas: await aliceWallet.getCurrentBaseFees(), + }); [ [initialAlicePublicBananas, initialBobPublicBananas, initialFPCPublicBananas],