Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

EOFTests #678

Merged
merged 3 commits into from
Sep 26, 2023
Merged

EOFTests #678

merged 3 commits into from
Sep 26, 2023

Conversation

gzanitti
Copy link
Contributor

@gzanitti gzanitti commented Aug 8, 2023

This PR adds the necessary code to run the EOF tests.

Note: The value of the exception field is not validated, following what we discussed in our first meeting.

@gzanitti gzanitti requested a review from gumb0 August 10, 2023 08:58
@gumb0
Copy link
Member

gumb0 commented Aug 10, 2023

Please squash the commits

test/CMakeLists.txt Outdated Show resolved Hide resolved
@gumb0
Copy link
Member

gumb0 commented Aug 11, 2023

Please squash the commits

circle.yml Show resolved Hide resolved
@gzanitti gzanitti force-pushed the EOFTests branch 2 times, most recently from e5c55e4 to 7c121f5 Compare August 11, 2023 23:08
test/eoftests/eoftests.hpp Outdated Show resolved Hide resolved
test/eoftests/eoftests_runner.cpp Outdated Show resolved Hide resolved
test/eoftests/eoftests.hpp Outdated Show resolved Hide resolved
Copy link
Member

@gumb0 gumb0 left a comment

Choose a reason for hiding this comment

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

Some minor nitpicks left.

test/eoftest/eoftest.hpp Outdated Show resolved Hide resolved
test/eoftest/eoftest_runner.cpp Outdated Show resolved Hide resolved
test/eoftest/eoftest_runner.cpp Outdated Show resolved Hide resolved
test/eoftest/eoftest_runner.cpp Outdated Show resolved Hide resolved
test/eoftest/eoftest_runner.cpp Outdated Show resolved Hide resolved
test/eoftest/eoftest_runner.cpp Outdated Show resolved Hide resolved
@gzanitti
Copy link
Contributor Author

@gumb0 some of the tests fail because of the emplace_back (probably a version problem), should I roll back the change?

@gumb0
Copy link
Member

gumb0 commented Aug 17, 2023

Looks good to me now, please squash once again

@gumb0 gumb0 requested review from axic, chfast and rodiazet August 17, 2023 14:58
@gzanitti gzanitti marked this pull request as ready for review August 18, 2023 14:28
@gumb0
Copy link
Member

gumb0 commented Aug 19, 2023

I rebased and changed commit message.

Copy link
Member

@gumb0 gumb0 left a comment

Choose a reason for hiding this comment

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

One failing test should be fixed when #619 is merged.

test/eoftest/eoftest.cpp Outdated Show resolved Hide resolved
test/eoftest/eoftest.hpp Outdated Show resolved Hide resolved
test/utils/utils.hpp Outdated Show resolved Hide resolved
test/eoftest/eoftest_runner.cpp Outdated Show resolved Hide resolved
test/eoftest/eoftest_runner.cpp Outdated Show resolved Hide resolved
test/eoftest/eoftest_runner.cpp Outdated Show resolved Hide resolved
test/eoftest/eoftest_runner.cpp Outdated Show resolved Hide resolved
test/eoftest/eoftest_runner.cpp Outdated Show resolved Hide resolved
test/eoftest/eoftest_runner.cpp Outdated Show resolved Hide resolved
test/utils/utils.cpp Outdated Show resolved Hide resolved
test/eoftest/eoftest.hpp Outdated Show resolved Hide resolved
@gumb0 gumb0 force-pushed the EOFTests branch 3 times, most recently from 0178637 to 00ef19c Compare August 25, 2023 12:54
@codecov
Copy link

codecov bot commented Aug 25, 2023

Codecov Report

Merging #678 (13fc171) into master (d50af72) will decrease coverage by 0.02%.
The diff coverage is 59.37%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #678      +/-   ##
==========================================
- Coverage   97.60%   97.59%   -0.02%     
==========================================
  Files          94       95       +1     
  Lines        8644     8644              
==========================================
- Hits         8437     8436       -1     
- Misses        207      208       +1     
Flag Coverage Δ
blockchaintests 62.37% <ø> (ø)
statetests 73.43% <ø> (-0.08%) ⬇️
statetests-silkpre 22.61% <83.33%> (ø)
unittests 95.58% <59.37%> (ø)

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

Files Coverage Δ
test/statetest/statetest.hpp 100.00% <ø> (ø)
test/statetest/statetest_loader.cpp 96.03% <ø> (+4.52%) ⬆️
test/utils/utils.hpp 100.00% <ø> (ø)
test/utils/utils.cpp 59.37% <59.37%> (ø)

... and 1 file with indirect coverage changes

@gumb0 gumb0 force-pushed the EOFTests branch 5 times, most recently from 172be76 to 74196ff Compare August 28, 2023 09:41
@gumb0 gumb0 requested a review from chfast August 28, 2023 10:13
gzanitti and others added 3 commits September 26, 2023 16:05
Co-authored-by: Andrei Maiboroda <andrei@ethereum.org>
Co-authored-by: Andrei Maiboroda <andrei@ethereum.org>
@gumb0 gumb0 merged commit 6ce4e70 into ethereum:master Sep 26, 2023
1 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants