Skip to content

Feat/property testing#18

Closed
0xteddybear wants to merge 18 commits intodevelopfrom
feat/property-testing
Closed

Feat/property testing#18
0xteddybear wants to merge 18 commits intodevelopfrom
feat/property-testing

Conversation

@0xteddybear
Copy link
Copy Markdown

this is a feature branch for property testing the suprechainERC20 contracts, consists of a merge of sc/liquidity-migration and feat/add-superchain-erc20-factory

until those branches are merged to develop, all property testing PRs will be made to this branch.

@0xteddybear 0xteddybear mentioned this pull request Aug 8, 2024
@0xteddybear 0xteddybear force-pushed the feat/property-testing branch from d469a49 to 5514694 Compare August 13, 2024 15:35
0xOneTony pushed a commit that referenced this pull request Mar 6, 2026
…thereum-optimism#19272)

* contracts: implement audit code fixes and add tests

Add onlyDelegateCall enforcement to upgradeSuperchain, upgrade, and
migrate functions (#17). Include msg.sender in deploy salt to prevent
cross-caller CREATE2 collisions (#17). Add duplicate instruction key
detection in upgrade validation (#9). Validate startingRespectedGameType
against enabled game configs (#10). Add code-existence check in
loadBytes (#18). Add setUp guard to VerifyOPCM.runSingle (#4). Remove
unused _findChar function (#5). Pass real AddressManager in migrator
proxy deploy args (#11). Add tests covering all audit fix behaviors.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* contracts: regenerate semver-lock.json for OPContractsManagerV2

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* contracts: bump OPContractsManagerV2 version to 7.0.10

Semver-diff requires a patch version bump when bytecode changes.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
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.

3 participants