test: fund accounts in one transaction and raise integration concurrency by igamigo · Pull Request #2467 · 0xMiden/rust-sdk

igamigo · 2026-08-28T22:06:59Z

Speeds up CI funding by paying all accounts in one transaction (goes from ~76 min to ~51 min).
Fixes fee handling in the client: custom-script notes sync correctly (for this we basically remove the fee notes from the post-tx screening process), fee conversion info is handled by auth type without panics, empty auth args no longer suppress it, and TransactionRequest::with_auth_arg supports custom commitments.
Adjusts wallets - we only need a handful as it becomes a bottleneck as long as it's close ot the amount of concurrent tests (or to be more precise, the amount of ongoing funding txs)
Fixes flaky tests by polling for NTXs, deploying accounts when required, and making funder setup/commit handling reliable.
Cleans up test infrastructure: typed genesis.toml, CI node logs, workspace toml, and correct AggLayer fee denomination.
Pushes agglayer tests to their own job

Batches the funder's payments and the deploys' commit wait, and lifts the integration test group's thread cap from 2 to 4.

…n uses Extracts the native conversion info decision into one helper shared by transaction execution and note screening.

At 4 the per-test durations inflate ~1.65x and the network-transaction tests fail, so the rayon provers are outrunning the runner's cores.

… faucet The genesis accounts named the mock chain's fee faucet, so the ntx-builder could not execute their transactions and their notes went unconsumed.

Serializes a GenesisConfig with serde instead of concatenating TOML by hand, so adding fields no longer means escaping braces in a format string.

An account's funding note is held until it next transacts and consumed there, so one transaction deploys the account, funds it and does the test's work instead of a dedicated deploy plus a wait for it.

The CLI drives accounts from its own process, so the funds have to be in the vault, not in a note this client holds. The funder also waits for its payment to commit before releasing the wallet.

The node builds them on its own schedule, which stretches with load.

Tests that execute without submitting bypassed the injection and hit the fee with an empty vault.

…ersion info

…rate The CLI, the bench and the integration tests all reached into the integration test binary for ClientConfig and the fee funder.

igamigo added 16 commits August 28, 2026 19:01

test: fund accounts in one transaction and raise integration concurrency

9d4b7cb

Batches the funder's payments and the deploys' commit wait, and lifts the integration test group's thread cap from 2 to 4.

fix(rust-client): screen notes under the fee conversion info executio…

ecc52c4

…n uses Extracts the native conversion info decision into one helper shared by transaction execution and note screening.

test: cap integration concurrency at 3

f479600

At 4 the per-test durations inflate ~1.65x and the network-transaction tests fail, so the rayon provers are outrunning the runner's cores.

fix(test-node): denominate the agglayer fee policy in the chain's fee…

2c831c6

… faucet The genesis accounts named the mock chain's fee faucet, so the ntx-builder could not execute their transactions and their notes went unconsumed.

refactor(test-node): build genesis.toml from typed structs

a9c63fc

Serializes a GenesisConfig with serde instead of concatenating TOML by hand, so adding fields no longer means escaping braces in a format string.

test: fold account funding into the account's first transaction

81575e0

An account's funding note is held until it next transacts and consumed there, so one transaction deploys the account, funds it and does the test's work instead of a dedicated deploy plus a wait for it.

ci: upload the test node's logs

2a7971d

chore: inherit toml from the workspace

59cd2f8

fix(test): deploy CLI accounts eagerly and settle funder payments

0015b4a

The CLI drives accounts from its own process, so the funds have to be in the vault, not in a note this client holds. The funder also waits for its payment to commit before releasing the wallet.

test: poll for agglayer network transactions instead of fixed waits

862d13f

The node builds them on its own schedule, which stretches with load.

test: fold funding into execute_transaction too

15ba331

Tests that execute without submitting bypassed the injection and hit the fee with an empty vault.

test: deploy Alice before the non-standard mint

2fdfd53

test: deploy accounts that mint as their first transaction

3f1c23c

test: deploy before the custom transaction

e309a17

fix(rust-client): classify auth components before committing fee conv…

042e631

…ersion info

test: run agglayer tests serially with a longer timeout

6a53966

igamigo marked this pull request as ready for review August 31, 2026 02:05

igamigo added 9 commits August 30, 2026 23:09

test: raise the bridge-in note wait budget

aa35dec

ci: run agglayer tests in their own job

87b9510

chore: gate the test-only request helper behind testing

5305771

refactor(test): extract the shared client test harness into its own c…

543ffaf

…rate The CLI, the bench and the integration tests all reached into the integration test binary for ClientConfig and the fee funder.

chore: drop deps the harness extraction left unused

fc0ae16

docs: tighten the comments this branch added

b4a2cce

docs: note that upstream #3710 would remove this workaround

fd44807

test: give the node's prover room for network transaction proofs

50414e0

test: declare 16 funder wallets at genesis

6f649a4

igamigo merged commit a771b6d into jmunoz-integration-test-with-fees Aug 31, 2026
19 checks passed

igamigo deleted the igamigo-improve-tests-startup branch August 31, 2026 04:14

igamigo mentioned this pull request Aug 31, 2026

test: run integration test with fees #2455

Open

Navigation Menu

Uh oh!

test: fund accounts in one transaction and raise integration concurrency - #2467

test: fund accounts in one transaction and raise integration concurrency#2467
igamigo merged 25 commits into
jmunoz-integration-test-with-feesfrom
igamigo-improve-tests-startup

igamigo commented Aug 28, 2026 •

edited

Loading

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

igamigo commented Aug 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

igamigo commented Aug 28, 2026 •

edited

Loading