Skip to content

[DO NOT MERGE] [MEL] Update rollup, challenge and osp tests#435

Draft
TucksonDev wants to merge 5 commits into
feat/mel-osp-changesfrom
feat/mel-osp-testing
Draft

[DO NOT MERGE] [MEL] Update rollup, challenge and osp tests#435
TucksonDev wants to merge 5 commits into
feat/mel-osp-changesfrom
feat/mel-osp-testing

Conversation

@TucksonDev

@TucksonDev TucksonDev commented Jun 11, 2026

Copy link
Copy Markdown
Contributor

!! WIP - DO NOT MERGE !!

Note

This is a preliminary version, and can completely change, based on changes in the design or the implementation of the feature in nitro.

This PR builds on top of #432 and updates the tests of the Rollup, Challenge and OSP contracts. These changes are added to a different PR to allow for the base PR to only include relevant changes to the contracts, and not increase its size too much.

Key changes

  • Changes related to Assertions handling and the GlobalState can be read in [DO NOT MERGE] [MEL] Changes to Rollup contracts #427
  • Changes related to OSP contracts can be read in [DO NOT MERGE] [MEL] Changes to OSP contracts #432
  • Challenge foundry tests have been moved from test/challengeV2 to test/foundry/challengeV2 so they are run on CI
  • New tests added for the OneStepProverHostIo (new opcode GetEndParentChainBlockHash, new paths in ReadPreimage, and deprecated ReadInbox opcodes)
  • New tests added for the HashProofHelper

Pending work

  • Add tests for the changes in OneStepProverHostIo
  • Add tests for the HashProofHelper

@TucksonDev TucksonDev changed the title Update challenge and osp tests [DO NOT MERGE] [MEL] Update challenge and osp tests Jun 11, 2026
@TucksonDev TucksonDev changed the title [DO NOT MERGE] [MEL] Update challenge and osp tests [DO NOT MERGE] [MEL] Update rollup, challenge and osp tests Jun 11, 2026
@TucksonDev TucksonDev marked this pull request as draft June 16, 2026 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant