Skip to content

op-e2e: Add the initial set of Super DG e2e tests#15036

Merged
Inphi merged 6 commits intodevelopfrom
inphi/opcm-e2e-fp
Apr 8, 2025
Merged

op-e2e: Add the initial set of Super DG e2e tests#15036
Inphi merged 6 commits intodevelopfrom
inphi/opcm-e2e-fp

Conversation

@Inphi
Copy link
Contributor

@Inphi Inphi commented Mar 25, 2025

Setup SuperSystem for e2e testing of the op-challenger for SuperFaultDisputeGames. This is done to ensure the system works as expected.

The output root e2e test cases will be migrated for the Super DG in a later PR. Observe the refactor of TestOutputCannonGame for a preview of how the migration will be done.

part of #14974

@codecov
Copy link

codecov bot commented Mar 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.07%. Comparing base (b1a4134) to head (e716fc8).
Report is 34 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #15036       +/-   ##
============================================
+ Coverage    46.37%   66.07%   +19.69%     
============================================
  Files         1216       63     -1153     
  Lines       102451     4781    -97670     
============================================
- Hits         47515     3159    -44356     
+ Misses       51580     1452    -50128     
+ Partials      3356      170     -3186     
Flag Coverage Δ
cannon-go-tests-32 62.08% <ø> (-2.00%) ⬇️
cannon-go-tests-64 57.13% <ø> (-1.64%) ⬇️
contracts-bedrock-tests ?

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

see 1164 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.

@Inphi Inphi force-pushed the inphi/opcm-e2e-fp branch from efc7f1f to 84c8d1e Compare March 26, 2025 20:14
Base automatically changed from inphi/opcm-e2e to develop March 27, 2025 14:09
@Inphi Inphi force-pushed the inphi/opcm-e2e-fp branch from 84c8d1e to eb393e5 Compare April 3, 2025 20:27
@Inphi Inphi requested a review from ajsutton April 3, 2025 21:01
@Inphi Inphi marked this pull request as ready for review April 3, 2025 21:01
@Inphi Inphi requested review from a team as code owners April 3, 2025 21:01
@Inphi Inphi requested a review from protolambda April 3, 2025 21:01
Copy link
Contributor

@ajsutton ajsutton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Game Arena abstraction is a good idea. Sounds very gladiatorial too. :)

@Inphi Inphi added this pull request to the merge queue Apr 8, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 8, 2025
@Inphi Inphi added this pull request to the merge queue Apr 8, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 8, 2025
@Inphi Inphi added this pull request to the merge queue Apr 8, 2025
Merged via the queue into develop with commit c8a051f Apr 8, 2025
55 checks passed
@Inphi Inphi deleted the inphi/opcm-e2e-fp branch April 8, 2025 17:05
iquidus pushed a commit to Layr-Labs/optimism that referenced this pull request Jul 24, 2025
…15036)

* op-e2e: Add the initial set of Super DG e2e tests

* dedup TestCannonGame setup

* op-e2e: Migrate tests on game mechanics to the Super DG

* ensure correct super root is safe

* Fix testDisputeRootBeyondProposedBlockValidOutputRoot
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

Comments