Skip to content

test: cover checkpoint boundary variants#24723

Draft
AztecBot wants to merge 1 commit into
nextfrom
cb/rollup-audit-remediation-1628-checkpoint-boundary-tests
Draft

test: cover checkpoint boundary variants#24723
AztecBot wants to merge 1 commit into
nextfrom
cb/rollup-audit-remediation-1628-checkpoint-boundary-tests

Conversation

@AztecBot

Copy link
Copy Markdown
Collaborator

Summary

The checkpoint/root variant tests missed several legal boundary cases around padding and single-child shapes. This adds committed coverage for the padding circuit output, root right-padding with a general checkpoint-root child, and all allowed first-block VKs in the checkpoint-root single-block path.

Fix

  • Assert the checkpoint padding rollup returns empty checkpoint public inputs.
  • Add root right-padding success coverage for a general one-checkpoint root child.
  • Expand checkpoint-root single-block success coverage across all allowed first-block VK indices.

Testing

  • PATH=/home/aztec-dev/aztec-packages/noir/noir-repo/target/debug:$PATH nargo test --package rollup_lib checkpoint_padding_is_empty with_left_general_root_and_right_padding with_single_block_root --silence-warnings

Issues

  • Closes AztecProtocol/aztec-claude#1628 — checkpoint padding and single-child variant tests miss legal boundary cases

Created by claudebox · group: rollup-audit-remediation

@AztecBot AztecBot added ci-draft Run CI on draft PRs. ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure claudebox Owned by claudebox. it can push to this PR. labels Jul 15, 2026
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. ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant