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

Transition tests to use PropCheck.jl #5

Open
Seelengrab opened this issue Jun 7, 2023 · 0 comments
Open

Transition tests to use PropCheck.jl #5

Seelengrab opened this issue Jun 7, 2023 · 0 comments
Labels
tests Related to tests

Comments

@Seelengrab
Copy link
Owner

Seelengrab commented Jun 7, 2023

The current tests are an ad-hoc construction of property based testing, but they are by no means exhaustive. It would be good to utilize PropCheck.jl for this, so that we can increase the trust in the correctness of the code.

This is particularly important for testing larger field sizes, as those are currently quite untested (in spite of coverage claiming that they are).

#=
Every struct created by FieldFlags is a multiple of 8 bit, so (for now) we're only going
to test 7, 8 and 9 fields, to catch obvious off-by-one errors. That should probably
cover multiples thereof as well, which (hopefully) is sufficient for now.
TODO: Use PropCheck.jl to generate structs for testing instead, once PropCheck.jl has a reliable interface
=#

@Seelengrab Seelengrab added enhancement tests Related to tests and removed enhancement labels Jun 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Related to tests
Projects
None yet
Development

No branches or pull requests

1 participant