Skip to content

fix: nargo fmt for backported nested_utility tests#23117

Merged
Thunkar merged 2 commits into
backport-to-v4-next-stagingfrom
claudebox/ea70ef48e7a85dfe-3
May 9, 2026
Merged

fix: nargo fmt for backported nested_utility tests#23117
Thunkar merged 2 commits into
backport-to-v4-next-stagingfrom
claudebox/ea70ef48e7a85dfe-3

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

@AztecBot AztecBot commented May 8, 2026

Summary

CI3 on PR #23080 failed in the noir-projects build prep step because the new nested_utility_contract/src/test.nr (added by the backport of #23064 — feat(txe) authorize cross-contract utility calls) wasn't formatted by v4-next's pinned nargo. Three statements were committed multi-line that v4-next's nargo fmt --check wants on a single line.

CI log: http://ci.aztec-labs.com/0ef270f34a64bc9b

Fix

Apply nargo fmt exactly per the CI diff:

  • same_contract_utility_call_from_private_succeeds: collapse the let result: Field = env.call_private(...) to one line.
  • cross_contract_utility_call_from_utility_denied_by_default: collapse the env.execute_utility( ..., ) to one line.
  • cross_contract_utility_call_from_private_denied_by_default: collapse the env.call_private( account, ..., ) to one line.

The two *_with_authorization tests already use multi-line because their bodies are too wide for one line — left unchanged.

Test plan

ClaudeBox log: https://claudebox.work/s/ea70ef48e7a85dfe?run=3

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels May 8, 2026
@Thunkar Thunkar marked this pull request as ready for review May 9, 2026 07:19
@Thunkar Thunkar requested a review from nventuro as a code owner May 9, 2026 07:19
@Thunkar Thunkar merged commit e0366c9 into backport-to-v4-next-staging May 9, 2026
13 checks passed
@Thunkar Thunkar deleted the claudebox/ea70ef48e7a85dfe-3 branch May 9, 2026 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants