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

statetest: Load tests from multiple dirs/files #512

Merged
merged 3 commits into from
Oct 25, 2022

Conversation

JSzymanskiJS
Copy link
Contributor

@JSzymanskiJS JSzymanskiJS commented Oct 5, 2022

Allow passing multiple paths to evmone-statetest to load tests from.

Closes #510

@chfast
Copy link
Member

chfast commented Oct 20, 2022

Please rebase this PR.

@JSzymanskiJS JSzymanskiJS force-pushed the statetests_multiple_dirs branch from 96048fc to be0c533 Compare October 21, 2022 09:26
@chfast chfast changed the base branch from statetests_multiple_dirs to master October 21, 2022 09:50
@chfast chfast force-pushed the statetests_multiple_dirs branch 2 times, most recently from 51161af to f5645f8 Compare October 25, 2022 13:29
@codecov
Copy link

codecov bot commented Oct 25, 2022

Codecov Report

Merging #512 (cd018c4) into master (f67b7da) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #512   +/-   ##
=======================================
  Coverage   99.06%   99.06%           
=======================================
  Files          54       54           
  Lines        5344     5352    +8     
=======================================
+ Hits         5294     5302    +8     
  Misses         50       50           
Flag Coverage Δ
blockchaintests 77.78% <ø> (ø)
statetests 7.63% <77.27%> (+0.07%) ⬆️
unittests 97.19% <100.00%> (+<0.01%) ⬆️

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

Impacted Files Coverage Δ
test/statetest/statetest.cpp 100.00% <100.00%> (ø)

@chfast chfast force-pushed the statetests_multiple_dirs branch from d79d44b to ef4eb75 Compare October 25, 2022 16:23
@chfast chfast changed the title Statetests multiple dirs statetest: Load tests from multiple dirs/files Oct 25, 2022
JSzymanskiJS and others added 3 commits October 25, 2022 18:40
Allow passing multiple paths to evmone-statetest to load tests from.

Co-authored-by: Paweł Bylica <pawel@hepcolgum.band>
@chfast chfast force-pushed the statetests_multiple_dirs branch from ef4eb75 to cd018c4 Compare October 25, 2022 16:40
@chfast chfast merged commit 6937f5d into ethereum:master Oct 25, 2022
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.

statetest: Load tests from multiple dirs / files
2 participants