Skip to content

Model testing #641

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

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from
Open

Model testing #641

wants to merge 3 commits into from

Conversation

krl
Copy link

@krl krl commented Jun 5, 2025

📝 Summary

This is an example on how to make model tests with quickcheck.

The keys being tested are of a fixed size, which seems to be a hidden requirement of at least the v2 structure. (see test crash_example_v2 included, but ignored)

💡 Motivation and Context

Expanding test coverage with model testing.

Todo

Actually also test the calculated root hashes.

@krl krl requested review from dvush and ZanCorDX as code owners June 5, 2025 18:14
@krl krl force-pushed the model-testing branch from b5d05fe to 6fc33d2 Compare June 5, 2025 18:15
@krl
Copy link
Author

krl commented Jun 5, 2025

Related to and fixes #638

@krl krl force-pushed the model-testing branch from 6fc33d2 to e195e06 Compare June 5, 2025 18:31
@dvush
Copy link
Contributor

dvush commented Jun 5, 2025

Have you seen tests module? It's already doing that kind of test. It's using proptest though.

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