Skip to content

test(state): account & storage writes#4339

Merged
rakita merged 10 commits intorakita/revm_statefrom
rkrasiuk/test-state-account-writes
Aug 28, 2023
Merged

test(state): account & storage writes#4339
rakita merged 10 commits intorakita/revm_statefrom
rkrasiuk/test-state-account-writes

Conversation

@rkrasiuk
Copy link
Contributor

@rkrasiuk rkrasiuk commented Aug 24, 2023

Description

Port account & storage write test to the new state API.

@rkrasiuk rkrasiuk added A-execution Related to the Execution and EVM C-test A change that impacts how or what we test labels Aug 24, 2023
@codecov
Copy link

codecov bot commented Aug 24, 2023

Codecov Report

Merging #4339 (c9ba804) into rakita/revm_state (759eaf5) will decrease coverage by 51.46%.
Report is 84 commits behind head on rakita/revm_state.
The diff coverage is 5.54%.

❗ Current head c9ba804 differs from pull request most recent head 0c06c41. Consider uploading reports for the commit 0c06c41 to get more accurate results

Impacted file tree graph

Files Changed Coverage Δ
bin/reth/src/args/pruning_args.rs 0.00% <0.00%> (-10.00%) ⬇️
bin/reth/src/args/rpc_server_args.rs 0.00% <0.00%> (-53.85%) ⬇️
bin/reth/src/chain/import.rs 0.00% <0.00%> (-6.96%) ⬇️
bin/reth/src/cli/ext.rs 0.00% <0.00%> (-1.79%) ⬇️
bin/reth/src/cli/mod.rs 0.00% <0.00%> (-34.05%) ⬇️
bin/reth/src/db/list.rs 0.00% <0.00%> (-2.57%) ⬇️
bin/reth/src/db/mod.rs 0.00% <0.00%> (-7.28%) ⬇️
bin/reth/src/db/tui.rs 0.00% <0.00%> (ø)
bin/reth/src/debug_cmd/execution.rs 0.00% <0.00%> (-1.16%) ⬇️
bin/reth/src/debug_cmd/in_memory_merkle.rs 0.00% <0.00%> (-1.30%) ⬇️
... and 138 more

... and 263 files with indirect coverage changes

Flag Coverage Δ
integration-tests 17.10% <5.54%> (+0.61%) ⬆️
unit-tests ?

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

Components Coverage Δ
reth binary 6.79% <22.85%> (-18.58%) ⬇️
blockchain tree 0.00% <0.00%> (-83.05%) ⬇️
pipeline 0.00% <0.00%> (-90.08%) ⬇️
storage (db) 19.08% <0.00%> (-55.52%) ⬇️
trie 0.00% <ø> (-94.72%) ⬇️
txpool 23.53% <6.45%> (-24.66%) ⬇️
networking 27.02% <8.60%> (-50.43%) ⬇️
rpc 25.94% <6.47%> (-31.54%) ⬇️
consensus 0.97% <0.00%> (-62.79%) ⬇️
revm 1.58% <1.23%> (-30.69%) ⬇️
payload builder 6.90% <0.00%> (+0.33%) ⬆️
primitives 22.17% <10.48%> (-65.62%) ⬇️

Copy link
Collaborator

@onbjerg onbjerg left a comment

Choose a reason for hiding this comment

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

smol nit

@rkrasiuk rkrasiuk changed the title test(state): account writes test(state): account & storage writes Aug 24, 2023
@rakita rakita merged commit c0d064f into rakita/revm_state Aug 28, 2023
@rakita rakita deleted the rkrasiuk/test-state-account-writes branch August 28, 2023 10:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-execution Related to the Execution and EVM C-test A change that impacts how or what we test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants