Skip to content

Conversation

@maurelian
Copy link
Contributor

@maurelian maurelian commented Jan 6, 2026

Just to see if the failure in #18704 is due to an external contributor issue.

* fix: remove extraInstructions in upgradeSuperchain test

* docs: add cgt extra instruction TODO

* test: add fuzzing to UpgradeOPChain test

* feat: add revert cases for setters and tests

* test: add input encoding verification

* fix: add v2 suffix in upgrade test

* docs: added natspec to upgradeOPChain tests
@codecov
Copy link

codecov bot commented Jan 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.54%. Comparing base (903584f) to head (cbd0cb2).
⚠️ Report is 7 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #18719      +/-   ##
===========================================
- Coverage    73.83%   73.54%   -0.30%     
===========================================
  Files          189      189              
  Lines        11244    11244              
===========================================
- Hits          8302     8269      -33     
- Misses        2796     2831      +35     
+ Partials       146      144       -2     
Flag Coverage Δ
cannon-go-tests-64 66.58% <ø> (-0.82%) ⬇️
contracts-bedrock-tests 77.42% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 5 files with indirect coverage changes

🚀 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.

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