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

Add criterion benches and restructure testdata #259

Merged
merged 1 commit into from
Oct 21, 2024

Conversation

JoseSK999
Copy link
Contributor

@JoseSK999 JoseSK999 commented Oct 18, 2024

Added criterion as a dev-dependency in floresta-chain and a few initial benchmarks. Also moved the testdata directory to the root of floresta-chain, I think it is more organized and visible there.

Finally, in chain_state.rs I created a setup_test_chain function in the unit tests, and we avoid calling get_best_block a second time in the check_chain_integrity method (get both values directly instead of calling it two times).

@Davidson-Souza
Copy link
Collaborator

ACK e5243d8

@Davidson-Souza Davidson-Souza merged commit 655ce8d into vinteumorg:master Oct 21, 2024
6 checks passed
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