Skip to content

fix 2.1.3 verification script#68

Open
godzillaba wants to merge 1 commit intomainfrom
ha/pr-65-review
Open

fix 2.1.3 verification script#68
godzillaba wants to merge 1 commit intomainfrom
ha/pr-65-review

Conversation

@godzillaba
Copy link
Copy Markdown
Contributor

quick test to demonstrate the script works by running before and after 2.1.3 on arb1

> INBOX_ADDRESS=$ARB1_INBOX forge script scripts/foundry/contract-upgrades/2.1.3/VerifyNitroContracts2Point1Point3Upgrade.s.sol --fork-url $ETH_URL
[⠊] Compiling...
No files changed, compilation skipped
Script ran successfully.

> INBOX_ADDRESS=$ARB1_INBOX forge script scripts/foundry/contract-upgrades/2.1.3/VerifyNitroContracts2Point1Point3Upgrade.s.sol --fork-url $ETH_URL --fork-block-number 20721817
[⠊] Compiling...
No files changed, compilation skipped
Traces:
  [317061] → new VerifyNitroContracts2Point1Point3Upgrade@0x9f7cF1d1F558E57ef88a59ac3D47214eF25B6A06
    └─ ← [Return] 1473 bytes of code

  [43046] VerifyNitroContracts2Point1Point3Upgrade::run()
    ├─ [0] VM::envAddress("INBOX_ADDRESS") [staticcall]
    │   └─ ← [Return] <env var value>
    ├─ [9711] 0x4Dbd4fc535Ac27206064B68FfCf827b0A60BAB3f::bridge() [staticcall]
    │   ├─ [2471] Inbox::bridge() [delegatecall]
    │   │   └─ ← [Return] 0x8315177aB297bA92A06054cE80a67Ed4DBd7ed3a
    │   └─ ← [Return] 0x8315177aB297bA92A06054cE80a67Ed4DBd7ed3a
    ├─ [9787] 0x8315177aB297bA92A06054cE80a67Ed4DBd7ed3a::sequencerInbox() [staticcall]
    │   ├─ [2471] Bridge::sequencerInbox() [delegatecall]
    │   │   └─ ← [Return] TransparentUpgradeableProxy: [0x1c479675ad559DC151F6Ec7ed3FbF8ceE79582B6]
    │   └─ ← [Return] TransparentUpgradeableProxy: [0x1c479675ad559DC151F6Ec7ed3FbF8ceE79582B6]
    ├─ [0] VM::etch(0x0000000000000000000000000000000000001234, 0x4321)
    │   └─ ← [Return]
    ├─ [0] VM::prank(0x0000000000000000000000000000000000001234, 0x0000000000000000000000000000000000001234)
    │   └─ ← [Return]
    ├─ [0] VM::expectRevert(custom error 0xf28dceb3: 00000000000000000000000000000000000000000000000000000000000000200000000000000000000000000000000000000000000000000000000000000004c8958ead00000000000000000000000000000000000000000000000000000000)
    │   └─ ← [Return]
    ├─ [10387] TransparentUpgradeableProxy::fallback(0, 0x, 0, 0x0000000000000000000000000000000000000000, 0, 0)
    │   ├─ [3034] SequencerInbox::addSequencerL2BatchFromOrigin(0, 0x, 0, 0x0000000000000000000000000000000000000000, 0, 0) [delegatecall]
    │   │   └─ ← [Revert] NotBatchPoster()
    │   └─ ← [Revert] NotBatchPoster()
    └─ ← [Revert] Error != expected error: custom error 0x2dd9fc97 != custom error 0xc8958ead


Error: script failed: Error != expected error: custom error 0x2dd9fc97 != custom error 0xc8958ead

@godzillaba godzillaba requested a review from yahgwai March 23, 2026 17:57
Base automatically changed from feat/380-dockerize to main March 24, 2026 16:33
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