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 property tests for math functions #88

Open
tomrijnbeek opened this issue Jul 3, 2017 · 2 comments
Open

Add property tests for math functions #88

tomrijnbeek opened this issue Jul 3, 2017 · 2 comments

Comments

@tomrijnbeek
Copy link
Member

We talked about adding property tests, and the math functions seem to be prime candidates for that. I think it would be great if we could sanity check the Mathf and Random functions at least.

@paulcscharf
Copy link
Member

By implementing the two lines of each method again in the test and comparing the results? :/

Not even sure how to test Random, except to make sure that only valid numbers are returned, but I don't wanna write statistical tests to check if the result is actually random.

@tomrijnbeek
Copy link
Member Author

I was indeed thinking about just sanity checking that the output lies within the supplied range.

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