Skip to content

Comments

chore(test): Add e2e test workflow for op-reth sequencer and verifier#649

Closed
emhane wants to merge 3 commits intounstablefrom
emhane/op-reth-only-e2e
Closed

chore(test): Add e2e test workflow for op-reth sequencer and verifier#649
emhane wants to merge 3 commits intounstablefrom
emhane/op-reth-only-e2e

Conversation

@emhane
Copy link

@emhane emhane commented Jan 28, 2026

Adds a third set up for e2e tests, running op-reth in both sequencer and verifier positions

Copilot AI review requested due to automatic review settings January 28, 2026 16:43
@emhane emhane added K-test Kind: test W-historical-proofs Workstream: historical-proofs labels Jan 28, 2026
@wiz-b4c72f16a4
Copy link

wiz-b4c72f16a4 bot commented Jan 28, 2026

Wiz Scan Summary

Scanner Findings
Vulnerability Finding Vulnerabilities -
Data Finding Sensitive Data -
Secret Finding Secrets -
IaC Misconfiguration IaC Misconfigurations 1 Medium 3 Low
SAST Finding SAST Findings -
Software Management Finding Software Management Findings -
Total 1 Medium 3 Low

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a third e2e test configuration to the workflow that tests op-reth running in both the sequencer and verifier positions simultaneously, complementing the existing tests where op-reth runs only as verifier (with op-geth as sequencer) or only as sequencer (with op-geth as verifier).

Changes:

  • Added new op-reth-as-sequencer-and-verifier job that runs e2e tests with op-reth in both roles
  • Updated the success check job to include the new test configuration as a dependency

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

emhane and others added 2 commits January 28, 2026 17:46
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Jan 28, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.73%. Comparing base (8746d94) to head (ad25a5f).
⚠️ Report is 1 commits behind head on unstable.

Additional details and impacted files

Impacted file tree graph
see 1 file with indirect coverage changes

Flag Coverage Δ
e2e 57.20% <ø> (-0.02%) ⬇️
unit 85.61% <ø> (ø)

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

Components Coverage Δ
reth binary ∅ <ø> (∅)
op historical proof 89.75% <ø> (+0.01%) ⬆️
🚀 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.

@emhane
Copy link
Author

emhane commented Jan 29, 2026

superseded by #650

@emhane emhane closed this Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

K-test Kind: test W-historical-proofs Workstream: historical-proofs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant