From bd232880f11d3fd70e54f3fca1e1c331ae2354d3 Mon Sep 17 00:00:00 2001 From: AztecBot Date: Mon, 8 Jun 2026 15:49:45 +0000 Subject: [PATCH 1/2] test(ci): skip non-deterministic expand::test_avm_test_contract snapshot --- noir-projects/contract-snapshots/bootstrap.sh | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/noir-projects/contract-snapshots/bootstrap.sh b/noir-projects/contract-snapshots/bootstrap.sh index f4ae19760903..5f9549946cb3 100755 --- a/noir-projects/contract-snapshots/bootstrap.sh +++ b/noir-projects/contract-snapshots/bootstrap.sh @@ -20,7 +20,12 @@ function test { echo "INSTA_UPDATE is not permitted in CI. Run 'cargo insta accept' locally and commit." >&2 exit 1 fi - INSTA_UPDATE=no cargo test --quiet + # TODO(#23895): `expand::test_avm_test_contract` is skipped because `nargo expand` + # emits the generated contract-interface functions in hashmap-iteration order, which + # is not stable across nargo builds/runs. The snapshot is correct but the test fails + # intermittently in the merge queue on order-only diffs. Re-enable once noir makes the + # expand output order deterministic (e.g. sort interface fns by name/selector). + INSTA_UPDATE=no cargo test --quiet -- --skip expand::test_avm_test_contract } function test_cmds { From c4eb15fb1f757d165b8835bf6dc5f0a2f0d33340 Mon Sep 17 00:00:00 2001 From: Maxim Vezenov Date: Mon, 8 Jun 2026 11:53:38 -0400 Subject: [PATCH 2/2] Apply suggestion from @vezenovm --- noir-projects/contract-snapshots/bootstrap.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/noir-projects/contract-snapshots/bootstrap.sh b/noir-projects/contract-snapshots/bootstrap.sh index 5f9549946cb3..8f2a7d2e92ac 100755 --- a/noir-projects/contract-snapshots/bootstrap.sh +++ b/noir-projects/contract-snapshots/bootstrap.sh @@ -20,7 +20,7 @@ function test { echo "INSTA_UPDATE is not permitted in CI. Run 'cargo insta accept' locally and commit." >&2 exit 1 fi - # TODO(#23895): `expand::test_avm_test_contract` is skipped because `nargo expand` + # TODO(#12933 on noir-lang/noir): `expand::test_avm_test_contract` is skipped because `nargo expand` # emits the generated contract-interface functions in hashmap-iteration order, which # is not stable across nargo builds/runs. The snapshot is correct but the test fails # intermittently in the merge queue on order-only diffs. Re-enable once noir makes the