Skip to content

Contract tx Text & Single Consensus Client Approach#52

Merged
svlachakis merged 5 commits intonethermindfrom
537
Jan 12, 2026
Merged

Contract tx Text & Single Consensus Client Approach#52
svlachakis merged 5 commits intonethermindfrom
537

Conversation

@svlachakis
Copy link

@svlachakis svlachakis commented Jan 7, 2026

Fixes Closes NethermindEth/nethermind-arbitrum#537

The single consensus node approach with WithExecutionClientMode(ExecutionClientModeComparison) works because both geth and Nethermind execution clients start from a common genesis state with the pre-funded account and receive identical block inputs from the same consensus node, ensuring they execute the exact same transactions. The two-node replica approach fails because geth which has already executed transactions like TransferBalance is causing state divergence and different block hashes.

This approach cannot be unified for all tests, but is necessary for this one or other similar tests

This approach Flow: L1 Message → Consensus Layer → Both Execution Clients

  • Added Contract Tx Deploy system test support
  • Added also methods to reduce boilerplate for replica approach

@svlachakis svlachakis changed the title Contract tx Text & Single Node Approach Contract tx Text & Execution Client Approach Jan 7, 2026
@svlachakis svlachakis changed the title Contract tx Text & Execution Client Approach Contract tx Text & Single Consensus Client Approach Jan 7, 2026
@svlachakis svlachakis marked this pull request as ready for review January 9, 2026 14:06
Copy link
Collaborator

@wurdum wurdum left a comment

Choose a reason for hiding this comment

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

I thought we've deprecated these env vars in favor of cmd args.
Also, makes sense to consider the moment when we migrate to Nitro's RPC implementation that also relies on cmd args.

@svlachakis
Copy link
Author

@wurdum yeah they are wrong I'll fix them

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