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

Remove redundant tests that have no preconsensus or consensus #203

Open
6 tasks
GalRogozinski opened this issue Apr 30, 2023 · 0 comments · May be fixed by #506
Open
6 tasks

Remove redundant tests that have no preconsensus or consensus #203

GalRogozinski opened this issue Apr 30, 2023 · 0 comments · May be fixed by #506
Labels

Comments

@GalRogozinski
Copy link
Contributor

GalRogozinski commented Apr 30, 2023

Description

Validator Registration doesn't have consensus or post consensus
Attester and Sync committees don't have preconsensus...

Yet they appear in various spectests that test different consensus/pre-consensus on them.
It should suffice to have a single spectest that proves that they don't run consensus/preconsenus.

In addition there are some duplicate tests.

Please review the following files:

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