Skip to content

test(poststate): storage change after selfdestruct#4198

Closed
rkrasiuk wants to merge 1 commit intomainfrom
rkrasiuk/test-change-after-selfdestruct
Closed

test(poststate): storage change after selfdestruct#4198
rkrasiuk wants to merge 1 commit intomainfrom
rkrasiuk/test-change-after-selfdestruct

Conversation

@rkrasiuk
Copy link
Contributor

Description

Add a test that showcases a bug of changed storage after selfdestruct.

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

codecov bot commented Aug 14, 2023

Codecov Report

Merging #4198 (b489fc5) into main (b2be35c) will decrease coverage by 52.20%.
Report is 1 commits behind head on main.
The diff coverage is 0.00%.

Impacted file tree graph

Files Changed Coverage Δ
crates/storage/provider/src/post_state/mod.rs 0.00% <0.00%> (-95.76%) ⬇️

... and 374 files with indirect coverage changes

Flag Coverage Δ
integration-tests 16.88% <0.00%> (-0.01%) ⬇️
unit-tests ?

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

Components Coverage Δ
reth binary 7.00% <ø> (-19.60%) ⬇️
blockchain tree 0.00% <ø> (-82.57%) ⬇️
pipeline 0.00% <ø> (-90.08%) ⬇️
storage (db) 17.63% <0.00%> (-57.21%) ⬇️
trie 0.00% <ø> (-94.72%) ⬇️
txpool 23.68% <ø> (-27.76%) ⬇️
networking 27.07% <ø> (-50.85%) ⬇️
rpc 25.98% <ø> (-32.69%) ⬇️
consensus 0.98% <ø> (-62.75%) ⬇️
revm 1.38% <ø> (-30.82%) ⬇️
payload builder 6.82% <ø> (ø)
primitives 22.42% <ø> (-65.45%) ⬇️

@rkrasiuk
Copy link
Contributor Author

closing in favor of #4199

@rkrasiuk rkrasiuk closed this Aug 14, 2023
@rkrasiuk rkrasiuk deleted the rkrasiuk/test-change-after-selfdestruct branch August 14, 2023 18:30
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.

1 participant