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

Added usage of sealEngine to blockchain tests #373

Merged
merged 1 commit into from
Oct 23, 2018

Conversation

holgerd77
Copy link
Member

This adds the usage of the new sealEngine parameter from the new blockchain test format (see this PR ethereum/tests#533 over on the tests repo).

This should both fix currently failing blockchain tests due to not comparing to PoW-related params (mixHash, see this comment e.g.) any more and at the same time significantly speed up blockchain test runs (we might be able to directly include these in the normal CI runs now, will do another blockchain test run after merge for getting the execution time).

@holgerd77 holgerd77 force-pushed the update-blockchain-test-format branch from 4b2a165 to 34e29a9 Compare October 23, 2018 11:47
@coveralls
Copy link

Coverage Status

Coverage remained the same at 91.677% when pulling 34e29a9 on update-blockchain-test-format into e24909e on master.

@holgerd77
Copy link
Member Author

Since this is fixing the before-failing blockchain test runs and is not touching production code, I will merge.

@holgerd77 holgerd77 merged commit 09906e0 into master Oct 23, 2018
@holgerd77 holgerd77 deleted the update-blockchain-test-format branch October 23, 2018 11:52
@holgerd77
Copy link
Member Author

Have also re-activated the temporarily deactivated exemplary blockchain tests runs (test_gas_limit, test_total_difficulty) as being mandatory for CI again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants