Skip to content

[evm-tracing-0] add evm-tracing test suite to e2e ts tests#1574

Merged
dmitrylavrenov merged 3 commits intoevm-debug-tracefrom
evm-debug-e2e-test
Jul 28, 2025
Merged

[evm-tracing-0] add evm-tracing test suite to e2e ts tests#1574
dmitrylavrenov merged 3 commits intoevm-debug-tracefrom
evm-debug-e2e-test

Conversation

@dmitrylavrenov
Copy link
Contributor

@dmitrylavrenov dmitrylavrenov commented Jul 19, 2025 •
edited
Loading

image

@dmitrylavrenov dmitrylavrenov changed the base branch from master to evm-debug-trace July 19, 2025 11:50
@dmitrylavrenov dmitrylavrenov mentioned this pull request Jul 19, 2025
Merged
9 tasks
MOZGIII
MOZGIII reviewed Jul 19, 2025
View reviewed changes
MOZGIII
MOZGIII reviewed Jul 19, 2025
View reviewed changes
MOZGIII
MOZGIII reviewed Jul 19, 2025
View reviewed changes
@dmitrylavrenov
Copy link
Contributor Author

dmitrylavrenov commented Jul 21, 2025

@MOZGIII

I would like to finalize the code we would like to have as a part of TDD for EVM tracing logic. At the current moment it's unclear to me what exactly should we have.

I started preparing a proper test to execute RPC request for debug_traceCall and discovered that current moonbeam implementation doesn't meet debug trace call specification. For example, tracerConfig doesn't contain enableReturnData at moonebeam version, but the spec tracerConfig does.

I was thinking in terms of TDD that we should take the specification and implement some simple tests at least that follow the spec. But it can be different from moonbeam implementation, like request call data type. So, I guess we should define request/response types and test data at least.

Currently I started with a374702.

@dmitrylavrenov
Copy link
Contributor Author

dmitrylavrenov commented Jul 21, 2025

@MOZGIII

I would like to finalize the code we would like to have as a part of TDD for EVM tracing logic. At the current moment it's unclear to me what exactly should we have.

I started preparing a proper test to execute RPC request for debug_traceCall and discovered that current moonbeam implementation doesn't meet debug trace call specification. For example, tracerConfig doesn't contain enableReturnData at moonebeam version, but the spec tracerConfig does.

I was thinking in terms of TDD that we should take the specification and implement some simple tests at least that follow the spec. But it can be different from moonbeam implementation, like request call data type. So, I guess we should define request/response types and test data at least.

Currently I started with a374702.

Separated the PR, moved the tests itself logic to #1575

@dmitrylavrenov dmitrylavrenov marked this pull request as ready for review July 21, 2025 14:15
@dmitrylavrenov dmitrylavrenov changed the title [evm-tracing-0] add a corresponding e2e tests [evm-tracing-0] add evm-tracing test suite to e2e bash/ts tests Jul 21, 2025
@dmitrylavrenov dmitrylavrenov mentioned this pull request Jul 25, 2025
Merged
@dmitrylavrenov
Copy link
Contributor Author

dmitrylavrenov commented Jul 25, 2025

Will rebase after #1580

@dmitrylavrenov dmitrylavrenov changed the title [evm-tracing-0] add evm-tracing test suite to e2e bash/ts tests [evm-tracing-0] add evm-tracing test suite to e2e ts tests Jul 28, 2025
@dmitrylavrenov dmitrylavrenov requested a review from MOZGIII July 28, 2025 16:52
@dmitrylavrenov dmitrylavrenov merged commit ff3f91a into evm-debug-trace Jul 28, 2025
23 checks passed
@dmitrylavrenov dmitrylavrenov deleted the evm-debug-e2e-test branch July 28, 2025 18:03
dmitrylavrenov added a commit that referenced this pull request Aug 18, 2025
@dmitrylavrenov
[evm-tracing-0] add evm-tracing test suite to e2e ts tests (#1574)
3f3ff09 
* Add build for evm-tracing at e2e code job

* Skip bash tests for evm-tracing test suite

* Add template for evm-tracing test suite
dmitrylavrenov added a commit that referenced this pull request Sep 15, 2025
@dmitrylavrenov
[evm-tracing-0] add evm-tracing test suite to e2e ts tests (#1574)
898bbcd 
* Add build for evm-tracing at e2e code job

* Skip bash tests for evm-tracing test suite

* Add template for evm-tracing test suite
dmitrylavrenov added a commit that referenced this pull request Dec 3, 2025
@dmitrylavrenov
[evm-tracing-0] add evm-tracing test suite to e2e ts tests (#1574)
ce49888 
* Add build for evm-tracing at e2e code job

* Skip bash tests for evm-tracing test suite

* Add template for evm-tracing test suite
github-merge-queue bot pushed a commit that referenced this pull request Dec 4, 2025
@dmitrylavrenov
Support EVM tracing (#1560)
9205fb4 
* Add CI check for build with evm-tracing feature

* Add evm-tracing feature to humanode-runtime

* [evm-tracing-0] add evm-tracing test suite to e2e ts tests (#1574)

* Add build for evm-tracing at e2e code job

* Skip bash tests for evm-tracing test suite

* Add template for evm-tracing test suite

* [evm-tracing-0-1] add debug e2e tests logic implementation (#1575)

* [evm-tracing-1] runtime level implementation (#1544)

Runtime level implementation

* [evm-tracing-2] сlient-side implementation (#1565)

Client-side implementation

* [evm-tracing-3] implement RPC side (#1568)

Implement RPC side

* [evm-tracing-4] cli part implementation (#1572)

* [evm-tracing] Add more e2e tests (#1622)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@MOZGIII MOZGIII MOZGIII approved these changes

@quasiyoke quasiyoke Awaiting requested review from quasiyoke

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@dmitrylavrenov @MOZGIII