Skip to content

feat: refactor test utils#2813

Merged
rakita merged 8 commits intomainfrom
rakita/test-utils
Jul 29, 2025
Merged

feat: refactor test utils#2813
rakita merged 8 commits intomainfrom
rakita/test-utils

Conversation

@rakita
Copy link
Copy Markdown
Member

@rakita rakita commented Jul 28, 2025

Moved ee tests to separate crate. They are now easier to test as serde is always enabled. llvm-cov would not correctly consume it and report cov.

Added sorted output of json, this will make it easier to spot changes in state (that is hashmap) as accounts and storages are ordered

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Jul 28, 2025

CodSpeed Performance Report

Merging #2813 will not alter performance

Comparing rakita/test-utils (4182316) with main (9008e93)

Summary

✅ 171 untouched benchmarks

@rakita rakita marked this pull request as ready for review July 28, 2025 18:43
@rakita rakita mentioned this pull request Jul 29, 2025
@rakita rakita merged commit f2d73c4 into main Jul 29, 2025
29 checks passed
This was referenced Jul 29, 2025
@rakita rakita deleted the rakita/test-utils branch July 30, 2025 19:29
This was referenced Sep 8, 2025
lwedge99 pushed a commit to sentioxyz/revm that referenced this pull request Sep 16, 2025
* feat: refactor test utils

* fix compilation

* feat: add ee-test to unify ee tests

* sort all json test fixtures

* fmt

* typo
This was referenced Oct 15, 2025
@github-actions github-actions bot mentioned this pull request Oct 30, 2025
This was referenced Nov 12, 2025
This was referenced Dec 23, 2025
@github-actions github-actions bot mentioned this pull request Jan 15, 2026
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 21, 2026
* feat: refactor test utils

* fix compilation

* feat: add ee-test to unify ee tests

* sort all json test fixtures

* fmt

* typo
@github-actions github-actions bot mentioned this pull request Mar 3, 2026
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.

1 participant