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

Blockchain: add genesisState Option Test Case #2020

Closed
holgerd77 opened this issue Jul 6, 2022 · 4 comments
Closed

Blockchain: add genesisState Option Test Case #2020

holgerd77 opened this issue Jul 6, 2022 · 4 comments

Comments

@holgerd77
Copy link
Member

The new genesisState option (not to be confused with the genesisState() method) introduced in #1916 for using a custom genesis state is still completely untested (apart from one direct use case in Client cli.ts). This needs some relatively urgent test case addition(s) in the Blockchain package to see if this option is a) working properly and b) can be used in a consistent way.

The format to pass in a custom genesis state has remained the same I guess than it has been in Common and a test case should be similar to the test removed in Common customChains.spec.ts I would assume.

We should minimally do this before the final releases, especially to be safe on b).

Once the test case is ready it should then be checked/tested if such a Blockchain instance - containing a modified custom genesis state - can then be used in the VM - by passing in the Blockchain instance - to run code on top of a modified genesis state.

Eventually this is also worth yet another test case, since this is an important scenario and we should be really sure that this is working and can be used in a practical way.

Will also cc in @ryanio here in case I am missing something obvious (e.g. test case is there but I overlooked it).

@holgerd77
Copy link
Member Author

Update: it would then also be nice to add a respective example in the respective VM README section (to be updated along #2016 before).

@g11tech g11tech self-assigned this Jul 6, 2022
@holgerd77
Copy link
Member Author

@g11tech can I assume that this one is (completely) outdated? 🤔 Or has this "uphold itself" over all refactors? 😂

@g11tech
Copy link
Contributor

g11tech commented Jul 10, 2023

@g11tech can I assume that this one is (completely) outdated? thinking Or has this "uphold itself" over all refactors? joy

lol,

yea i think there are enough tests covering the usage of custom genesis state

@holgerd77
Copy link
Member Author

Oh, thanks, that was an extensive answer!!! 🙏 😅

Ok, will close then. 🙂

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

No branches or pull requests

2 participants