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

Tests/generate mystique #437

Merged
merged 5 commits into from
Jan 10, 2022
Merged

Tests/generate mystique #437

merged 5 commits into from
Jan 10, 2022

Conversation

meowsbits
Copy link
Contributor

Generates and includes GeneralStateTests for the ETC_Mystique test configuration.
This configuration is spec'd here: https://github.com/ethereumclassic/ECIPs/pull/452/files#diff-d5496c21eb89f07be78b9bbf3399a610defa66676e25cc5f803eef279a873b35.

I ran the generation program twice to provide some redundancy given potentially nondeterministic conditions (at least Go's un-guaranteed map ordering). The first of these is used for this pull request; the latter should be discarded after review.

https://github.com/etclabscore/core-geth/tree/tests/generate-mystique
https://github.com/etclabscore/core-geth/tree/tests/generate-mystique2

Date: 2022-01-03 07:14:42-08:00
Signed-off-by: meows <[email protected]>
Date: 2022-01-03 07:41:53-08:00
Signed-off-by: meows <[email protected]>
…ls on ETC_Mystique suite

Date: 2022-01-03 13:32:29-08:00
Signed-off-by: meows <[email protected]>
…ulti-geth config

This configuration cannot be represented
in the multi-geth/ethereum data type.

The state tests are skipped when running
the core-geth-ONLY tests that convert
data types and rerun tests to ensure
interoperability.

Date: 2022-01-04 05:06:54-08:00
Signed-off-by: meows <[email protected]>
Date: 2022-01-04 05:21:59-08:00
Signed-off-by: meows <[email protected]>
@meowsbits meowsbits requested a review from ziogaschr January 10, 2022 15:02
Copy link
Member

@ziogaschr ziogaschr left a comment

Choose a reason for hiding this comment

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

LGTM

@meowsbits meowsbits merged commit 2005145 into master Jan 10, 2022
@meowsbits meowsbits deleted the tests/generate-mystique branch January 10, 2022 19:11
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