Skip to content

feat: pos-portal bridge/withdraws and fix validator tests#259

Closed
leovct wants to merge 14 commits intomainfrom
fix/typos
Closed

feat: pos-portal bridge/withdraws and fix validator tests#259
leovct wants to merge 14 commits intomainfrom
fix/typos

Conversation

@leovct
Copy link
Copy Markdown
Collaborator

@leovct leovct commented Apr 24, 2026

leovct and others added 12 commits April 3, 2026 09:22
…ing L1 event processing

Move "update signer" after delegation and removal-prerequisite tests so the
Heimdall node's key still matches the registered validator signer while staking
events (StakeUpdate, UnstakeInit) need to be picked up from L1.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…tion

Move "remove validator" before "update signer" so the validator's new key
only takes effect after all staking events (StakeUpdate, UnstakeInit) have
been picked up by Heimdall. With the old order, updateSigner at position 7
severed L1 event delivery before "remove validator" could complete.

Add --gas-limit 200000 to all four Plasma bridge withdraw cast send calls
to bypass Bor's pending-state gas estimation bug: in a multi-validator
devnet, eth_estimateGas with block=pending returns null balance, causing
ErrInsufficientFundsForTransfer even when the account has sufficient funds.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Adds tests/pos/bridge/pos.bats covering the pos-portal (RootChainManager /
ChildChainManager) bridge: ETH, ERC20, ERC721, ERC1155 deposit + withdraw.
Moves the existing plasma suite to tests/pos/bridge/plasma.bats so both
bridges live side by side, with a README.md explaining the split
(POL/MATIC/native -> plasma, everything else -> pos-portal).

Extracts the state-sync / checkpoint / exit-proof helpers shared by both
suites into core/helpers/scripts/pos-bridge.bash, and teaches pos-setup.bash
to resolve pos-portal addresses from the matic-contract-addresses-with-pos-portal
artifact when the devnet deploys it.

Inventory + README test_tags regenerated via scripts/update-tests-inventory.sh.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@leovct leovct marked this pull request as ready for review April 27, 2026 12:47
@leovct leovct requested review from a team and jhkimqd and removed request for a team April 27, 2026 12:47
Signed-off-by: Ji Hwan <jkim@polygon.technology>
Signed-off-by: Ji Hwan <jkim@polygon.technology>
@jhkimqd
Copy link
Copy Markdown
Contributor

jhkimqd commented Apr 27, 2026

Closing in favor of https://github.com/0xPolygon/pos-e2e/pull/1

@jhkimqd jhkimqd closed this Apr 27, 2026
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.

2 participants