Skip to content

Commit

Permalink
fix: use current base fee for public fee payment (#10230)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
LHerskind authored Nov 26, 2024
1 parent 15ffeea commit f081d80
Showing 1 changed file with 5 additions and 6 deletions.
11 changes: 5 additions & 6 deletions yarn-project/end-to-end/src/e2e_fees/public_payments.test.ts
Original file line number Diff line number Diff line change
@@ -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';
Expand Down Expand Up @@ -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],
Expand Down

0 comments on commit f081d80

Please sign in to comment.