Skip to content

test(l1): add full-sync smoke test #4658

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 3 commits into
base: add-storage-slot-reorg-test
Choose a base branch
from
Draft

test(l1): add full-sync smoke test #4658

wants to merge 3 commits into from
+24 −0

Conversation

MegaRedHand
Copy link
Collaborator

@MegaRedHand MegaRedHand commented Sep 25, 2025 •
edited
Loading

Blocked by #4652

Motivation

Full-sync appears not to be working.

Description

This PR adds a test that is currently failing, related to full-sync. No longer fails since 9d63811. The error was due to a missing FCU.

@github-actions github-actions bot added the L1 Ethereum client label Sep 25, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 25, 2025 •
edited
Loading

Lines of code report

Total lines added: 16
Total lines removed: 0
Total lines changed: 16

Detailed view 
+----------------------------------------+-------+------+
| File                                   | Lines | Diff |
+----------------------------------------+-------+------+
| ethrex/tooling/reorgs/src/main.rs      | 226   | +8   |
+----------------------------------------+-------+------+
| ethrex/tooling/reorgs/src/simulator.rs | 448   | +8   |
+----------------------------------------+-------+------+

@MegaRedHand MegaRedHand marked this pull request as ready for review September 26, 2025 19:03
@MegaRedHand MegaRedHand requested a review from a team as a code owner September 26, 2025 19:03
@ethrex-project-sync ethrex-project-sync bot moved this to In Review in ethrex_l1 Sep 26, 2025
@gianbelinche gianbelinche mentioned this pull request Sep 26, 2025
Draft
@MegaRedHand MegaRedHand moved this from In Review to Blocked in ethrex_l1 Sep 26, 2025
@MegaRedHand MegaRedHand marked this pull request as draft September 26, 2025 21:15
@ethrex-project-sync ethrex-project-sync bot moved this from Blocked to In Progress in ethrex_l1 Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gianbelinche gianbelinche gianbelinche approved these changes

Assignees

@MegaRedHand MegaRedHand

Labels
L1 Ethereum client
Projects
ethrex_l1
Status: In Progress
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MegaRedHand @gianbelinche