Skip to content

Conversation

@VGau
Copy link
Contributor

@VGau VGau commented Nov 20, 2025

This PR implements issue(s) #

Checklist

  • I wrote new tests for my new core changes.
  • I have successfully ran tests, style checker and build against my new changes locally.
  • I have informed the team of any breaking changes if there are any.

Note

Adds e2e tests for OpcodeTester and introduces L2 opcodeTesterAddress with accessor in setup/config.

  • Tests:
    • Add e2e/src/opcodes.spec.ts with:
      • Gas estimation via linea_estimateGas for executeAllOpcodes.
      • Execution test invoking executeAllOpcodes and asserting state change.
  • Config:
    • Extend L2 config types to include opcodeTesterAddress in e2e/src/config/tests-config/types.ts.
    • Set opcodeTesterAddress in local env e2e/src/config/tests-config/environments/local.ts.
    • Add getL2OpcodeTesterContract accessor and include opcodeTesterAddress in isLocalL2Config guard in e2e/src/config/tests-config/setup.ts.

Written by Cursor Bugbot for commit a10a0a3. This will update automatically on new commits. Configure here.

@codecov-commenter
Copy link

codecov-commenter commented Nov 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.49%. Comparing base (1aaf88c) to head (a10a0a3).

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #1760   +/-   ##
=========================================
  Coverage     64.48%   64.49%           
- Complexity     1522     1523    +1     
=========================================
  Files           396      396           
  Lines         14381    14381           
  Branches       1545     1545           
=========================================
+ Hits           9274     9275    +1     
  Misses         4497     4497           
+ Partials        610      609    -1     
Flag Coverage Δ *Carryforward flag
hardhat 96.20% <ø> (ø) Carriedforward from 1aaf88c
kotlin 62.05% <ø> (+<0.01%) ⬆️

*This pull request uses carry forward flags. Click here to find out more.
see 1 file 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.

@VGau VGau merged commit 72ee65e into main Nov 21, 2025
52 of 53 checks passed
@VGau VGau deleted the feat/add-opcodetester-contract-to-e2e-tests branch November 21, 2025 16:47
jonesho pushed a commit that referenced this pull request Nov 24, 2025
* parse new responses format

* add class name

* add class name
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.

4 participants