Skip to content

Conversation

@pahor167
Copy link

No description provided.

pahor167 and others added 30 commits January 23, 2025 11:25
* working deploy2.s.sol from celo10

* Deployment refactor

* Compilation fix

* removal of deploy_orig
* working deploy2.s.sol from celo10

* Deployment refactor

* Contract Deployment: Split FP initialization

* Minnor format fixes

---------

Co-authored-by: pahor167 <[email protected]>
* working deploy2.s.sol from celo10

* Deployment refactor

* Compilation fix

* removal of deploy_orig

* Setup new contract for Celo-specific guardian role

* Test the actual new contract

* Add getter for global SuperchainConfig

* Make sure global guardian can't directly pause Celo config

* Rename test contracts

* Return combined Superchain and Celo paused status

* Allow pausing of Celo config when Superchain is paused

* Ensure Celo stays paused if Superchain still paused

* Add CeloSuperchainConfig to Specs test

* Exclude CeloSuperchainConfig from Initializable test for now

* Make test view

* Use CeloSuperchainConfig interface

* Check and propagate superchain paused status in one function

* Setup CeloSuperchainConfig in deploy script

* Point OptimismPortal to CeloSuperchainConfig

* Formatting

* Use CeloSuperchainConfig in contracts that used SuperchainConfig

* Fix CrossDomanMessenger test

* Add NatSpecs

* Add unit test NatSpecs

* Add reference comment to deploy script

* Add comments

* Emit event on config update

* Initialize as paused if Superchain paused at init time

* Test Celo and Superchain pauses on OptimismPortal2

* Test Celo and Superchain pauses on OptimismPortal

* Test Celo and Superchain pauses on L1CrossDomainMessenger

* Test Celo and Superchain pauses on L1ERC721Bridge

* Use checkAndPause in L1ERC721Bridge

* Add enum and event to interface

* Use CeloSuperChainConfig in ChainAssertions

* Remove unused script

* Add CeloSuperchainConfig chain assertion

---------

Co-authored-by: pahor167 <[email protected]>
* Check CeloSuperchainConfig in Initializable test

* Ignore Celo contracts from Initializable test
@codecov-commenter
Copy link

codecov-commenter commented Mar 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.65%. Comparing base (d8c2369) to head (97d131d).

Additional details and impacted files
@@                   Coverage Diff                    @@
##           pahor/genesisGeneration     #371   +/-   ##
========================================================
  Coverage                    66.65%   66.65%           
========================================================
  Files                           56       56           
  Lines                         4666     4666           
========================================================
  Hits                          3110     3110           
  Misses                        1384     1384           
  Partials                       172      172           
Flag Coverage Δ
cannon-go-tests-32 61.88% <ø> (ø)
cannon-go-tests-64 54.16% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Base automatically changed from pahor/genesisGeneration to celo-contracts/v1.8.0-1 May 19, 2025 15:22
ezdac pushed a commit that referenced this pull request Jun 16, 2025
* feat: add mul access lists test (#370)

* feat: add mul access lists test

* chore: polish

* chore: pre pr

* feat: add more tests

* feat: wip to add more tests

* fix: multiple msgs tests

* fix: semgrep (#371)

---------

Co-authored-by: Disco <[email protected]>
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.

8 participants