Skip to content

Conversation

@rodiazet
Copy link
Member

@rodiazet rodiazet commented Aug 9, 2023

No description provided.

@rodiazet rodiazet requested review from chfast and gumb0 August 9, 2023 14:05
@codecov
Copy link

codecov bot commented Aug 9, 2023

Codecov Report

Merging #683 (35356f1) into master (095f83a) will decrease coverage by 0.01%.
The diff coverage is 97.56%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #683      +/-   ##
==========================================
- Coverage   97.48%   97.48%   -0.01%     
==========================================
  Files          85       85              
  Lines        8177     8215      +38     
==========================================
+ Hits         7971     8008      +37     
- Misses        206      207       +1     
Flag Coverage Δ
blockchaintests 62.71% <ø> (ø)
statetests 73.93% <38.46%> (-0.23%) ⬇️
statetests-silkpre 23.12% <12.82%> (-0.08%) ⬇️
unittests 95.01% <97.56%> (+0.01%) ⬆️

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

Files Changed Coverage Δ
test/state/state.hpp 100.00% <ø> (ø)
test/unittests/state_transition.hpp 0.00% <ø> (ø)
test/unittests/state_transition.cpp 94.59% <50.00%> (-2.63%) ⬇️
test/state/state.cpp 98.57% <100.00%> (+0.07%) ⬆️
test/statetest/statetest_loader.cpp 90.33% <100.00%> (+0.20%) ⬆️
test/statetest/statetest_runner.cpp 100.00% <100.00%> (ø)
test/unittests/state_transition_block_test.cpp 100.00% <100.00%> (ø)
...est/unittests/statetest_loader_block_info_test.cpp 100.00% <100.00%> (ø)

@rodiazet rodiazet changed the title Blockchain tests fixing: Add ommer block rewarding Blockchain tests support: Add ommer block rewarding Aug 17, 2023
@rodiazet
Copy link
Member Author

Moved to #682

@rodiazet rodiazet closed this Aug 21, 2023
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.

2 participants